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>

Oops, don't forget to remove the CASSERT=true line

Have CASSERT enabled/disabled via configure, and passed through config.h
instead of as a -D

Patch for Solaris 2.5 from: adrian@waltham.harvard.net

Various minor HP related patches from:

Morten Kjeldgaard <mok@monster.kemi.aau.dk>

A small fix, where default: condition in case had not 'break;'...not required,
but, IMHO, cleaner

From: Thomas Lockhart <Thomas.G.Lockhart@jpl.nasa.gov>
Subject: [HACKERS] Money integration patches

Here are patches to integrate the money data type. I have included
some math and aggregate functions and have made the locale support optional
by #ifdef USE_LOCALE bracketing of functions.

Modules affected are:
builtins.h.patch
cash.c.patch
cash.h.patch
main.c.patch
pg_aggregate.h.patch
pg_operator.h.patch
pg_proc.h.patch
pg_type.h.patch

I changed the data type to be pass-by-reference rather than by-value
to pave the way for a larger internal representation (64-bit ints?).
Also, I changed the tabbing of cash.c and cash.h to match most of
the other Postgres source code files (4 space indent, 8 spaces == 1 tab).

The locale stuff should be tested under another convention (Russian?)
but I don't know what the correct results should be so perhaps someone
else can give them a try. Will update docs and regression tests in
the next few days.

2. The file /usr/local/pgsql/src/backend/lipq/pgcomprim.c has two
invalid macro definitions, the compiler complains about:

"pqcomprim.c", line 48.9: 1506-275 (S) Unexpected text ';' ignored.
"pqcomprim.c", line 61.9: 1506-275 (S) Unexpected text ';' ignored.

The ';' terminating the macro definition ntoh_s(n) on line 27 and
ntoh_l(n) on line 28 should be removed.

Pointed out by: Olaf Mittelstaedt <MSTAEDT@va-sigi.va.fh-ulm.de>

Fix up the -Wno-error problem for non-gcc compilers

Pointed out by many many ppl

Add ccsym to tools to determine OS/compiler specific symbols

Change BSD44_derived to __FreeBSD__ in numutils.c (need to know what
NetBSD is referred to as...someone?)

Fixes for [] escaping pointed out by Adrian

scripts required by configure to bypass its inabilty to use []'s inside of
configure ([] have special means to m4 *sigh*)

Make sure that Makefile.${PORTNAME} is link'd and that teh port/*.h
file that has moved to include is also link'd properly

Remove port specific Makefile 'targets' (ie. shared libraries) from
Makefile.global and move them to seperate 'include' makefiles

Over time, should become even more port specific:

ie. Makefile.BSD44_derived should be broken down into netbsd/freebsd
specific ports

More more of the include files under the include directory

remove the port related .h files from the top level directory.

Moved to include/port

Fix detection of install vs installbsd

Submitted by: adrian@waltham.harvard.net

Make sure postgres.h is included before we start checking #ifdef's for other
header files.

Pointed out by: Edmund Mergl <E.Mergl@bawue.de>

Get rid of the .sql files out of the Makefile

Pointed out by: afc@teri.superlink.net

Major cleanup of Install instructions

Provided by: adrian@waltham.harvard.net

modifications to pg_dump towards supporting dumping of ACLs (doesn't work yet!)

modification to c.h so that bool isn't typedef'd under __cplusplus

Change BACKEND to CFLAGS

Finish removing the TEST_MAIN stuff, which was mean for standalone
testing

Fix #include "cash.h" to be #include <utils/cash.h>

Remove the TEST_MAIN stuff at the top...

Update man page, listing new \z value

Add a \z command to psql that lists off grant/revoke permissions

- if someone can pick a better \? for this, plesae let me know...all
  the good ones seem taken :(

Copy indexkeys in _copyIndexPath.

Much improved configure that integrates the build script right into it

Submitted by: adrian@waltham.harvard.net

remove the old regression test files.  have copies saved in my directory here,
but it gets rid of the temptation to modify the old source files :)

Add in D'Arcy's cash code

pg_proc.h still needs modifying, but this gets it in there so that we can
get around any compiler bugs.  Will try and get the pg_proc.h entries done
up later tonight...

From: "Martin S. Utesch" <utesch@aut.tu-freiberg.de>
Subject: Re: [HACKERS] GEQO and views (rules)

Oke, this was caused by a classic bug :-/
I thougth, root->base_relation_list_ could be
represented as relid string 1-2-3-4- etc.

Instead, in case of views, the count of relids doesn't start with "1" but
maybe 4-5-6- etc . :-(

GEQO patch follows ... views are now all right.

GNUmakefile.in - remove backend/utils/Gen_fmgrtab.sh on distclean
varlena.c - part of Thomas' most recent patch

Now we have #define _CPU_INDEX_PAGE_WEIGHT_ 0.033 (/* CPU-index-to-page cost
weighting factor */) in addition to
#define _CPU_PAGE_WEIGHT_  0.065 (/* CPU-heap-to-page cost weighting factor
*/).

getattnvals(): if attnvals in pg_attribute is 0 then use
ATTNVALS_SCALE/reltuples (instead of reltuples).

1. Enable to have different _CPU_PAGE_WEIGHT_ for heap and index.
2. PageWeights are variables now.
3. Fixed using ceil((double)selec*indextuples) as estimation
for expected heap pages: ceil((double)selec*relpages) now.

Fix (hack) IndexSelectivity():
use sum(npages)/((nkeys == 1) ? 1 : nkeys + 1) as expected index page
estimation for multi-key quals - instead of sum(npages).
In old code npages for x > 10 and x < 20 is twice as for x > 10 - cool ?

Slight mods to explain (or try to) the new regression tests

Fix btabstimecmp ().

Get rid of queries.source...its all in the sql directory.

regress.sh modified to get rid of queries.sql tests, as they are
performed vis sql/*.sql

More splits and cleanups...

Its starting to actually take shape and look as expected...

More splits and cleanups...

Add in support so that build will at least guess which template file
should be used :)

remove create.{source,sql} as they are now down in the 'sql' directory
partially split

Change Postgres95 to PostgreSQL

The whole file needs updating, but will work on that after finishing
with the splits

More splits of the regression tests in order to make them more
user-friendly (and more useful)

Purge out tests/expected that are now in sql/expected subdirectories from
'master' file

Commit mods to regress.sh so that split out tests are run...look forward
to finding out how to do a proper redirect to continue visual cleanup :)

Again, add more tests

Add more expected.out results

There are the broken out 'sql' queries from queries.source

tests allows us to have a 'for...done' loop inside of regress.sh for
both doing the tests, and determining fail/ok results

These are the broken down 'expected.output' files created so far