This is a quick patch to fix a crash in pgquery_dictresult() introduced
recently. I just ran into it while running a set of python test scripts,
and I'm not sure who the normal maintainer is for interfaces/python.

John Nield

The following small patch provides a couple of minor updates (against
CVS HEAD):

Amended "strings" regression test. TOAST tests now insert two values
with storage set to "external", to exercise properly the TOAST slice
routines which fetch only a subset of the chunks.

Changed now-misleading comment on AlterTableCreateToastTable in
tablecmds.c, because both columns of the index on a toast table are now
used.

John Gray

Add mention of foreign key dependency and SERIAL in 7.2 data restores.

Note can probably be removed after a couple of releases.

Rod Taylor

Dump ALTER DATABASE/USER ... SET ...

Remove:

< * Remove PGPASSWORD because it is insecure on some OS's, in 7.4

Update info about relkind and pg_type entries for composite-types patch.

Allow FOR UPDATE to appear after LIMIT/OFFSET to match MySQL syntax and as
a more logical ordering.

Fix compile warning.

Flush output streams before calling pg_dump subprocess.

Add:

>  o Add SET SCHEMA

Disable use of -c and -a together in pg_dump;  they don't make sense together.

Bring comments back in sync with code.

Throw error on pg_atoi(''), regression adjustments.

Enable locale, so case conversion (identifier processing) and number
formatting (\timing) works correctly.  Change "Total time" to "Time"
since there is nothing that "total" refers to.  Remove non-multibyte
code.

Reimplement pg_dumpall in C.  Currently no change in functionality,
except that it's more robust, reconnects less often, and is NLS'ed.

Add '+' mention in \? help.  More clean of \? to do.

Make change for pg_locks table.

The attached patch implements the password packet length sanity check
(using an elog(LOG) ), as well as includes a few more comment fixes.

Neil Conway

Add:

> * Allow free space map to be auto-sized or warn when it is too small

Add error code emails.

Add most of Neil Conway's cleanups.

More connection timeout cleanups.

Cleanup of libpq connection timeout code.

Add:

> * Allow REINDEX to rebuild all indexes, remove /contrib/reindex
>  o Allow CLUSTER to cluster all tables, remove clusterdb

Add fsm sizes.

PREPARE/EXECUTE statements.  Patch by Neil Conway, some kibitzing
from Tom Lane.

Done:

>  o -Cluster all tables at once using pg_index.indisclustered set during

Add to mmap discussion.

This patch updates the lock listing code to use Joe Conway's new
anonymous return type SRF code. It gets rid of the superflous
'pg_locks_result' that Bruce/Tom had commented on. Otherwise, no
changes in functionality.

Neil Conway

This is a 2 line patch to src/interfaces/perl5/GNUMakefile that fixes
the 'override CPPFLAGS' to include the source directory during compile,
and makes the install target look in the proper place for the man page.

Changes are only required when building outside the source directory.

J. R. Nield

This patches replaces a few more usages of strcpy() and sprintf() when
copying into a fixed-size buffer (in this case, a buffer of
NAMEDATALEN bytes). AFAICT nothing to worry about here, but worth
fixing anyway...

Neil Conway

Cleanup of SGML

I attach a little patch to make CLUSTER set and reset the indisclustered
bit on the indexes.

I also attach clusterdb and clusterdb.sgml; both of them are blatant
rips of vacuumdb and vacuumdb.sgml, but get the job done.  Please review
them, as I'm probably making a lot of mistakes with SGML and I can't
compile it here.

vacuumdb itself is not very comfortable to use when the databases have
passwords, because it has to connect once for each table (I can probably
make it connect only once for each database; should I?).  Because of
this I added a mention of PGPASSWORDFILE in the documentation, but I
don't know if that is the correct place for that.

Alvaro Herrera

Add to skip list in check_guc

Comment cleanup.

Script cleanups.

Clean up script.

Add check_guc utility to compare guc.c and postgresql.conf.sample.

Fix units in postgresql.conf.

Add OS X link line example for external functions.

Add discussion of pre-write pages to WAL.

Add:

>   partial page writes [wal]

Add:

> * Add CORRESPONDING BY to UNION/INTERSECT/EXCEPT

Add units to postgresql.conf file.

Remove:

<  o Add plsh server-side shell language (Peter E)

*** empty log message ***

Add to selfjoin DELETE todo.

Add:

> * Allow DELETE to handle table aliases for self-joins [delete]

Add delete selfjoin item.

Add info on disabled triggers.

Add:

> * Allow triggers to be disabled [trigger]

Increase WIDTH_THRESHOLD from 256 to 1K.  This addresses recent observation
that ANALYZE would not gather any stats for a CHAR(255) column.  I still
think a width threshold is appropriate for the reasons mentioned in the
code, but we can loosen it at least.

Modify array operations to include array's element type OID in the
array header, and to compute sizing and alignment of array elements
the same way normal tuple access operations do --- viz, using the
tupmacs.h macros att_addlength and att_align.  This makes the world
safe for arrays of cstrings or intervals, and should make it much
easier to write array-type-polymorphic functions; as examples see
the cleanups of array_out and contrib/array_iterator.  By Joe Conway
and Tom Lane.

Add 'return' file.

Add return tuple count item to TODO.

Add:

> * Return proper effected tuple count from complex commands [return]

Add anther sequential scan email.

Add mention of 1 terrabyte databases.

*** empty log message ***

Add raw file discussion to performance TODO.detail.

Done:

> * -Disallow TRUNCATE on tables that are involved in referential constraints

Add to tablespaces discussion.

Add to tablespaces file.

Add Bob Devine's name to the optimizer README.

Add file describing error message format.

Add:

> * Make error messages more consistent [error]

Clean up comments to be careful about the distinction between variable-
width types and varlena types, since with the introduction of CSTRING as
a more-or-less-real type, these concepts aren't identical.  I've tried to
use varlena consistently to denote datatypes with typlen = -1, ie, they
have a length word and are potentially TOASTable; while the term variable
width covers both varlena and cstring (and, perhaps, someday other types
with other rules for computing the actual width).  No code changes in this
commit except for renaming a couple macros.

Update docs for 7.2.2 release info.

Update Japanese FAQ, from Jun Kuwamura

The cstring datatype can now be copied, passed around, etc.  The typlen
value '-2' is used to indicate a variable-width type whose width is
computed as strlen(datum)+1.  Everything that looks at typlen is updated
except for array support, which Joe Conway is working on; at the moment
it wouldn't work to try to create an array of cstring.

Done:

> * -SELECT cash_out(2) crashes because of opaque

Enhancements to how queries with bind values are stored internally and sent to
the server.  Previously we allocated a new String object for the entire final
query we were sending to the database.  If you had a big query, or especially
if you had large bind values you ended up with essentially two copies in memory.
This change will reuse the existing objects and therefore should take 1/2 the
memory it does today for a given query.  This restructuring will also allow
in the future the ability to stream bytea data to the server instead of the current approach of pulling it all into memory.
I also fixed a test that was failing on a 7.2 database.
Also renamed some internal variables and some minor cleanup.

 Modified Files:
  jdbc/org/postgresql/core/QueryExecutor.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSet.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/test/jdbc2/DatabaseMetaDataTest.java

Further cleanup around the edges of OPAQUE/pseudotype changes.  Correct
the declarations of some index access method support functions.  Support
SQL functions returning VOID.

Adapt for SRF(Set Returning Function).

Not sure how I fixed it the first time, but here's a fix for another
instance in which the docs mention that 'NAMEDATALEN == 32".

Neil Conway

Add space.k>

Add:

> * Allow xlog directory location to be specified during initdb, perhaps
>   using symlinks

Patch resolve ERROR problem for non-goog query_txt.

Teodor Sigaev

This patch updates the CREATE OPERATOR sgml docs for the new default
settings of NAMEDATALEN. I looked through the docs for other
references to NAMEDATALEN, but this is the only one I could find.

Neil Conway

Update Russian FAQ.

Add:

> * Add GUC parameter to print queries that generate errors

Very minor copy-editing.

Add note that explains that \df omits functions it thinks are I/O functions.

Update reference to value of NAMEDATALEN.  Someone needs to troll the
docs and find the other obsolete statements that no doubt lurk.

Update contrib regression tests for OPAQUE datatype changes.

Since the plpgsql test has been part of the standard regression tests
for a good long while, I see no reason to maintain this original copy.

Transpose info from src/pl/plpgsql/test/README into standard regression test file.

Mark 7.2.2 as newest release.

Update FAQ.

Remove all traces of the ODBC driver, which is now on GBorg as the psqlodbc
project ...

Fix bit rot in pg_dump's ability to dump from 7.2 and 7.1 servers.

Fixed-size buffer in dumpClasses is not big enough anymore given the
addition of a column list clause to the COPY command.  Spotted by
Martin Renters.

Improve wording of upgrade section.

Code review for recent TRUNCATE changes.  Tighten relation-kind check,
tighten foreign-key check (a self-reference should not prevent TRUNCATE),
improve error message, cause a relation's TOAST table to be truncated
along with the relation.

Commit updated repeat() patch, from  Neil Conway

Fix for documention:

>>" It's also possible to select no escape character by writing ESCAPE ''.
>>In this case there is no way to turn off the special meaning of
>>underscore and percent signs in the pattern."

Joe Conway

This patch should fix the problem. Doesn't include my previous patch
for repeat(). Again, somewhat off-the-cuff, so I might have missed
something...

test=# select lpad('xxxxx',1431655765,'yyyyyyyyyyyyyyyy');
ERROR:  Requested length too large
test=# select rpad('xxxxx',1431655765,'yyyyyyyyyyyyyyyy');
ERROR:  Requested length too large

(That's on a Unicode DB, haven't tested other encodings but AFAICT
this fix should still work.)

Neil Conway

repeat() fix:

> Neil Conway <neilc@samurai.com> writes:
> > +   /* Check for integer overflow */
> > +   if (tlen / slen != count)
> > +           elog(ERROR, "Requested buffer is too large.");
>
> What about slen == 0?

Good point -- that wouldn't cause incorrect results or a security
problem, but it would reject input that we should really accept.

Revised patch is attached.

Neil Conway

> > I had great difficulty in finding how to change the search path, so here
> > is a patch to add some cross-referencing.

Oliver Elphick

# Disallow TRUNCATE on tables that are involved in referential
constraints

The issue with finding and removing foreign key constraints is no longer
an issue, so please apply the attached.

It does NOT check for rules or on delete triggers (old style foreign
keys) as those are difficult to deal with (remove, truncate, re-add).

Rod Taylor