Uppercase SQL keywords in query for clarity.

This simple patch fixes a typo, ensure pg_config is removed in win32
and correct some splitted message mistake.

Euler Taveira de Oliveira

Fix pg_dumpall to use double-quote for Win32 arguments.

Luiz K. Matsumura

The Win32 (mingw) build of libpq seems to have lost it's SSL libs from
the link step resulting in a build failure. The attached patch fixes
that.

Dave Page

Use #define instead of typedef for plperl because perl defines
uid_t/gid_t too.

Update win32ver.rc to 8,0,000,000.  Mention it needs updating in
RELEASE_CHANGES.

Improve consistency of win32 version sed script.

Fix for Win32 file version sed macro, per report from Dave Page.

Here is a patch to fix win32 ssl builds. Summary of changes:

* Links with -leay32 and -lssleay32 instead of crypto and ssl. On win32,
"crypto and ssl" is only used for static linking.

* Initializes SSL in the backend and not just in the postmaster. We
cannot pass the SSL context from the postmaster through the parameter
file, because it contains function pointers.

* Split one error check in be-secure.c. Previously we could not tell
which of three calls actually failed. The previous code also returned
incorrect error messages if SSL_accept() failed - that function needs to
use SSL_get_error() on the return value, can't just use the error queue.

* Since the win32 implementation uses non-blocking sockets "behind the
scenes" in order to deliver signals correctly, implements a version of
SSL_accept() that can handle this. Also, add a wait function in case
SSL_read or SSL_write() needs more data.

Magnus Hagander

Allow plpython to build on Win32.

Magnus Hagander

Adjustment to test on unix domain socket variable for pg_hba.conf
default settings, rather than just Win32.

Remove pg_hba.conf 'local' line for Win32 because it doesn't support unix domain
connections.

Andrew Dunstan

please find attached an alternate submission which addresses open item
"make pgxs install by default". It is up to the committers to chose.

(1) there is only one "install" target. no more "install-all-headers".
    it simplifies/changes several makefiles.

(2) the documentation reflects the change.

(3) a minor fix on pgxs to use a nicer patch without a double slash.

Fabien Coelho

Slightly rearrange psql startup code so that we read psqlrc before printing
the startup banner. This allows "\set QUIET on" in psqlrc to do what the
user probably intended. Patch from Sean Chittenden, editorializing from
Neil Conway.

Add Win32 version info to client binaries.

Magnus Hagander

Add distclean target to pgevent.

Another tightening of Win32 typedefs.

Fix typo in recent Win32 typedef change.

Remove unused Win32 defines, convert them to typdefs, per suggestion
from Peter.

'::' is no longer interpreted as a variable in a prepare statement.

Pickup fix from upstream OpenBSD sources: mark a read-only local array as
"static" to reduce size of generated code slightly.

License cleanup: crypt.c and qsort.c to latest NetBSD CVS sources, to
pickup license clarification (3-clause BSD is now used). Add license
terms to memcmp.c (also from NetBSD), which previously had none.
Finally, pickup an upstream fix to crypt.c (const-ify some arrays).

Refer to ll_to_earth() function by its actual name.

Remove arithmetic operators on the 1-byte-char datatype, as per proposals
made several times in the past.  Add coercion functions between "char"
and integer so that a workaround is possible if needed.

Initdb forced.

Correct the volatility labeling of ten timestamp-related functions,
per discussion from Friday.  initdb not forced in this commit but I intend
to do that later.

PortalRun must guard against the possibility that the portal it's
running contains VACUUM or a similar command that will internally start
and commit transactions.  In such a case, the original caller values of
CurrentMemoryContext and CurrentResourceOwner will point to objects that
will be destroyed by the internal commit.  We must restore these pointers
to point to the newly-manufactured transaction context and resource owner,
rather than possibly pointing to deleted memory.
Also tweak xact.c so that AbortTransaction and AbortSubTransaction
forcibly restore a sane value for CurrentResourceOwner, much as they
have always done for CurrentMemoryContext.  I'm not certain this is
necessary but I'm feeling paranoid today.
Responds to Sean Chittenden's bug report of 4-Oct.

Make sure no libs use libpgport by removing it from the link line.

Fix MinGW link failure with new libpgport processing.

Update comment on int64 printf speciifications for MinGW:

# MinGW uses '%I64d', though gcc throws an warning with -Wall,
# while '%lld' doesn't generate a warning, but doesn't work.

Readd -DFRONTEND to client makefiles because it still affects win32.h
defines.

Add CVS URL for docs.

Fix warnings from crypt.c compile.

Comment out unused variable.

Suppress getppid test on WIN32, per Dave Page.

Detect overflow in integer arithmetic operators (integer, smallint, and
bigint variants).  Clean up some inconsistencies in error message wording.
Fix scanint8 to allow trailing whitespace in INT64_MIN case.  Update
int8-exp-three-digits.out, which seems to have been ignored by the last
couple of people to modify the int8 regression test, and remove
int8-exp-three-digits-win32.out which is thereby exposed as redundant.

Make libpgport be front-end only and make libpgport_srv be a backend
library that uses palloc, ereport, etc.  This simplifies the makefiles
for client applications.

Fix error in setseed() docs: "int32" is not a (SQL) type, but "integer" is.

Document that the semicolon following the final "END" in a function body
is optional.

Translation update

Translation updates

Add regression test inspired by Sebastian Böck.

Clean up handling of inherited-table update queries, per bug report
from Sebastian Böck.  The fix involves being more consistent about
when rangetable entries are copied or modified.  Someday we really
need to fix this stuff to not scribble on its input data structures
in the first place...

Update Farsi FAQ in ASCII.

Update length from 75 to 79.

Convert pg_stat_get_backend_idset to use the existing SRF support.
This seems the cleanest way of fixing its lack of a shutdown callback,
which was preventing it from working correctly in a query that didn't
run it to completion.  Per bug report from Szima GÄbor.

fmgr_security_definer had better do a PG_TRY to ensure the outer userid
is restored on error exit.

Adjust postmaster to recognize that a lockfile containing its parent's PID
must be stale.  Tweak example startup scripts to not use pg_ctl but launch
the postmaster directly, thereby ensuring that only the postmaster's direct
parent shell will be a postgres-owned process.  In combination these should
fix the longstanding problem of the postmaster sometimes refusing to start
during reboot because it thinks the old lockfile is not stale.

Don't assume PQdb() will return a valid result from a failed connection.

Don't assume PQdb() will return a valid result from a failed connection.

Fallout from changing index locking rules: we can reduce the strength
of locking used by REINDEX.  REINDEX needs only ShareLock on the parent
table, same as CREATE INDEX, plus an exclusive lock on the specific index
being processed.

Code review for NOWAIT patch: downgrade NOWAIT from fully reserved keyword
to unreserved keyword, use ereport not elog, assign a separate error code
for 'could not obtain lock' so that applications will be able to detect
that case cleanly.

Remove DROPs from contrib object creation scripts, per Dave Page.

Use 'override' when appending text to LDFLAGS, so the build doesn't break
if the user has defined LDFLAGS themselves.

Remove more traces of libpgtcl from the source tree. Also, make some
semi-related SGML cleanup. Original patch from ljb220@mindspring.com,
additional cleanup by Neil Conway.

Adjust index locking rules as per my proposal of earlier today.  You
now are supposed to take some kind of lock on an index whenever you
are going to access the index contents, rather than relying only on a
lock on the parent table.

Add variant regression file to handle machines that convert -1e-700
to minus zero rather than zero; this includes at least Mac OS X 10.3
and Solaris 2.8.

Remove unnecessary use of index_open just to get the index name.

isinf.c needs <float.h> on some platforms, per Darcy Buskermolen.

Add a note suggesting that users should use the newer version of pg_dump
to perform upgrades, and cleanup some nearby text. Patch from Robert
Treat, editorializing by Neil Conway.

Add more index entries for reference pages related to prepare queries.
From Alvaro Herrera, editorializing by Neil Conway.

Update Brazilian FAQ.

Euler Taveira de Oliveira

Add an index entry for tablespaces. From Kris Jurka.

Come to think of it, functions in FROM have the same syntactic restriction
as CREATE INDEX did, and can be fixed the same way, for another small
improvement in usability and reduction in grammar size.

Split out everything that looks like a function call from c_expr into
a separate production func_expr.  This allows us to accept all these
variants in the backwards-compatible syntax for creating a functional
index; which beats documenting exactly which things work and which don't.
Interestingly, it also seems to make the generated state machine a little
bit smaller.

Add Farsi version of FAQ.

Mahmoud Taghizadeh

Minor adjustments to improve the accuracy of our computation of required
shared memory size.

Minor documentation improvement.

Adjust obsolete comment about memory needs for lock table.

Arrange to preallocate all required space for the buffer and FSM hash
tables in shared memory.  This ensures that overflow of the lock table
creates no long-lasting problems.  Per discussion with Merlin Moncure.

Code review for recent dbsize changes.  Fix some thinkos, enforce coding
style and message style standards, improve documentation.

 An include path got missed out of the patch - see the attached fix.
This also adds debug build support to src/bin/psql/win32.mak.

Dave Page

Fix typo in docs.

Remove duplicate PQclear(res) operations leading to double free() and
subsequent core dump.  It looks like at one time DBLINK_RES_ERROR_AS_NOTICE
didn't include a PQclear, but now it does and so these other ones are
duplicate.

Revert patch that removed BUFSIZ usage.  The memory has to hold the
structures plus pointers used by the structure.

Small WIN32_CLIENT_ONLY cleanup for new build patch.

Remove use of large BUFSIZ for buffers and use the proper struct sizes.
This greatly helps threaded libpq programs.

Add new macro as shorthand for MS VC and Borland C++:

+ #if   defined(_MSC_VER) || defined(__BORLANDC__)
+ #define       WIN32_CLIENT_ONLY
+ #endif

More MS VC and BCC psql compile cleanups.

More Win32 cleanups for Dave Page.

Make adjustment for MS VC and BCC compile of psql, from Dave Page.

Improve MS VC builds for psql by handlling flex properly and prevent
rename prototype conflict.

Translation update

Translation updates

Translation update

New translations

Update docs for Borland CC build.

Remove inclusion of windows.h now that it is included in c.h, per idea
from Peter.

Add:

>
> * Consider automatic caching of queries at various levels:
>  o Parsed query tree
>  o Query execute plan
>  o Query results

Remove completed items:

< Last updated: Sat Sep 25 21:33:44 EDT 2004
> Last updated: Mon Sep 27 10:15:31 EDT 2004
13,19d12
< Remove items before beta?
<
< Urgent
< ======
<
< * -Point-in-time data recovery using backup and write-ahead log
< * -Create native Win32 port
25d17
< * -Incremental backups
28d19
< * -Allow configuration files to be specified in a different directory
32,34d22
< * -Add the concept of dataspaces/tablespaces (Gavin)
< * -Allow logging of only data definition(DDL), or DDL and modification statements
< * -Allow log lines to include session-level information, like database and user
54d41
< * -Allow external interfaces to extend the GUC variable set
126d112
< * -Change factorial to return a numeric (Gavin)
141,142d126
< * -Allow pg_dump to dump sequences using NO_MAXVALUE and NO_MINVALUE
< * -Prevent whole-row references from leaking memory, e.g. SELECT COUNT(tab.*)
147d130
< * -Make LENGTH() of CHAR() not count trailing spaces
150d132
< * -Support composite types as table columns
198,200d179
< * -Prevent mismatch of frontend/backend encodings from converting bytea
<   data from being interpreted as encoded strings
< * -Fix upper()/lower() to work for multibyte encodings
217d195
< * -Order duplicate index entries on creation by ctid for faster heap lookups
242d219
< * -Be smarter about insertion of already-ordered data into btree index
265,266d241
< * -Allow SELECT * FROM tab WHERE int2col = 4 to use int2col index, int8,
<   float4, numeric/decimal too
282d256
< * -Allow command blocks to ignore certain types of errors
302,303d275
< * -Allow savepoints / nested transactions (Alvaro)
< * -Use nested transactions to prevent syntax errors from aborting a transaction
306,307d277
< * -Prevent COMMENT ON DATABASE from using a database name
< * -Add NO WAIT LOCKs
325,326d294
< * -COMMENT ON [ CAST | CONVERSION | OPERATOR CLASS | LARGE OBJECT | LANGUAGE ]
<   (Christopher)
334d301
< * -Allow more ISOLATION LEVELS to be accepted
347d313
< * -Add GUC setting to make created tables default to WITHOUT OIDS
365,369d330
<  o -ALTER TABLE ADD COLUMN does not honor DEFAULT and non-CHECK CONSTRAINT
<  o -ALTER TABLE ADD COLUMN column DEFAULT should fill existing
<    rows with DEFAULT value
<  o -ALTER TABLE ADD COLUMN column SERIAL doesn't create sequence because
<           of the item above
371,373d331
<  o -Allow ALTER TABLE to modify column lengths and change to binary
<    compatible types
<  o -Add ALTER DATABASE ... OWNER TO newowner
390,393d347
<  o -Add ALTER DOMAIN, AGGREGATE, CONVERSION ... OWNER TO
<  o -Add ALTER SEQUENCE ... OWNER TO
<  o -Add ALTER INDEX that works just like ALTER TABLE already does
<    on an index
404d357
<  o -Add ALTER TABLE table SET WITHOUT CLUSTER (Christopher)
411d363
<  o -Allow dump/load of CSV format
464d415
<  o -Allow Java server-side programming
473d423
<  o -Allow PL/pgSQL parameters to be specified by name and type during definition
493,495d442
< * -Allow psql \du to show users, and add \dg for groups
< * -Have psql \dn show only visible temp schemas using current_schemas()
< * -Have psql '\i ~/<tab><tab>' actually load files it displays from home dir
509,511d455
<  o -Allow pg_dump to dump CREATE CONVERSION (Christopher)
<  o -Make pg_restore continue after errors, so it acts more like pg_dump
<    scripts
545d488
<  o -Implement SET DESCRIPTOR
592,596d534
< * -Have AFTER triggers execute after the appropriate SQL statement in a
<   function, not at the end of the function
< * -Print table names with constraint names in error messages, or make constraint
<   names unique within a schema
< * -Issue NOTICE if foreign key data requires costly test to match primary key
614,615d551
< * -Use dependency information to dump data in proper order
< * -Have pg_dump -c clear the database using dependency information
694,695d629
< * -Provide automatic running of vacuum in the background in backend
<   rather than in /contrib (Matthew)
828d761
< * -Use background process to write dirty shared buffers to disk
843d775
< * -Change representation of whole-tuple parameters to functions
850,852d781
< * -Add checks for fclose() failure (Tom)
< * -Change CVS ID to PostgreSQL
< * -Exit postmaster if postgresql.conf can not be opened

Synced parser.
Removed Oracle transaction syntax to fix shift/reduce error.

Addition to documentation about cvsup. From Andrew Dunstan, minor SGML
tweak by Neil Conway.

Remove references to the ODBC driver from the main source tree. From Kris
Jurka.

A few minor list-related cleanups:

(1) Replace while loop with the new forboth() construct in
parser/analyze.c

(2) Replace lcons() with lappend() in SearchCatCacheList(). Since these
now have the same performance, there is no reason to prefer lcons() in
this case, and using lappend() leads to cleaner code.

(3) Improve the name of the second parameter to for_each_cell()

Code cleanup: don't bother casting the argument to pfree() to void *
from another pointer type. Per C89, this is unnecessary, and it is common
practice throughout the rest of the tree anyway.

update for beta3, and update Copyright date to 2004

Rewrite ReadArrayStr() to avoid O(N^2) behavior on large strings,
and hopefully improve code clarity while at it.  One intentional
semantics change: a backslashed space will not be treated as removable
trailing whitespace, as the prior coding would do.  ISTM that if it
wouldn't be considered removable leading whitespace, it shouldn't be
stripped at the end either.

Fix misstatement in docs: CREATE TABLE AS is now part of the standard.

Fix multiple breakages in our support for SSL certificates.

Use ULL not LL in UINT64CONST.

Here is a patch that adds the version info from libpq.rc to the DLL
build in mingw. The MSVC build already did this, but it was not linked
into the mingw one.

This is not the same as the versioninfo patch that's in the queue.
Please apply this one before beta-3 if at all possible.

Magnus Hagander