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

Print 'Group' as name of Group plan.

Changes for GROUP BY func_results:
AddGroupAttrToTlist() is not called from anywhere now.

Changes for GROUP BY func_results.

Now we can GROUP BY func_results.

Fix for 'SET var_name TO var_value': var_name already defined.

Put resdom into GroupClause (GROUP BY func_results)

Check for attributeList is NULL in ConstructTupleDescriptor ().

Submitted by Raymond Toy.

From: Thomas Lockhart <Thomas.G.Lockhart@jpl.nasa.gov>
Subject: [HACKERS] Fix for European dates

This apparently fixes the European date reading problem reported
by several (European) bleeding edge adopters. I tried a few test
cases and it doesn't break the non-EuroDate cases in my test suite.

Move YACC and YFLAGS into the template files

Clean up the .sample files...comment out all sample entries except for
the localhost one

This commit represents a clean compile with the new templates under
FreeBSD

The Makefile(s) have all been cleaned up such that there is a single
LDFLAGS vs LD_ADD or LDADD or LDFLAGS or LDFLAGS_BE.  The Makefile(s)
should be alot more straightforward then they were before...and
consistent

Got AROPT setting backwards...correct it

Add distclean to target listing

Again, needs float.h

needs float.h for DBL_MIN under FreeBSD

Remove some files that were inadvertantly created
Clean up format of linux-elf

One helluva mess.

Further extended Makefile.global/build/configure so that we can
have a 'template' file for each OS (and each version of OS, as in BSDi)
which is used as much as possible to generate Makefile.global

Any future ports should look at using the template file as a basis,
before moving over to Makefile.global.

This will most probably break alot of the ports, atho I've tried to
be very neat about it...

Install os.h when we install the other headers...

Pointed out by: System Administrator <sysadmin@sba.miami.edu>

More modifications to make building more interactive:

Allow installer to change DEF_PGPORT
Allow installer to disable HBA

Remove code associated with !ACLGROUP_PATCH, and appropriate #ifdef's

Various improvements to reduce questions :)

Remove USE_LOCALE from Makefile.global.in
Add USE_LOCALE to build/configure/config.h

Add check for BUILDRUN in configure to make sure that build is run before
configure

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

I sent in changes previously and they were rejected because they didn't
follow ANSI spec.  Here is the input part of the changes again.  Even
though it allows more flexibility for inputting different formats, it
is also backwards compatible with the standard version.  I have also
not changed the output format so it will still output the ANSI forms.
Is this acceptable to everyone?