Release note improvements (Neil, Tom)

Remove inappropriate double-quoting in connectby() code; adjust
regression test to avoid using VALUE as a name.  From Joe Conway.

Redesign internal logic of nodeLimit so that it does not need to fetch
one more row from the subplan than the COUNT would appear to require.
This costs a little more logic but a number of people have complained
about the old implementation.

Fix a dozen or so places that were passing unpredictable data strings
as elog format strings.  Hai-Chen Tu pointed out the problem in
contrib/dbmirror, but it wasn't the only such error.

Fix elog in dbmirror to use %s.

Hai-Chen Tu

Quote database name properly when invoking pg_dump.  Per report from
Christopher Kings-Lynne.

TAS code originally written for s390 (32-bit) does not work for s390x
(64-bit).  Fix it.  Per report from Permaine Cheung.

Fixups for man pages

Regenerate

Fix breakage in new-in-7.3 timetz_zone() function: was giving random
results due to doing arithmetic on uninitialized values.  Add some
documentation about the AT TIME ZONE construct.  Update some other
date/time documentation that seemed out of date for 7.3.

Wups, stuck ORDER BY on wrong query while trying to eliminate random
ordering of GROUP BY output due to hash-based aggregation.

Add:

> * Have sequence dependency track use of DEFAULT sequences, seqname.nextval

Add mention of kern.ipc.shm_use_phys for locking shared memory.

Code review for superuser_reserved_connections patch.  Don't try to do
database access outside a transaction; revert bogus performance improvement
in SIBackendInit(); improve comments; add documentation (this part courtesy
Neil Conway).

NetBSD versions 1.6 and (we hope) later can use the standard geometry
comparison file, instead of geometry-positive-zeros.  Per report from
Patrick Welche.

Done:

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

Finish implementation of hashed aggregation.  Add enable_hashagg GUC
parameter to allow it to be forced off for comparison purposes.
Add ORDER BY clauses to a bunch of regression test queries that will
otherwise produce randomly-ordered output in the new regime.

Update ports list.

Fix a dumb cut and paste error from my last commit and update some of the
jdbc3 metadata responses

 Modified Files:
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/jdbc3/AbstractJdbc3DatabaseMetaData.java

Fixed bug reported by Marko Strukelj and Keith Wannamaker.  Using executeBatch
on a preparedStatement would reset the prepared statment causing subsequent
uses of the preparedStatement to fail (i.e. the following series of calls
would fail: addBatch() executeBatch() addBatch() executBatch()).  This is
a regression from 7.2 where this worked correctly.  The regression test has
also been modified to explicitly test for this case.

 Modified Files:
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/test/jdbc2/BatchExecuteTest.java

Add an at-least-marginally-plausible method of estimating the number
of groups produced by GROUP BY.  This improves the accuracy of planning
estimates for grouped subselects, and is needed to check whether a
hashed aggregation plan risks memory overflow.

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