Repair bug #2694 concerning an ARRAY[] construct whose inputs are empty
sub-arrays.  Per discussion, if all inputs are empty arrays then result
must be an empty array too, whereas a mix of empty and nonempty arrays
should (and already did) draw an error.  In the back branches, the
construct was strict: any NULL input immediately yielded a NULL output;
so I left that behavior alone.  HEAD was simply ignoring NULL sub-arrays,
which doesn't seem very sensible.  For lack of a better idea it now
treats NULL sub-arrays the same as empty ones.

Remove temporary Windows-specific debugging code; it seems the problem
with fopen() not using FILE_SHARE_DELETE was indeed the bug we were after,
given lack of recent reports.

Update release notes in preparation for RC1.

Revert to the pre-8.2 method of probing for libm, that is, always
include it if it links properly.  It seems too risky to assume that
standard functions like pow() are not special-cased by the compiler.
Per report from Andreas Lange that build fails on Solaris cc compiler
with -fast.  Even though we don't consider that a supported option,
I'm worried that similar issues will arise with other compilers.

Get rid of some unnecessary dependencies on DataDir: wherever possible,
the backend should rely on its working-directory setting instead.
Also do some message-style police work in contrib/adminpack.

Minor fix for LDAP authentication: if an error occurs, we need to
manually release the LDAP handle via ldap_unbind(). This isn't a
significant problem in practice because an error eventually results
in exiting the process, but we can cleanup correctly without too
much pain.

In passing, fix an error in snprintf() usage: the "size" parameter
to snprintf() is the size of the destination buffer, including space
for the NUL terminator. Also, depending on the value of NAMEDATALEN,
the old coding could have allowed for a buffer overflow.

Fix recently-identified PITR recovery hazard: the base backup could contain
stale relcache init files (pg_internal.init), and there is no mechanism for
updating them during WAL replay.  Easiest solution is just to delete the init
files at conclusion of startup, and let the first backend started in each
database take care of rebuilding the init file.  Simon Riggs and Tom Lane.

Back-patched to 8.1.  Arguably this should be fixed in 8.0 too, but it would
require significantly more code since 8.0 has no handy startup-time scan of
pg_database to piggyback on.  Manual solution of the problem is possible
in 8.0 (just delete the pg_internal.init files before starting WAL replay),
so that may be a sufficient answer.

Fix recently-understood problems with handling of XID freezing, particularly
in PITR scenarios.  We now WAL-log the replacement of old XIDs with
FrozenTransactionId, so that such replacement is guaranteed to propagate to
PITR slave databases.  Also, rather than relying on hint-bit updates to be
preserved, pg_clog is not truncated until all instances of an XID are known to
have been replaced by FrozenTransactionId.  Add new GUC variables and
pg_autovacuum columns to allow management of the freezing policy, so that
users can trade off the size of pg_clog against the amount of freezing work
done.  Revise the already-existing code that forces autovacuum of tables
approaching the wraparound point to make it more bulletproof; also, revise the
autovacuum logic so that anti-wraparound vacuuming is done per-table rather
than per-database.  initdb forced because of changes in pg_class, pg_database,
and pg_autovacuum catalogs.  Heikki Linnakangas, Simon Riggs, and Tom Lane.

Add note about space usage of 'manual' approach to clustering, per
suggestion from Sergey Koposov.  Also some other minor editing.

Correct documentation error: in 8.1 and 8.2, %p in archive and restore
command strings inserts relative not absolute path of file to process.
This is a side-effect of 2005-07-04 change that makes the server use
relative paths in general.  Noted by Bernd Helmle.

Mention 'void' as the proper return type when there's nothing to
return, per suggestion from Joachim Wieland.

Fix "failed to re-find parent key" btree VACUUM failure by revising page
deletion code to avoid the case where an upper-level btree page remains "half
dead" for a significant period of time, and to block insertions into a key
range that is in process of being re-assigned to the right sibling of the
deleted page's parent.  This prevents the scenario reported by Ed L. wherein
index keys could become out-of-order in the grandparent index level.

Since this is a moderately invasive fix, I'm applying it only to HEAD.
The bug exists back to 7.4, but the back branches will get a different patch.

pg_restore failed on tar-format archives if they contained large objects
(blobs) with comments, per bug #2727 from Konstantin Pelepelin.
Mea culpa for not having tested this case.
Back-patch to 8.1; prior branches don't dump blob comments at all.

Update zic database to tzdata2006n.

Add description of new features

Fix misstatement about the results of using -1 with BEGIN.

Add text to the VACUUM, REINDEX, DROP TABLESPACE and CREATE TABLESPACE
reference pages documenting that these commands cannot be used within
a transaction block. Also make some minor improvements to the psql
reference page. Patch from Simon Riggs, minor editorialization by
myself.

Code the unknown options check without using m4 diversions.  Otherwise this
code relies on the checking macro actually being called at the end, or the
automatic undiversion will produce garbage.  These sort of implicit
side-effects undermine the modularity of the macros and happen to break the
ODBC driver which makes use of them.

Also put the warnings at the very end of configure, so there is an even
better chance of seeing them.

Fix typo in backup docs. Patch from Euler Taveira de Oliveira.

Fix typo in docs.

Fix spelling mistake in docs.

Add missing file for documentation section on failover, replication,
load balancing, and clustering options.

Add documentation section on failover, replication, load balancing, and
clustering options.

expression_tree_walker failed to let walker function see the immediate child
node of a SubLink or SubPlan testexpr field.  Bug resulted from replacing
the old lefthand/exprs list fields with a simple expression field, and not
remembering that expression_tree_walker is coded to save a few cycles by
recursing directly to self on list fields (on the assumption the walker
isn't interested in List nodes per se).  On non-list fields it must of
course call the walker.  Possibly that hack isn't worth the risk of more
such bugs, but I'll leave it be for now.  Per bug report from James Robinson.

Fix check for whether a clauseless join has to be forced in the presence of
outer joins.  Originally it was only looking for overlap of the righthand
side of a left join, but we have to do it on the lefthand side too.
Per example from Jean-Pierre Pelletier.

Tweak zic.c to compile cleanly on MSVC: use CopyFile instead of
CopyFileEx.  This avoids a warning about the function not being
present on older Windows versions.  Magnus Hagander

Correctino, add URL on :

* Add estimated_count(*) to return an estimate of COUNT(*)
>   http://archives.postgresql.org/pgsql-hackers/2005-11/msg00943.php

Add URL for:

* Speed up COUNT(*)
>   http://archives.postgresql.org/pgsql-hackers/2005-11/msg00943.php

Rewrite discussion of ORDER BY to emphasize the SQL99 expression case
instead of the SQL92 output-column-ID case.

Stamp 8.2beta2.

Add some real descriptions to the multiargument aggregate functions rather
than just showing the incomprehensible formulas.

Fix broken markup.

Spellchecking and such

Fix list of environment variables for configure.  Remove some variables
that are almost never used.  Put documentation-related variables into the
documentation chapter.

Small parser sync

Remove an unnecessary HOLD_INTERRUPTS/RESUME_INTERRUPTS pair.
This was required back when RESUME_INTERRUPTS could actually
execute ProcessInterrupts, but that hasn't been true since 2001...

Code cleanup for pg_buffercache, from Mark Kirkwood.

Various MSVC build fixes from Magnus; also remove stray Windows
newlines.

Some desultory copy-editing.

Some marginal editorial improvements and updates in the tutorial.

Translations update

Further release-note editorialization.

Editorial improvements for recent plpython doc updates.

Update PQserverVersion() to use 8.1.X as an example, rather than 7.4
because the function didn't exist in 7.4.X.

Update release notes for new pgcrypto additions.

Marko Kreen

Add URL for async I/O:

>   http://archives.postgresql.org/pgsql-hackers/2006-10/msg00820.php

Rewrite discussion of string constant syntax to bring it into line with
the politically correct view that backslash escapes are deprecated.

Make index entry more specific.

pgbench: More fix with handling default scaling factor in the default
scenarios. With multiple clinets, only the first client got the right
scaling factor and this gave a illusion of better performance in case
of the scaling factor greater than 1.

Update release notes to current, and do a pass of editorial corrections.

Minor doc tweak: make a reference to pg_locks into a link.

Fix typo.

Marginal code cleanups in pg_logdir_ls: use ReadDir not readdir,
and avoid scribbling on its result (might be safe but why risk it)

Try to fix the AIX getaddrinfo mess in a way that works on all versions.
Going to wait for buildfarm results before backpatching, this time.

Clarify note about interaction of log_statement logging with errors.
Remove obsolete note about logging of statements generated by plpgsql
function execution ... that doesn't happen anymore.

Marginal improvement in logging: include the function name when logging
a fastpath function call.

Rename our substitute qsort to pg_qsort at the link-symbol level (but
provide a macro so code can still just say qsort).  Avoids linker warnings
on pickier platforms such as Darwin, and outright failure on MSVC.

Add externs for optarg/optind where apparently needed.  Per Magnus.

Handle missing M_PI the same way we've been doing in the core code,
instead of inserting an MSVC dependency.

Further MSVC portability fixes from Magnus.

Remove qsort TODO.detail.  All items completed.

Done:

< * Improve port/qsort() to handle sorts with 50% unique and 50% duplicate
<   value [qsort]
<
<   This involves choosing better pivot points for the quicksort.
<

Fix a couple of places that were assuming debug_query_string couldn't
be NULL ... seems an unsafe assumption.

Make sure that debug_query_string contains the original query text,
if available (which it usually should be), during processing of Bind
and Execute protocol messages.  This improves usefulness of
log_min_error_statement logging for extended query protocol.

Clean up local redeclarations of variables with DLLIMPORT, per report
from Magnus that MSVC complains about this.

Fix typo.

Simplify contrib Makefiles by removing unnecessary SRCS macro,
per Magnus.

Work around reported problem that AIX's getaddrinfo() doesn't seem to zero
sin_port in the returned IP address struct when servname is NULL.  This has
been observed to cause failure to bind the stats collection socket, and
could perhaps cause other issues too.  Per reports from Brad Nicholson
and Chris Browne.

Add some code to CREATE DATABASE to check for pre-existing subdirectories
that conflict with the OID that we want to use for the new database.
This avoids the risk of trying to remove files that maybe we shouldn't
remove.  Per gripe from Jon Lapham and subsequent discussion of 27-Sep.

Fix up timetz input so that a date is required only when the specified
timezone actually has a daylight-savings rule.  This avoids breaking
cases that used to work because they went through the DecodePosixTimezone
code path.  Per contrib regression failures (mea culpa for not running
those yesterday...).  Also document the already-applied change to allow
GMT offsets up to 14 hours.

Fix array operator refererence in release notes, per David Wheeler.

Fix up some problems in handling of zic-style time zone names in datetime
input routines.  Remove the former "DecodePosixTimezone" function in favor of
letting the zic code handle POSIX-style zone specs (see tzparse()).  In
particular this means that "PST+3" now means the same as "-03", whereas it
used to mean "-11" --- the zone abbreviation is effectively just a noise word
in this syntax.  Make sure that all named and POSIX-style zone names will be
parsed as a single token.  Fix long-standing bogosities in printing and input
of fractional-hour timezone offsets (since the tzparse() code will accept
these, we'd better make 'em work).  Also correct an error in the original
coding of the zic-zone-name patch: in "timestamp without time zone" input,
zone names are supposed to be allowed but ignored, but the coding was such
that the zone changed the interpretation anyway.

Mark 8.2 release as 2006-??, not 2005-??, per observation by David
Wheeler.

Attached files fix the link problem in FAQ_DEV.html, remove some parts
related to website development and change the link to the FAQ_DEV.html.

Devrim GUNDUZ

Fixes for CREATE CONSTRAINT manual page.

Michael Paesold

Fix unexpected side-effect of changes for case insensitivity of timezone names.

Remove use of whrandom module, which was removed in Python 2.5.

Arrange for timezone names to be recognized case-insensitively; for
example SET TIME ZONE 'america/new_york' works now.  This seems a good
idea on general user-friendliness grounds, and is part of the solution
to the timestamp-input parsing problems I noted recently.

CREATE CONSTRAINT manual page wording improvements.

Michael Paesold

Update to CREATE CONSTRAINT manual page.

Michael Glaesemann

I updated RPM related parts in FAQ_DEV against HEAD to be more current.

Devrim GUNDUZ

Remove use of '<' and '>' in SGML, use '&' escapes.

Update find_gt_lt to allow grep parameters to be passed into it.

Punt when trying to build with threaded Python on FreeBSD.

Also cut back on excessive use of *** to decorate configure error messages.
If it's an error message, you are sure to see it without any decoration.

Redesign the patch for allocation of shmem space and LWLocks for add-on
modules; the first try was not usable in EXEC_BACKEND builds (e.g.,
Windows).  Instead, just provide some entry points to increase the
allocation requests during postmaster start, and provide a dedicated
LWLock that can be used to synchronize allocation operations performed
by backends.  Per discussion with Marc Munro.

Adjust plperl to ensure that all strings and hash keys passed to Perl
are marked as UTF8 when the database encoding is UTF8.  This should
avoid inconsistencies like that exhibited in bug #2683 from Vitali Stupin.

Update Japanese FAQ.

Jun Kuwamura

FAQ updates --- This is a small cosmetic patch that adds pt_BR IRC
channel to the lists of IRC channels, fixes a typo in the OID's
question, and corrects the PGCluster's project name.

Euler Taveira de Oliveira

Code review for --no-data-for-failed-tables patch.  Instead of trashing
one of the program's core data structures, make use of the existing
ability to selectively exclude TOC items by ID.  Slightly more code but
much less likely to create future maintenance problems.

Code and docs review for ALTER TABLE INHERIT/NO INHERIT patch.

Make unknown-option-warning code more portable.  echo -n is not portable,
and neither is "|" or "\|" in basic regular expressions.

Update Japanese FAQ.

Jun Kuwamura

Fix typo in version number.

Fix test_fsync compile on MinGW(win32)

Hiroshi Saito

Fix infinite sleep and failes of send in Win32.

1) pgwin32_waitforsinglesocket(): WaitForMultipleObjectsEx now called with
finite timeout (100ms) in case of FP_WRITE and UDP socket. If timeout occurs
then pgwin32_waitforsinglesocket() tries to write empty packet goes to
WaitForMultipleObjectsEx again.

2) pgwin32_send(): add loop around WSASend and pgwin32_waitforsinglesocket().
The reason is: for overlapped socket, 'ok' result from
pgwin32_waitforsinglesocket() isn't guarantee that socket is still free,
it can become busy again and following WSASend call will fail with
WSAEWOULDBLOCK error.

See http://archives.postgresql.org/pgsql-hackers/2006-10/msg00561.php

Remove:

<  o Issue a notice if CREATE TABLE ... ON COMMIT { DELETE ROWS |
<    DROP } is issued outside a multi-statement transaction

Add url's for hints:

>   http://archives.postgresql.org/pgsql-hackers/2006-10/msg00517.php
>   http://archives.postgresql.org/pgsql-hackers/2006-10/msg00663.php

Add:

>  o Issue a notice if CREATE TABLE ... ON COMMIT { DELETE ROWS |
>    DROP } is issued outside a multi-statement transaction

Make some incremental improvements and fixes to the documentation on
Continuous Archiving. Plenty of editorial work remains...

Update release notes for SQL functions vs triggers fix.

In the installation docs, add some links to the documentation on LDAP
to the discussion of the "--with-ldap" configure option. Patch from
Albe Laurenz.