Code review for pg_locks feature.  Make shmemoffset of PROCLOCK structs
available (else there's no way to interpret the list links).  Change
pg_locks view to show transaction ID locks separately from ordinary
relation locks.  Avoid showing N duplicate rows when the same lock is
held multiple times (seems unlikely that users care about exact hold
count).  Improve documentation.

*** empty log message ***

Add expected tuple descriptor to ReturnSetInfo information for table
functions, per suggestion from John Gray and Joe Conway.  Also, fix
plpgsql RETURN NEXT to verify that returned values match the expected
tupdesc.

Updates to reflect availability of autocommit option.

AUTOCOMMIT mode is now an available backend GUC variable; setting it
to false provides more SQL-spec-compliant behavior than we had before.
I am not sure that setting it false is actually a good idea yet; there
is a lot of client-side code that will probably be broken by turning
autocommit off.  But it's a start.

Loosely based on a patch by David Van Wie.

Fix for breakage of C-coded SRFs, from Joe Conway.

Add attisinherited column to pg_attribute; use it to guard against
column additions, deletions, and renames that would let a child table
get out of sync with its parent.  Patch by Alvaro Herrera, with some
kibitzing by Tom Lane.

Back out unintented tab change.

Remove pgaccess from Makefile.

Run autoconf.

Update documentation for --with-perl.

Improve wording after wal_files removal.

Complete TODO item:

* Remove wal_files postgresql.conf option because WAL files are
  now recycled

Remove pgaccess from CVS.  It has its own home now.

Re-enable perl configure tests for psql help file generation.

Back out password packet length check.

Improve wording of pre-7.3 syntax mention.

perl5 interface moved to gborg

libpgeasy moved to gborg ...

Fix off-by-one allocation error in PasswordFromFile(), per Gordon Runkle.

Add comment on old sytax for SELECT FOR UPDATE/LIMIT and COPY.

Revert _LARGEFILE64_SOURCE addition; good try but doesn't work,
at least not on HPUX 10.20, and there's no reason to think it
is needed on later versions.

intagg subdirectory was missing.

Group-manipulating code neglected to fill in array element type,
which is now required.

PL/pgSQL functions can return sets.  Neil Conway's patch, modified so
that the functionality is available to anyone via ReturnSetInfo, rather
than hard-wiring it to PL/pgSQL.

Fix a bug introduced in 7.2.

> > > > If you want to put in security restrictions that are actually useful,
> > > > where is the code to verify that PGPASSWORDFILE points at a
> > > > non-world-readable file?  That needs to be there now, not later, or
> > > > we'll have people moaning about backward compatibility when we finally
> > > > do plug that hole.

Alvaro Herrera

Sir Mordred The Traitor <mordred@s-mail.com> writes:
> Upon invoking a polygon(integer, circle) function a
> src/backend/utils/adt/geo_ops.c:circle_poly() function will gets
> called, which suffers from a buffer overflow.
>
> 2) A src/backend/adt/utils/geo_ops.c:path_encode() fails to detect a
> buffer overrun condition. It is called in multiple places, the most
> interesting are path_out() and poly_out() functions.

> 5) A src/backend/utils/adt/geo_ops.c:path_add() also fails to detect
> a simple buffer overrun.

I've attached a patch which should fix these problems.

Neil Conway

Add cube changes file.

The changes I have made are described in CHANGES. This was based on
diffs to 7.3-devel and may not be applicable to 7.2. I have included a
change covered by a previous bugfix patch I submitted (the problem with
-.1 not being accepted by cube_in). It does not include a fix for the
potential buffer overrun issue I reported for cube_yyerror in
cubeparse.y.

Bruno Wolff III

Make pg_resetxlog options parsing more standard and prepare messages for
translation.

Create "po" subdirectory before writing there.

Workaround for broken large file support on HP-UX

Prevent problem with extra-long password packets from allocating lots of
memory.

Neil Conway

This patch reserves the last superuser_reserved_connections slots for
connections by the superuser only.

This patch replaces the last patch I sent a couple of days ago.

It closes a connection that has not been authorised by a superuser if it would
leave less than the GUC variable ReservedBackends
(superuser_reserved_connections in postgres.conf) backend process slots free
in the SISeg. This differs to the first patch which only reserved the last
ReservedBackends slots in the procState array. This has made the free slot
test more expensive due to the use of a lock.

After thinking about a comment on the first patch I've also made it a fatal
error if the number of reserved slots is not less than the maximum number of
connections.

Nigel J. Andrews

please apply small patch for README.tsearch.

I've documented space usage and using CLUSTER command

Oleg Bartunov

The unit for statement_timeout is s/1000, not s/1000000

Manfred Koizar

Back out patch, that was alpha, which I think still needs just -O.

Change FreeBSD to use -O2, important for MemSet.

Adjust nodeFunctionscan.c to reset transient memory context between calls
to the table function, thus preventing memory leakage accumulation across
calls.  This means that SRFs need to be careful to distinguish permanent
and local storage; adjust code and documentation accordingly.  Patch by
Joe Conway, very minor tweaks by Tom Lane.

Push down outer qualification clauses into UNION and INTERSECT subqueries.
Per pghackers discussion from back around 1-August.

Cause REINDEX to regard TOAST tables as regular relations, not system
tables that need special defenses.  I believe this is okay even for
TOAST tables that belong to system tables.

chkpass_rout returns text so change PG_RETURN_CSTRING to PG_RETURN_TEXT_P.
This is currently a cosmetic difference but I make the change now in case
the macros diverge one day.

Remove MULTIBYTE

Remove #ifdef MULTIBYTE per hackers list discussion.

Rephrase 'Cannot insert into a view' and related messages, per
pghackers discussion around 31-Jul-02.

A few more fixes for the <replaceable>able> fiasco.

update mbregress expected files

Produce a somewhat-useful error message, namely
ERROR:  Cannot display a value of type RECORD
rather than a random integer when someone tries to SELECT a tuple
value.  Per pghackers discussion around 26-May-02.

FOUND patch was a bit over-enthusiastic: SQL commands that are not
INSERT, UPDATE, or DELETE shouldn't change FOUND.  IMHO anyway.
Also, try to make documentation a little clearer.

Remove support for version-0 FE/BE protocol, per pghackers discussion.
This breaks support for 6.2 or older client libraries.

Fix ruleutils to dump column definition lists for anonymous record types
defined in the FROM clause.  From Joe Conway, with some tweaks.

Code review for standalone composite types, query-specified composite
types, SRFs.  Not happy with memory management yet, but I'll commit these
other changes.

Improve description of tablefunc.

Initial Spanish translation from Karim Mribti <karim@inlosa.com>

Updates from Dennis Bjorklund

Break up the long usage messages in pg_dump and pg_restore and
unify the wording, to make life easier for translators.

from Dennis Bjorklund

Remove TIOGA files from CVS current;  they remain in repositiry.

This trivial patches fixes the error message returned by CREATE INDEX
when it finds an existing relation with the same name as the
to-be-created index.

Old error message:

nconway=# create table foo (a int);
CREATE TABLE
nconway=# create index foo on foo (a);
ERROR:  index named "foo" already exists

I replaced 'index' with 'relation' in the error message.

Neil Conway

backend where a statically sized buffer is written to. Most of these
should be pretty safe in practice, but it's probably better to be safe
than sorry.

I was actually looking for cases where NAMEDATALEN is assumed to be
32, but only found one. That's fixed too, as well as a few bits of
code cleanup.

Neil Conway

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.