I'm quite fond of doing VPATH builds, i.e. building outside the source
tree. This also catches lots of little Makefile bugs, so here's a small
patch for one of them (replacing an explicit reference to thread.c with
a reference to it as the first prerequsite of the rule makes make look
for it in the place where it was found (the source tree) rather than in
the build tree. (using GNU make 3.79.1)

John Gray

Include 'IPv4', 'IPv6', or 'Unix' in socket-creation failure messages,
in hopes of soothing fears of those with partial IPv6 support.  Still an
open question whether we should report EAFNOSUPPORT errors at all,
though.

Cope with NoData message from backend.  Needed for case where
PQexecParams is used with a statement that doesn't return data.

Marginal hacks to move some processing out of the per-client-message
processing loop; avoids extra overhead when using parse/bind/execute
messages instead of single Query message.

Change some frequently-reached elog(DEBUG...) calls to ereport(DEBUG...)
for speed reasons.  (ereport falls out much more quickly when no output
is needed than elog does.)

Avoid unnecessary work when stats collection is disabled.  Tighten
search loop in pgstat_initstats.  Per report from Gavin Sherry.

Applied patch from Oliver Jowett to clean up the jdbc regression test build

 Modified Files:
  jdbc/build.xml
  jdbc/org/postgresql/test/jdbc3/Jdbc3TestSuite.java

Rewriter and planner should use only resno, not resname, to identify
target columns in INSERT and UPDATE targetlists.  Don't rely on resname
to be accurate in ruleutils, either.  This fixes bug reported by
Donald Fraser, in which renaming a column referenced in a rule did not
work very well.

Add:

> * Prevent unneeded quoting in psql \d output using fmtId()

Applied patch from Oliver Jowett to better handle invalid input for getArray
(no longer throw an index out of range exception)

 Modified Files:
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSet.java

Applied patch from Oliver Jowett to clean up some aditional warning messages
from ant.

 Modified Files:
  jdbc/build.xml

Applied patch from Oliver Jewett to fix a deprecation in newer versions of ant

 Modified Files:
  jdbc/build.xml

Applied patch from Oliver Jewett to clean up the testing README file

 Modified Files:
  jdbc/org/postgresql/test/README

Applied patch by Oliver Jowett to clean up some exception handling

 Modified Files:
  jdbc/org/postgresql/core/QueryExecutor.java
  jdbc/org/postgresql/util/PSQLException.java

Applied patch from Oliver Jowett to improve a buffer sizing.

 Modified Files:
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java

Applied patch from Kim Ho to fix a regression against a 7.4 server.  The result
of transaction isolation level changed from uppercase to lower case between 7.3 and 7.4.  In testing, a regression was also fixed in this area when talking to
a 7.2 server due to changes in how notice messages are processed in the current
code.

 Modified Files:
  jdbc/build.xml jdbc/org/postgresql/core/BaseStatement.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java

Code cleanup inspired by recent resname bug report (doesn't fix the bug
yet, though).  Avoid using nth() to fetch tlist entries; provide a
common routine get_tle_by_resno() to search a tlist for a particular
resno.  This replaces a couple uses of nth() and a dozen hand-coded
search loops.  Also, replace a few uses of nth(length-1, list) with
llast().

Do not link in libwsock32 on non-win32 platforms.  Improve grammar
of thread-safety comments.

Translation updates

Repair potential deadlock created by recent changes to recycle btree
index pages: when _bt_getbuf asks the FSM for a free index page, it is
possible (and, in some cases, even moderately likely) that the answer
will be the same page that _bt_split is trying to split.  _bt_getbuf
already knew that the returned page might not be free, but it wasn't
prepared for the possibility that even trying to lock the page could
be problematic.  Fix by doing a conditional rather than unconditional
grab of the page lock.

Make various links point to specific places instead of entire parts
(doubtless these are hangovers from the old separate-books days).

Request verbose error message output in bug reports.

Update docs for 7.4 array features and polymorphic functions.
This is Joe Conway's patch of 7-Aug plus further editorializing
of my own.

I get the following warning when I compile src/bin/pg_id/pg_id.c:

gcc -pipe -g -Wall -Wmissing-prototypes -Wmissing-declarations
-I../../../src/include   -c -o pg_id.o pg_id.c -MMD
pg_id.c: In function `main':
pg_id.c:35: warning: unused variable `optarg'

The attached trivial patch fixes the warning by removing the variable.

Neil Conway

This makes psql use the new pg_get_viewdef function.  It also puts one
extra line break in to improve formatting.

Christopher Kings-Lynne

Another pgindent run with updated typedefs.

pgindent fix for new typedefs.

Add:

> * Allow clients to query WITH HOLD cursors and prepared statements
503d503
<

Remove simple_prompt from /contrib C files, now that it is in /port.

They had the old versions anyway.

Avoid malloc(0).  Although standard mallocs seem not to object,
some debugging malloc packages do.

Fix nasty little order-of-operations bug in _SPI_cursor_operation.
Per report from Mendola Gaetano.

Update:

<  o -Implement SQLDA
>  o Implement SQLDA

Remove, no in /port.

Update.

Update items.

Update as done:

< * Allow easy display of usernames in a group
> * -Allow easy display of usernames in a group
88,89d87
< * -Delay resolution of array expression type so assignment coercion
<   can be performed on empty array expressions (Joe)
94c92,94
<  o Support construction of array result values in expressions
>  o -Support construction of array result values in expressions (Joe)
>  o Delay resolution of array expression type so assignment coercion
>    can be performed on empty array expressions (Joe)
148c148
< * Allow LIKE indexing optimization for non-ASCII locales
> * -Allow LIKE indexing optimization for non-ASCII locales using special index
173c173
< * Return proper effected tuple count from complex commands [return]
> * -Return proper effected tuple count from complex commands [return]
236c236
<  o Allow SHOW of non-modifiable variables, like pg_controldata
>   o -Allow SHOW of some non-modifiable variables, like pg_controldata
257a258
>  o Add capability to create and call PROCEDURES
272c273
<
> * Allow psql \du to show groups, and add \dg for groups
424c425
< * Improve Subplan list handling
> * -Improve Subplan list handling

Not needed anymore.

Done:

<  o Allow array declarations and other data types in PL/PgSQL DECLARE
>  o -Allow array declarations and other data types in PL/PgSQL DECLARE
254c254
<  o Allow PL/PgSQL to support array element assignment
>  o -Allow PL/PgSQL to support array element assignment (Joe)

Remove:

<  o Add PL/PgSQL PROCEDURES that can return multiple values

sprompt not needed in libpq.

Suppress unused-variable warnings when building without Asserts.

Conversion functions must be STRICT to prevent them from getting null inputs.

Fix null checking of type decimal datums without indicator in Informix
mode.

from Dave Cramer

Be a little bit more careful about using sqlstate, in case libpq returned
NULL for it.

Translation updates by Dennis Björklund

Remove postgres_ext.h inclusion -- not necessary.

Move simple_prompt()/sprompt.c into /port.

Makefile cleanup.

Cleanup.

Typo cleanup.

Makefile cleanup.

More thread.c cleanup.

More threading cleanups.

more thread.c consistency.

threads.c -> thread.c, be consistent.

Update done items:

< * Allow elog() to return error codes, module name, file name, line
<   number, not just messages (Peter E)
< * Add error codes (Peter E)
< * Make error messages more consistent [error]
> * -Allow elog() to return error codes, module name, file name, line
>   number, not just messages (Tom)
> * -Add error codes (Tom)
> * -Make error messages more consistent
40c40
< * Add GUC log_statement_and_duration to print statement and >= min duration
> * -Add GUC log_statement_and_duration to print statement and >= min duration
84c84
< * Allow current datestyle to restrict dates;  prevent month/day swapping
> * -Allow current datestyle to restrict dates;  prevent month/day swapping
86c86
< * Prevent month/day swapping of ISO dates to make invalid dates valid
> * -Prevent month/day swapping of ISO dates to make invalid dates valid
88c88
< * Delay resolution of array expression type so assignment coercion
> * -Delay resolution of array expression type so assignment coercion
93c93
<  o Allow arrays to be ORDER'ed
>  o -Allow arrays to be ORDER'ed
116c116
< * Remove Cyrillic recode support
> * -Remove Cyrillic recode support
144c144
< * Certain indexes will not shrink, e.g. indexes on ever-increasing
> * -Certain indexes will not shrink, e.g. indexes on ever-increasing
185c185
< * Have SELECT '13 minutes'::interval display zero seconds in ISO datestyle
> * -Have SELECT '13 minutes'::interval display zero seconds in ISO datestyle
196c196
<  o -Add ALTER TABLE tab SET WITHOUT OIDS (Rod)
>  o --Add ALTER TABLE tab SET WITHOUT OIDS (Rod)
221c221
<    stored in the backend
>    stored in the backend (Gavin)
235c235
<  o Allow EXPLAIN EXECUTE to see prepared plans
>  o -Allow EXPLAIN EXECUTE to see prepared plans
241d240
<  o Add untrusted version of plpython
265c264
< * Allow psql to show transaction status if backend protocol changes made
> * -Allow psql to show transaction status if backend protocol changes made
272,273c271,272
< * Modify pg_get_triggerdef() to take a boolean to pretty-print,
<  and use that as part of pg_dump along with psql
> * -Modify pg_get_triggerdef() to take a boolean to pretty-print,
>   and use that as part of pg_dump along with psql
292c291
<  o Add SQLSTATE
>  o -Add SQLSTATE
296c295
<  o Implement SQLDA (do we really need this?)
>  o -Implement SQLDA
364d362
< * Allow binding query args over FE/BE protocol
378c376,377
< * Provide automatic running of vacuum in the background (Tom) [vacuum]
> * Provide automatic running of vacuum in the background in backend
>   rather than in /contrib [vacuum]
427c426
< * Allow Subplans to use efficient joins(hash, merge) with upper variable
> * -Allow Subplans to use efficient joins(hash, merge) with upper variable
429c428
< * Allow merge and hash joins on expressions not just simple variables (Tom)
> * -Allow merge and hash joins on expressions not just simple variables (Tom)
474c473
< * Remove memory/file descriptor freeing befor elog(ERROR)  (Bruce)
> * Remove memory/file descriptor freeing before ereport(ERROR)  (Bruce)
489,490c488,489
<  o Show transaction status in psql
<  o Allow binding of query parameters, support for prepared queries
>  o -Show transaction status in psql
>  o -Allow binding of query parameters, support for prepared queries
492,494c491,493
<  o Remove hard-coded limits on user/db/password names
<  o Remove unused elements of startup packet (unused, tty, passlength)
<  o Fix COPY/fastpath protocol?
>  o -Remove hard-coded limits on user/db/password names
>  o -Remove unused elements of startup packet (unused, tty, passlength)
>  o -Fix COPY/fastpath protocol
496,497c495
<  o Replication support?
<  o Error codes
>  o -Error codes
500d497
<  o ecpg improvements?
503c500
<  o Report server version number, database encoding, client encoding
>  o -Report server version number, database encoding, client encoding

Remove space between function name and opening paren in pg_dump, for
clarity.

No need for TODO.detail error.

Fix floating-point timestamp comparisons to not go nuts if NaN is
encountered; per bug report from Christian van der Leeden 8/7/03.
Also, adjust larger/smaller routines (MAX/MIN) to share code with
comparisons for timestamp, interval, timetz.

Clean up function header.

Fix typo.

Don't assume that struct option is available just because we can find a
getopt_long().  This is more or less the same problem as we saw earlier
with getaddrinfo() and struct addrinfo, and for the same reason: random
user-added libraries might contain the subroutine, but there's no
guarantee we will find the matching header files.

SCO_ACCEPT_BUG code didn't get updated for new SockAddr struct definition.

create_unique_plan() should not discard existing output columns of the
subplan it starts with, as they may be needed at upper join levels.
See comments added to code for the non-obvious reason why.  Per bug report
from Robert Creager.

Don't use HAVE_STRUCT_ADDRINFO as a guide to whether netdb.h defines
macros like AI_NUMERICHOST; instead, test the macros individually.
Should fix recent reports of trouble on AIX and Unixware.

Make table column type TEXT.

Add iteration option to thread test program.

Update pgindent readme.

Remove unnecessary use of multiple cat processes to feed psql; this
reduces the number of concurrent processes launched during parallel
regression tests, possibly avoiding failures such as exceeding a user's
max number of processes.  This essentially completes the reversion
of 1.27->1.28.

Move dollar sign items to object section.

Update typedef names for pgindent 7.4.

Update ecpg thread testing program to be more automated.

markup fixes

Include postgres_ext.h instead of postgres_fe.h.  This allows ecpg
output C files to proper compile again.

Noticed broken markup in ecpg.sgml. Attached patch fixes it. Please
apply.

Joe Conway

Add -lm for ecpg/pgtypeslib/Makefile link so -lm isn't required for ecpg
compiles.

Sometimes the third time is the charm.  Third try to fix the sql injection
vulnerability.  This fix completely removes the ability (hack) of being able
to bind a list of values in an in clause.  It was demonstrated that by allowing
that functionality you open up the possibility for certain types of
sql injection attacks.  The previous fix attempts all focused on preventing
the insertion of additional sql statements (the semi-colon problem:
xxx; any new sql statement here).  But that still left the ability to
change the where clause on the current statement or perform a subselect
which can circumvent applicaiton security logic and/or allow you to call
any stored function.

 Modified Files:
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java

Update release items from Tom.

Applied doc patch for the jdbc docs submitted by Nic Ferrier for functionality
he supplied a few months ago, but didn't get around to docing until now.  And
he also added some doc for calling stored functions in general from jdbc that was missing.

 Modified Files:
  sgml/jdbc.sgml

Add:

>  o Add ALTER DATABASE ... OWNER TO newowner

Rename fields of DestReceiver to avoid collisions with (ill-considered)
macros in some platforms' sys/socket.h.

Adopt a random backoff algorithm for sleep delays when waiting for a
spinlock.  Per recent pghackers discussion.

Fix compiler-detected problem for Alphas: it seems strlen returns
something wider than int on that platform.  Also, remove bogus
assumption that sizeof("INT_MAX") has something to do with the maximum
number of digits in an int.

OpenFTS vs Tsearch2 comment

Add documentation about ts_debug

Add ts_debug function for debugging  configurations

Applied patch from kho@redhat.com to fix a problem with trying to use a fetch
when a cursor wasn't being used.

 Modified Files:
  jdbc/org/postgresql/jdbc1/AbstractJdbc1ResultSet.java

Add ecpg thread testing file.

Improve documentation of ParseDateTime().  Reorder tests to prevent
writing one more value into return arrays than will fit.  This is
potentially a stack smash, though I do not think it is a problem in
current uses of the routine, since a failure return causes elog anyway.

Fix several places where fractional-second inputs were misprocessed
in HAVE_INT64_TIMESTAMP cases, including two potential stack smashes
when more than six fractional digits were supplied.  Per bug report
from Philipp Reisner.

Modify:

Prevent interval from supressing ':00' seconds display

Remove Tom mention on array items.

Add mention of :00 seconds.

Update thread wording.

Modify tsearch2 to be Oleg,Teodor

Add tsearch2 mention.

Add Philip Yarra to threads item, remove my name.

Update release items from Joe Conway.

can't mix and match .gz and .bz2 in here ... won't build

seeing if building bz2 distributions actually works ...