Bug: backend crashes in btbeginscan()->btrescan()->_bt_orderkeys()
when btree used in innerscan with run-time key which value
passed by pointer.

Fix: keys ordering stuff moved to _bt_first().

Pointed by Thomas Lockhart.

Add substitution from output/.. to input/.. to match templates from inputs.

Update to work with new regression tests and directory structure.

New test data for aggregate functions.

Fix old bug in _hash_first() for scan without keys:
if 1st bucket chain is empty then need to continue scan in the rest buckets.

More informative NOTICEs in tuple checks.

creation for postgresql-6.1

Reorder tests and move aggregate table data to data/.

Added initial set of expected outputs for new regression testing.
Modified a few tests to match results on RedHat Linux/gcc for v6.1beta.

SET geqo TO ON|OFF

#ifdef GEQO
bool _use_geqo_ = true;
#else
bool _use_geqo_ = false;
#endif

_use_geqo_ is settable via SET var TO ...

Fix GroupBy: enable functions over aggregates and GroupBy-ed fields
in target list.

Fix configure so that installbsd gets -c also

From: Lee Roth <roth@tpusa.com>

Add a netbsd template file and update linux-elf and .similar

From: "D'Arcy J.M. Cain" <darcy@druid.net>
Subject: [HACKERS] Inputting money

I notice that I have to put single quotes around money amounts if there
is a decimal point in the value.  I appears to be happening because there
is something changing things like "123.45" to "123.450000" and the code
has a problem with that.  There may be a better way to fix this but here
is a simple change to cash.c that lets it accept trailing zeroes.

Provide template for the misc tests which contain path names.

Fix path name templates.

Update SQL query results for lists of operators and functions.
Add a little info on count().

Change mixed-case routines to lower-case for case-insensitive SQL.
Add mixed-case #define synonyms to avoid changing more source code.
Add comparison operators for boolean.
Add aggregate min() and max() for datetime and timespan.

Change mixed-case routines to lower-case in pg_proc.h
Add comparison operators for boolean.
Add aggregate min() and max() for datetime and timespan.
Fix duplicate OID in pg_proc.h

Change mixed-case routines to lower-case if referenced in pg_proc.h

Change mixed-case routines to lower-case if referenced in pg_proc.h
Add comparison operators to boolean and smaller/larger operators to datetime
and timespan. Fix int4 overflow math problem in timespan comparison operators.

Modify references to function manager to use lower-case calls.

From: Keith Parks <emkxp01@mtcc.demon.co.uk>
Subject: [PATCHES] to make regress.sh shell friendly to echo.

Hi,

I needed to make the following change to regress.sh to make it more
shell friendly.

The Solaris /bin/sh, and others, use \c to supress the newline.

There, I'll leave this alone until Thomas catchs up *grin*

split out a bunch more tests from misc.source so that the tester knows
what's being tested :)

Change Postgres95 to PostgreSQL

Pointed out by: Christopher Hicks <chicks@chicks.net>

Alot of regression test fixes, mainly to compensate for moving
the DROP TABLE calls from the destroy.sql file to the 'types' .sql files,
so that they are self-contained

btree_index, hash_index and misc all fail as there seems to be missing
a 'misc.out' expected file...have asked Thomas for one...

Get first four tests to pass:

=============== destroying old regression database... =================
=============== creating new regression database...   =================
=============== running regression queries...         =================
create_function_1 .. ok
create_type .. ok
create_table .. ok
create_function_2 .. ok

Make these self-contained tests...they are testing types, so the tables
that are created should only exist as long as the test requires them...

things are just toooooo spread around

Trying to continue breaking down the large files into smaller tests so that
debugging is a little (well, alot!) easier

clean up misc.source and create *individual test* files ... trying to move
away from one big massive confusing file

*shrug*  I don't know anymore...

*shrug*

Massive regression test patches from Thomas *woo hoo!*

A few changes to reflect things properly suggested by David

Reverse check for TR so that trbsd gets checked for first

Pointed out by: adrian@waltham.harvard.net

Add a results directory and a Makefile so that 'make clean' cleans it out

Missed an -I directive

More misc patches from Thomas for regression tests and linux templates

Clean out/up some files that are causing me great headaches since I didn't
do this completely last time and Thomas is creating patches on files that
aren't supposed to exist :(

clean up the bin/*/Makefiles...up version.h to v6.1 instead of v6.0

Remove bin/Makefile.global since it wasn't actually *doing* anything
that Makefile.global hadn't already done

Fix for bootstrap.c problem (F_* values missing from fmgr.h)

More timezone patches by Thomas:

Here are patches which should help fix timezone problems in the
datetime and abstime code. Also, I repatched varlena.c to add in
some comments and a little error checking on top of Vadim's earlier
repairs. There are slight mods to the circle data type to have the
distance operator between circles measure the distance between
closest points rather than between centers.

From: "Pedro J. Lobo" <pjlobo@euitt.upm.es>
Subject: [PATCHES] Patches for compiling 6.1 on Digital Unix 3.2c

Attached to this message are the patches I needed to compile 6.1 cleanly
under Digital Unix 3.2c with DEC cc.

I hope these are the last ones. At least, the number of files needing a
patch has decreased noticeably since I sent my previous patches. Nice work
:-)

One of the patches is a bug fix, but I'm including it here anyway.

With these patches applied, the beast seems to work properly. However,
I've done only some preliminary tests. More on this later (but hopefully
before the April 30 deadline... :-)

Fixes:

postgres backend processes end up as so called zombies. It seems that
only Linux a.out (libc.4.6.27) systems are affected.

By:

Wolfgang Roth <roth@statistik.uni-mannheim.de>

Various bug fixes based on a bug report submitted by
Doug Neuhauser <doug@seismo.berkeley.edu> on April 15th

Added comments about FASTBUILD.
Added #define BTREE_VERSION_1.

 * Hack for non-functional btree npages estimation:
     * npages = index_pages * selectivity_of_1st_attr_clause(s)

Have to copyObject((void*)clauseinfo) for a relation'
joininfo (in add_join_clause_info_to_rels()) : clause may be
commutted later.

Bug (yet unfixed but not fatal) found: Re-moving index clauses from the
nestloop's join clauses doesn't work in some cases:
         *    1. fix_indxqual_references may change varattno-s in
         *       inner_indxqual;
         *    2. clauses may be commuted

Save Var' order in index_outerjoin_references().

Add innercost to result in cost_hashjoin.

_bt_endpoint fixed: set currentItemData to Invalid if no result.

Enable to set _cpu_page_wight_ & _cpu_index_page_wight_ via
SET cost_heap(cost_index) TO ...

#define _CPU_PAGE_WEIGHT_: 0.065 --> 0.033
#define _CPU_INDEX_PAGE_WEIGHT_: 0.033 --> 0.017

Set SRCDIR=..

Remove chunk from regression tests about regress.out vs expected.out...the
tests now let you know if it is ok/failed :)

-Wno-error should only be enabled for gcc compilers, it seems

Somewhat backwards #ifdef's for HAVE_GETRUSAGE

Various patches not commited, plus a linux/sparc patch from
Satoshi Ishikawa <power@sekine.densi.gifu-u.ac.jp>

Change HAVE_RUSAGE to HAVE_GETRUSAGE, which is the more appropriate

Vacuum help changed (ANALYZE).

Analyze data only if specified and specified attrs only.

1. SHOW/RESET var fixed.
2. vacuum() call changed (ANALYZE).

1. SHOW/RESET var fixed.
2. New VACUUM syntax (ANALYZE ...).

1. VariableShowStmt and VariableResetStmt added.
2. VacuumStmt changed (for VACUUM ANALYZE...).

There were too many comments.

To: Thomas Lockhart <Thomas.G.Lockhart@jpl.nasa.gov>
Subject: Re: [PATCHES] SET DateStyle patches

On Tue, 22 Apr 1997, Thomas Lockhart wrote:

> Some more patches! These (try to) finish implementing SET variable TO value
> for "DateStyle" (changed the name from simply "date" to be more descriptive).
> This is based on code from Martin and Bruce (?), which was easy to modify.
> The syntax is
>
> SET DateStyle TO 'iso'
> SET DateStyle TO 'postgres'
> SET DateStyle TO 'sql'
> SET DateStyle TO 'european'
> SET DateStyle TO 'noneuropean'
> SET DateStyle TO 'us'         (same as "noneuropean")
> SET DateStyle TO 'default'    (current same as "postgres,us")
>
> ("european" is just compared for the first 4 characters, and "noneuropean"
> is compared for the first 7 to allow less typing).
>
> Multiple arguments are allowed, so SET datestyle TO 'sql,euro' is valid.
>
> My mods also try to implement "SHOW variable" and "RESET variable", but
> that part just core dumps at the moment. I would guess that my errors
> are obvious to someone who knows what they are doing with the parser stuff,
> so if someone (Bruce and/or Martin??) could have it do the right thing
> we will have a more complete set of what we need.
>
> Also, I would like to have a floating point precision global variable to
> implement "SET precision TO 10" and perhaps "SET precision TO 10,2" for
> float8 and float4, but I don't know how to do that for integer types rather
> than strings. If someone is fixing the SHOW and RESET code, perhaps they can
> add some hooks for me to do the floats while they are at it.
>
> I've left some remnants of variable structures in the source code which
> I did not use in the interests of getting something working for v6.1.
> We'll have time to clean things up for the next release...

From: System Administrator <igor@sba.miami.edu>
Subject: [PORTS] Configure for DEC-Alpha

Configure script properly detects alpha-dec-osf4.0 machine, but
sets a default GENERIC template for it. I modified tempplate/.similar to
add alpha-dec-osf4.0=alpha. Then configure properly set the template to
alpha.

Add fcvt() as a check to configure so that we can get rid of the BSD44_derived
type check in numutils.c:ftoa()

Pointed out by: "Martin J. Laubach" <mjl@emsi.priv.at>

From: adrian@waltham.harvard.net
Subject: [PATCHES] Patch for configure.in to not ask for CASSERT

The following patch defaults to CASSERT, so it doesn't ask you.  You can
still use --enable-cassert and --disable-cassert to do it explicitly.

Default: disabled

From: Thomas Lockhart <Thomas.G.Lockhart@jpl.nasa.gov>
Subject: [PATCHES] date/time timezone patches (mail bounced?)

Here are some hacks to get timezone behavior for the various time
data types to be compatible with v6.0. Although we have some hooks
already installed to get timezone info from the client to the
server, it still isn't clear if that can correctly transfer enough
timezone info to make the behavior the same as if timezone info
were derived from the server as is now the case. We certainly
won't resolve it in a day, so I think we are stuck with server-only
timezones for v6.1.

Major patch from Thomas Lockhart <Thomas.G.Lockhart@jpl.nasa.gov>

OK, here are a passel of patches for the geometric data types.
These add a "circle" data type, new operators and functions
for the existing data types, and change the default formats
for some of the existing types to make them consistant with
each other. Current formatting conventions (e.g. compatible
with v6.0 to allow dump/reload) are supported, but the new
conventions should be an improvement and we can eventually
drop the old conventions entirely.

For example, there are two kinds of paths (connected line segments),
open and closed, and the old format was

'(1,2,1,2,3,4)' for a closed path with two points (1,2) and (3,4)
'(0,2,1,2,3,4)' for an open path with two points (1,2) and (3,4)

Pretty arcane, huh? The new format for paths is

'((1,2),(3,4))' for a closed path with two points (1,2) and (3,4)
'[(1,2),(3,4)]' for an open path with two points (1,2) and (3,4)

For polygons, the old convention is

'(0,4,2,0,4,3)' for a triangle with points at (0,0),(4,4), and (2,3)

and the new convention is

'((0,0),(4,4),(2,3))' for a triangle with points at (0,0),(4,4), and (2,3)

Other data types which are also represented as lists of points
(e.g. boxes, line segments, and polygons) have similar representations
(they surround each point with parens).

For v6.1, any format which can be interpreted as the old style format
is decoded as such; we can remove that backwards compatibility but ugly
convention for v7.0. This will allow dump/reloads from v6.0.

These include some updates to the regression test files to change the test
for creating a data type from "circle" to "widget" to keep the test from
trashing the new builtin circle type.

Fix for Hash and arrays

Document --enable-cassert/--disable-cassert configure options

Fix for text_lt/text_le to avoid warnings if not def USE_LOCALE.

#include <string.h>

Fix for -Wno-error

Here is the Mismatched input/output patch for tintervals as reported over
the last week on Hackers...(A coulpe of clippings of the final
verdict are included below + the diff).

From: Wayde Nie <niew@phoenix.cis.mcmaster.ca>

Add a --with-compiler switch that allows someone to specify:

--with-compiler=xlc

Requested by: Darren King <aixssd!darrenk@abs.net>

From: adrian@waltham.harvard.net
Subject: [HACKERS] Another patch to configure.in

I heard very little in objections/approvals to defaulting some of the
parameters to configure.  Enclosed is a patch to configure.in which
removes the questions for

        PGPORT
        USE_LOCALE
        NOHBA

By default (i.e. assuming you don't put anything extra in the configure
command line), it assumes PGPORT=5432, USE_LOCAL=no and NOHBA=no (i.e.
HBA is turned on)

        --with-pgport=PGPORT_NO         Over-rides the PGPORT value
        --enable-locale                 enables USE_LOCALE
        --disable-hba                   disables HBA

Just for completeness:

        --prefix=BASEDIR                Defaults to /usr/local/pgsql
        --with-template=TEMPLATE        Defaults to asking you

Fix for -Wno-error

CommitInfoNeedsSave[buffer - 1] = 0
added to WriteBuffer(), FlushBuffer(), WriteNoReleaseBuffer().

#ifdef BTREE_BUILD_STATS enables to get executor stats for btree
building.

1. Declaration
static const char *num_word(Cash value);
moved to cash.c.
2. 'extern ' added to funcs prototypes.

Declaration
static const char *num_word(Cash value);
moved here from cash.h

PrintBufferUsage() changed to report about shared, local and direct
blocks transfferes.

Fix bttextcmp() to use unsigned char*.
#ifdef USE_LOCALE added.

From: Raymond Toy <toy@rtp.ericsson.se>
Subject: [PATCHES] 970417:  some large object patches

Two patches here, made against 970417.  Both have to do with large
objects:

        1.  lobjfuncs was not initialized in PQconnectdb.  This causes
            failure later if large objects are used.  (Someone already
            caught this error in PQsetdb.)

        2.  Postgres functions lo_import and lo_export sometimes
            produce garbage for the file names because the filename
            strings aren't always terminated by \0.  (VARDATA isn't
            necessarily null terminated.)

From: Raymond Toy <toy@rtp.ericsson.se>
Subject: [PATCHES] 970417:  two more patches for large objects

Here are two more patches:

        1.  pg_getint doesn't properly set the status flag when
            calling pqGetShort or pqGetLong.  This is required when
            accessing large objects via libpq.  This, combined with
            problem 1 above causes postgres to crash when postgres
            tries to print out the message that the status was not
            good.

        2.  ExceptionalCondition crashes when called with detail =
            NULL.  This patch prevents dereferencing the NULL.

From: "Martin J. Laubach" <mjl@CSlab.tuwien.ac.at>
Subject: [HACKERS] Patch: set date to euro/us postgres/iso/sql

  Here a patch that implements a SET date for use by the datetime
stuff. The syntax is

        SET date TO 'val[,val,...]'

  where val is us (us dates), euro (european dates), postgres,
iso or sql.

  Thomas is working on the integration in his datetime module.
I just needed to get the patch out before it went stale :)

Some slight changes to the Wisconsin Benchmark tests since postgres requires
a -D<datadir> option

The patch fixes a rare bug that may occur when one tries to vacuum a single
table. The table name is de-allocated by the CommitTransactionCommand()
in vc_init() before it is copied in VacRel.data and sometimes this causes
a SIGSEGV. My patch simply moves the strcpy before vc_init.

Submitted by Massimo Dal Zotto <dz@cs.unitn.it>.

Fix a problem with Linux where yacc is *really* bison -y

connectDB(): setsockopt (..., TCP_NODELAY, ...) added.

StreamConnection(): setsockopt (..., TCP_NODELAY, ...) added.

1. BTREE_VERSION_1: using bti_itup->t_tid as unique identifier for a given
index tuple (logical position within A LEVEL). bti_oid & bti_dummy
taken off from BTItemData.
2. Fix for multi-column indices (nbtsearch.c):
   _bt_binsrch() - for searches on internal pages having keysize <
number of attrs we point at the last item < the scankey, not at the
first item = the scankey;
   _bt_moveright() - if keysize < number of attrs we compare scankey with
_last_ item on current page to decide should we move right or
not.

BTREE_VERSION_1: using bti_itup->t_tid as unique identifier for a given
index tuple (logical position within A LEVEL). bti_oid & bti_dummy
taken off from BTItemData.

A couple of cleanups from Scott Harrison <Scott_Harrison@next.com>