When compiling a plpgsql trigger function, include the OID of the table
the trigger is attached to in the hashkey.  This ensures that we will
create separate compiled trees for each table the trigger is used with,
avoiding possible datatype-mismatch problems if the tables have different
rowtypes.  This is essentially the same bug recently identified in plpython
--- though plpgsql doesn't seem as prone to crash when the rowtype changes
underneath it.  But failing robustly is no substitute for just working.

Done:

> * -Add btree index support for reltime, tinterval, regproc (Tom)

Fix ARRAY[] construct so that in multidimensional case, elements can
be anything yielding an array of the proper kind, not only sub-ARRAY[]
constructs; do subscript checking at runtime not parse time.  Also,
adjust array_cat to make array || array comply with the SQL99 spec.

Joe Conway

Clean up locktable init code per recent gripe from Kurt Roeckx.
No change in behavior, but old code would have failed to detect
overrun of MAX_LOCKMODES.

Somebody forgot to include any actual documentation for ADD_MISSING_FROM.

Update documentation to reflect the fact that ORDER BY, GROUP BY, etc
are now driven by the default btree opclass, rather than assuming that
particular operator names have the needed semantics.

Fix broken markup.

Create a 'type cache'  that keeps track of the data needed for any particular
datatype by array_eq and array_cmp; use this to solve problems with memory
leaks in array indexing support.  The parser's equality_oper and ordering_oper
routines also use the cache.  Change the operator search algorithms to look
for appropriate btree or hash index opclasses, instead of assuming operators
named '<' or '=' have the right semantics.  (ORDER BY ASC/DESC now also look
at opclasses, instead of assuming '<' and '>' are the right things.)  Add
several more index opclasses so that there is no regression in functionality
for base datatypes.  initdb forced due to catalog additions.

Add to release notes.

Add:

> * Allow PREPARE of cursors

I almost forgot mark in docs "to_char(interval)" as deprecated function.
This useless routine will removed in 7.5. It's already discussed (see
hackers list archive).

Karel Zak

Add:

> * Fix upper()/lower() to work for multibyte encodings
>

This patch makes two minor fixes to the docs: (1) fixes a
spelling mistake in the PREPARE ref page (2) Makes some
English more consistent, in the ref pages for some of the
client apps (3) Adds a link to the libpq docs in the
vacuumdb ref page.

Neil Conway

There is a misstatement in the CLOSE reference page, now that we
have cursors that might outlive their creating transactions. A
patch is attached that fixes this (suggestions on better wording
are welcome).

Neil Conway

Add:

> * Allow pooled connections to query prepared queries
> * Allow pooled connections to close all open WITH HOLD cursors

I just noticed that the sample pg_hba.conf in the docs doesn't contain
any use of CIDR masks - here's a patch that adds  a couple of  sample
lines and associated comments.

Andrew Dunstan

The attached patch adds some clarification to the documentation of the
binary format read/created by COPY BINARY. It also mentions the
contrib/binarycopy module.

Lee Kindness.

Add:

> * Add ALTER DOMAIN, AGGREGATE, CONVERSION, SEQUENCE ... OWNER TO
>

Add:

> * Add TRUNCATE ... CASCADE

Add:

> * Add pg_get_acldef(), pg_get_typedefault(), and pg_get_attrdef()

(I always forget what the magic numbers 0 through 2 means for the
"syslog" option.)

By the way: The "virtual_host" parameter is a bad name for that
particular option, I think. "Virtual host" signals that PostgreSQL will
behave differently according to which IP address it's contacted (like
Apache's virtual host support which makes the web-server serve different
sites according to different criteria). A better word for the options
would be "tcpip_listen_addr" or something like that.

Troels Arvin

Add mention of sort_mem in the restore docs.

Update for 7.3 tips URL.

Add to libpq changes made by Tom for new protocol.

Not needed, already in CVS.

Add description of error style.

Add mention that Kerberos 4 isn't recommended.

Make NEED_REENTRANT_FUNC_NAMES _require_ *_r functions, and add tests to
configure to report if they are not found.

Updated the blob regression test to actually use the getBlob/getClob methods
and test them, in addition to testing the underlying LargeObject API methods.

 Modified Files:
  jdbc/build.xml jdbc/org/postgresql/test/jdbc2/BlobTest.java

Fixed improper message length for the connection termination message 'X' when
using the V3 protocol.

 Modified Files:
  jdbc1/AbstractJdbc1Connection.java

Eliminate function start_parse_fh and macro YY_INPUT

Rewrite array_cmp to not depend on deconstruct_array.  Should be a little
faster, but more importantly does not leak memory.  Still needs more work
though, per my recent note to pgsql-hackers.

Add disk rotation idea to WAL todo emails.

Minor copy-editing.

Adjustment for unixware threading.

Mark unixware as having threaded support.

Fix longstanding thinko in SSL protocol documentation: the server's
initial response is 'S', not 'Y', when it is willing to do SSL.

Add missing quotes.

Someone moved PageOutput here from common.c, but forgot to move all
the #includes it depends on.

Remove unnecessary #include's (see c.h).

Workaround for platforms that have getaddrinfo() without AI_NUMERICHOST.
We don't actually need the flag, so just #define it as zero in such cases.

Handle double-quotes correctly in user names in ACL lists.

Christopher Kings-Lynne

Document threading status.

Update to POSIX getpwuid_r() function.

Adjust for compiler options for compiler bug.

Larry Rosenman

libpq failed to cope with COPY FROM STDIN if the command was issued
via extended query protocol, because it sends Sync right after Execute
without realizing that the command to be executed is COPY.  There seems
to be no reasonable way for it to realize that, either, so the best fix
seems to be to make the backend ignore Sync during copy-in mode.  Bit of
a wart on the protocol, but little alternative.  Also, libpq must send
another Sync after terminating the COPY, if the command was issued via
Execute.

Add PQexecPrepared() and PQsendQueryPrepared() functions, to allow
libpq users to perform Bind/Execute of previously prepared statements.
Per yesterday's discussion, this offers enough performance improvement
to justify bending the 'no new features during beta' rule.

Move MemoryContextCheck() call from bottom of PostgresMain loop to
just before CommitTransactionCommand().  This is a more sensible place
to put it since commit discards a lot of contexts, and we'd not find
out about stomps affecting only transaction-local contexts.

Correct error messages

More accuracy work with MinWords option of headline function

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.