Add my name:

> * Allow savepoints / nested transactions [transactions] (Bruce)

Update regression for ALL missing from CLUSTER.

Most of the code follows the American spelling of the word, which is
"canceled", so I changed the one remaining usage of the British
spelling ("cancelled") over to the former, and updated the translation
files appropriately.

Neil Conway

Back out NetBSD/arm32 port.

Update ports list.

Fix typo and markup.

Update SGML problem in ports.

Remove ALL from CLUSTER ALL.  Use just CLUSTER.

Update ports list.

Put back error test for DECLARE CURSOR outside a transaction block ...
but do it correctly now.

Suppress compile warning, avoid possible problems with signed vs. unsigned
comparisons in recently-added CheckPointWarning code.

Re-order REINDEX options in manual for consistency.  REINDEX
DATABASE/TABLE/INDEX is the proper order.

Reduce need for palloc/pfree overhead in varstr_cmp() by using fixed-size
buffers on stack for short strings.

Revise SQL features list.

Fix CLUSTER regression failure caused by new patch, from Neil Conway

Add relcache support for system catalogs with attnotnull columns.
The code was not making TupleConstr structs for such catalogs in
several places; with the consequence that the not-null constraint
wasn't actually enforced.  With this change,
INSERT INTO pg_proc VALUES('sdf');
generates a 'Fail to add null value' error instead of a core dump.

Synced parser.

SGML markup fixes.

SGML improvements.

Neil Conway

New version attached.  The following is implemented:

- CLUSTER ALL clusters all the tables that have some index with
  indisclustered set and the calling user owns.
- CLUSTER tablename clusters the named table, using the index with
  indisclustered set.  If no index has the bit set, throws elog(ERROR).
- The multi-relation version (CLUSTER ALL) uses a multitransaction
  approach, similar to what VACUUM does.

Alvaro Herrera

Clean up psql:

Add simple ALTER DATABASE, ALTER TRIGGER, CHECK POINT, CREATE
CONVERSION, CREATE DOMAIN, CREATE LANGUAGE, DEALLOCATE, DROP CONVERSION,
DROP DOMAIN, DROP LANGUAGE, EXECUTE, PREPARE

Complete CAST in CREATE CAST and DROP CAST but doesn't suggest what
should follow.

Add many more SET / SHOW variables to the list.  Taken from SHOW ALL
output.

Complete a case sensitive search to allow \dD, \dd, \dS, \ds, \h, \H to
complete properly.  But there are no matches, then try a case
insensitive search to allow case conversion.  Add all missing help
options.

\Q<tab>    -> \q
\dD<tab>   -> \dD
\dd<tab>   -> \dd
\D<tab><tab><tab> -> \d (with listing of \d? commands)
sel<tab>   -> SELECT

Rod Taylor

OpenSSL 0.9.6g in Debian/unstable stopped working with pgcrypto.  This
is pgcrypto bug as it assumed too much about inner workings of OpenSSL.

Following patch stops pgcrypto using EVP* functions for ciphers and lets
it manage ciphers itself.

This patch supports Blowfish, DES and CAST5 algorithms.

Marko Kreen

pg_dump for domain constraints.

Rod Taylor

Add DOMAIN check constraints.

Rod Taylor

Add checkpoint_warning to warn of excessive checkpoints caused by too
few WAL files.

Push qual clauses containing subplans to the back of the qual list
at each plan node.  Per gripe from Ross Reedstrom.

Rename hostname_lookup to log_hostname.

Rename show_source_port to log_source_port.

Rename show_btree_build_stats to log_btree_build_stats

Rename:

! #show_parser_stats = false
! #show_planner_stats = false
! #show_executor_stats = false
! #show_statement_stats = false

TO:

! #log_parser_stats = false
! #log_planner_stats = false
! #log_executor_stats = false
! #log_statement_stats = false

Rename server_min_messages to log_min_messages.  Part of consolidation
of log_ settings.

Improve warning messages from tsearch trigger function; clean up some
casting infelicities.  Allow char(n) fields to be indexed.  Per
Bjoern Metzdorf.

Fix planning bug introduced in recent code reorganization for hashed
aggregates: tuple_fraction has to be adjusted before passing it to
compare_fractional_path_costs().

Update ports list.

Fix Magnus's name.

Fix version number for FreeBSD/alpha on ports list.

Update ports list.

Applied patch submitted by Mats Lofkvist fixing serious threading problem introduced in beta3.
Fixed bug with using setNull()(or setXXX(x, null))  and serverside prepare statements.
Improved error message when using a connection object that has already been closed.

 Modified Files:
  jdbc/org/postgresql/errors.properties
  jdbc/org/postgresql/core/Encoding.java
  jdbc/org/postgresql/core/QueryExecutor.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java

Update ports list.

There was an error in the gcc portion of the patch I sent you.  Here is
the  fix.

Billy G. Allie

Update ports list.

Allow +1300 as a numeric timezone specifier; we already accept FJST as meaning +1300.

Use cat instead of echo or awk, to avoid portability problems with backslashes.

Update xact.c comments for clarity.

Add:

> * Fix SELECT ... INTO and CREATE TABLE AS to have appopriate OID column
>  o -MOVE 0 should not move to end of cursor (Bruce)

Make MOVE/FETCH 0 actually move/fetch 0.  Add MOVE LAST to move to end
of cursor.

Add new palloc0 call as merge of palloc and MemSet(0).

Split MemSet into three parts to constant comparisons can be optimized
away by the compiler;  used by palloc0.

Add AIX as supported platform.

Update AIX resultmap for horology tests, per Andreas.

Improve comment: add note that grotty special case in mdread() is
required by hash index implementation.

Fix broken GB18030 <--> UTF-8 conversion map

Remove NO_MKTIME_BEFORE_1970.  I had speculated that it was not needed
anymore given the mktime() workaround now done in DetermineLocalTimeZone.
This has now been confirmed by Robert Bruccoleri for Irix, and I'm going
to extrapolate to AIX as well.

Code review for ON COMMIT patch.  Make the actual on-commit action happen
before commit, not after :-( --- the original coding is not only unsafe
if an error occurs while it's processing, but it generates an invalid
sequence of WAL entries.  Resurrect 7.2 logic for deleting items when
no longer needed.  Use an enum instead of random macros.  Editorialize
on names used for routines and constants.  Teach backend/nodes routines
about new field in CreateTable struct.  Add a regression test.

Merge documentation updates from 7.3 branch.

Merge release note updates from 7.3 branch.

Revise platform FAQs.

Improve wording.

Update jdbc doc to reflect correct name of the PGConnection class.

Fixes bug where join to pg_description was incorrect.  Also modifies the
regression test to test for this case.  Patch submitted by Kris Jurka.

 Modified Files:
  jdbc/org/postgresql/jdbc1/AbstractJdbc1DatabaseMetaData.java
  jdbc/org/postgresql/test/jdbc2/DatabaseMetaDataTest.java

Add cast to suppress compile warning on Alphas.

Back out use of palloc0 in place if palloc/MemSet.  Seems constant len
to MemSet is a performance boost.

Update russian FAQ, from Viktor Vislobokov.

Merge palloc()/MemSet(0) calls into a single palloc0() call.

Add palloc0 function to inline MemSet for newNode call.

This trivial patch removes 2 unused defines from pg_config.h.in:
FASTBUILD and DATEDEBUG weren't referenced by any other code.

Neil Conway

I am attaching a patch file that will replace the previous one.  It only
contains the patches to Makefile.global.in and Makefile.unixware.  The
Makefile.unixware patch has been updated to include the contents of
LD_LIBRARY_PATH, if present, to the -rpath (-R) option.  This change
will  simplify configuring and building PostgreSQL on systems that
support  LD_LIBRARY_PATH.  You can set LD_LIBRARY_PATH to include all
the directorys  you want to have searched for additional libraries, run
configure, then run  make.  The paths in LD_LIBRARY_PATH will then be
embedded in the executables  via the -rpath (-R) option to the linker,
and so will not require  LD_LIBRARY_PATH in order to run.

Billy G. Allie

This patch makes a minor cleanup to the implementation of PERFORM in
PL/PgSQL. Previously, it had been bundled together with the assign
statement implementation, for some reason that wasn't clear to me
(they certainly don't share any code with one another). So I separated
them and made PERFORM a statement like any other. No changes in
functionality.

Along the way, I added some regression tests for PERFORM, added a
bunch more SGML tags to the PL/PgSQL docs, and removed an obsolete
comment relating to the implementation of RETURN NEXT.

Neil Conway

> > I'll re-check that with the ppc architecture guy here.
>
> ... he is now about to write an inlined version that can go into
> s_lock.h . I'll send the new patch later on...

OK, here it comes:

An inlined version of tas(), that works for both, powerpc and
powerpc64. The patch is against 7.3b5 and passes the test suite on
both architectures.

Reinhard Max

Add documentation for array_upper/lower().

I just discovered, that there is missing a const when passing a buffer
to  PQescapeBytea and PQunescapeBytea. I fixed it and tried to create a
usable  diff (I'm not so familar to diff).

Tommi M?kitalo

Tweak CREATE SEQUENCE grammar to be more SQL1999 standards compliant.

Neil Conway

Add code to handle [ON COMMIT { PRESERVE ROWS | DELETE ROWS | DROP }]
for temp tables.

Gavin Sherry

Update discussion of floating-point problems in regression tests.

This patch removes a bunch of superfluous #include directives: if
postgres.h or c.h includes a system header (such as stdio.h or
stdlib.h), there's no need to specifically include it in any of the .c
files in the backend.

Neil Conway

Clean up format of SQL.

The attached patch defines functions for getting distances between
points on the surface of the earth and locating points within a
specified distance using an index based on the contrib/cube package. The
new functions are all of language type sql. A couple of bugs in the old
earthdistance function based on the point datatype are fixed. A
regression test has been added for both sets of functions. The README
file has been updated to include documentation on the new stuff. There
are comments about how this package is also useful for Astronomers.

Bruno Wolff III

Revise geometry regression testing to eliminate most cross-platform
variation.  To do this, set extra_float_digits to -3 in the geometry
test, and tweak the CIRCLE_TBL dataset to avoid values that suffer
from severe cancellation error (eg, circles that just touch an axis).
We still need two geometry 'expected' files to account for the
difference between platforms that display minus zero as '-0' and those
that just say '0', but with luck that's all we'll need.

Here is a patch that does just that, while maintaining the
"traditional" behavior, so the change should be transparent. Use the
command "\pset pager always" to turn it on. Anything else does the
normal toggle between "on" and "off"

Greg Sabino Mullane

Replace imprecise value of PI with a better one, and tweak circle_poly
in hopes of reducing platform-to-platform variations in its results.
This will cause the geometry regression test to start failing on some
platforms.  I plan to update the test later today.

Add name:

> * -Add GUC variables to control floating number output digits (Pedro Ferreira)

Mark as done:

> * -Add GUC variables to control floating number output digits

Add extra_float_digits GUC parameter to allow adjustment of displayed
precision for float4, float8, and geometric types.  Set it in pg_dump
so that float data can be dumped/reloaded exactly (at least on platforms
where the float I/O support is properly implemented).  Initial patch by
Pedro Ferreira, some additional work by Tom Lane.

The "Allow easy display of usernames in a group (pg_hba.conf uses groups
now)" item on the open items, and subsequent plpgsql function I sent in,
made me realize it was too hard to get the upper and lower bound of an
array. The attached creates two functions that I think will be very
useful when combined with the ability of plpgsql to return sets.

array_lower(array, dim_num)
- and -
array_upper(array, dim_num)

They return the value (as an int) of the upper and lower bound of the
requested dim in the provided array.

Joe Conway

Update FAQ_SCO from Larry Rosenman, correction from Tom Lane.

Update libpq PQcmdTuples() and PQcmdStatus() docs, remove const param
doc mention to match code and include files.

I'm using libpgtcl with PostgreSQL-7.2.3. There is a discrepancy between
the documentation and behavior of "pg_result $res -oid" when $res is not
from an INSERT.  The documentation says it should return an empty
string, but in fact it returns 0. I think it used to return an empty
string around PostgreSQL-7.1.3 when PQoidStatus() was used, but now it
uses PQoidValue() which returns InvalidOid, 0 in this case.

Assuming the current behavior is desired, here is a patch to the
documentation doc/src/sgml/libpgtcl.sgml to match what really happens:

ljb

Add:

> * Add GUC variables extra_float_digits and extra_double_digits to
>   control output digits
419a422
> * Research interaction of setitimer() and sleep() used by statement_timeout

Update release docs for 7.3.

Add #define _GNU_SOURCE to work around what seems to be Perl 5.8.0's
problem.  Per recent discussions about plperl failing to build on Linux.

Move substitute extern declarations to end of file, so that they are
not read until after we've read the port-specific header file.  In
particular this should make it safer to #include system headers for
inet_aton; in general it seems that the port header file ought to be
in a position to set definitions before we do stuff based on having
a definition or not.

Remove inappropriate inclusions of OpenSSL internal header e_os.h,
as well as unnecessary (and incorrect on Windows) assignments to
errno/SOCK_ERRNO.

Synced parser one more time.

Remove no-longer-needed inclusions to improve backward compatibility
with older bison versions.

Phase 2 of hashed-aggregation project.  nodeAgg.c now knows how to do
hashed aggregation, but there's not yet planner support for it.

Remove GPL file pursuant to relicensing.

Remove:

< * Move dbmirror and rserv from /contrib to gborg.postgresql.org

Change dbmirror lincense to BSD.

Not done:

> * Add hash for evaluating GROUP BY aggregates (Tom)

Completed:

> * -Add hash for evaluating GROUP BY aggregates (Tom)