Reorder 8.2 release note items more logically.

Update migration changes, more cleanups.

Properly mention in the release notes that fillfactor controls both heap
and indexes.  Other cleanups.

Wrap long lines in 8.2 release notes.

Updates from Jim Nasby.

Add built-in userlock manipulation functions to replace the former
contrib functionality.  Along the way, remove the USER_LOCKS configuration
symbol, since it no longer makes any sense to try to compile that out.
No user documentation yet ... mmoncure has promised to write some.
Thanks to Abhijit Menon-Sen for creating a first draft to work from.

Move 8.2 release documentation into SGML.

Make the order of the CASCADE and RESTRICT keywords in the DROP OWNED
syntax summary consistent with the other SQL reference pages. Patch
from Euler Taveira de Oliveira.

Documentation for VALUES lists.  Joe Conway and Tom Lane

Add URL for commenting postgresql.conf:

>    http://archives.postgresql.org/pgsql-hackers/2006-09/msg01481.php

Add URL for UUID:

>
>   http://archives.postgresql.org/pgsql-patches/2006-09/msg00209.php
>

Fix problems with column name list of CREATE TABLE AS being applied to
the input query's target list too soon, causing it to affect processing
of ORDER BY in the input query.

Add item:

>
> * Set client encoding based on the client operating system encoding
>
>   Currently client_encoding is set in postgresql.conf, which
>   defaults to the server encoding.

Add URL to UPDATE unique index case x=x+1:

<
>    http://archives.postgresql.org/pgsql-hackers/2006-09/msg01458.php

Improve wordings by David Fuhry <dfuhry@cs.kent.edu>

Fix CREATE TABLE ... AS VALUES ... to work rather than Assert'ing;
oversight in original implementation of VALUES.  Also fix an oversight
in recent addition of options to CREATE TABLE AS: they weren't getting
propagated if the query was a set-operation such as UNION.

Change ANALYZE to take ShareUpdateExclusiveLock not AccessShareLock on
the table being analyzed.  This prevents two ANALYZEs from running
concurrently on the same table and possibly suffering concurrent-update
failures while trying to store their results into pg_statistic.  The
downside is that a database-wide ANALYZE executed within a transaction
block will hold ShareUpdateExclusiveLock on many tables simultaneously,
which could lead to concurrency issues or even deadlock against another
such ANALYZE.  However, this seems a corner case of less importance
than getting unexpected errors from a foreground ANALYZE when autovacuum
elects to analyze the same table concurrently.  Per discussion.

Marginal cleanup in arrangements for ensuring StrategyHintVacuum is cleared
after an error during VACUUM.  We have a PG_TRY block anyway around the only
call sites, so just reset it in the CATCH clause instead of having
AtEOXact_Buffers blindly do it during xact end.  I think the old code was
actively wrong for the case of a failure during ANALYZE inside a
subtransaction --- the flag wouldn't get cleared until main transaction end.
Probably not worth back-patching though.

Update TODO for short header versions:

<  o Reorder physical storage order to reduce padding?
<
<    This involves having the user-specified order of columns
<    be different from the physical order.  SELECT * would
<    need to reorder the physical values to match the
<    user-specified ordering.
<
<  o Store disk pages with no alignment/padding?
<
<    This necessitates adding CPU-required padding when moving
<    rows from disk to memory.
<
<    One idea is to store the header in network byte order (high bits
<    first), and read the high bits to determine the header length.
<    http://archives.postgresql.org/pgsql-hackers/2006-09/msg00848.php
>    One idea is to create zero-or-one-byte-header versions
>    of varlena data types.  In involves setting the high-bit and
>    0-127 length in the single-byte header, or clear the high bit
>    and store the 7-bit ASCII value in the rest of the byte.
>    The small-header versions have no alignment requirements.
>    http://archives.postgresql.org/pgsql-hackers/2006-09/msg01372.php

Rename the recently-added pg_timezonenames view to pg_timezone_abbrevs,
and create a new view pg_timezone_names that provides information about
the zones known in the 'zic' database.  Magnus Hagander, with some
additional work by Tom Lane.

Update timezone data files to release 2006k of the zic database.

Rename xml_valid() to xml_is_well_formed(), but provide a temporary
alias with the old name for backwards compatibility.  Per discussion,
the old name is actively wrong because validity and well-formedness
have different meanings in XML.

Define errcode as __msvc_errcode not __vc_errcode for MSVC builds,
per Magnus.

Fix some more uses of str[n]casecmp that should be pg_str[n]casecmp.

Remove emacs info from footer of SGML files.

Document Warm Standby for High Availability

Includes sample standby script.

Simon Riggs

Document WAL recovery now uses checkpoints.

Simon Riggs

Several fixes for MSVC build scripts, from Magnus.

Fix some uses of str[n]casecmp that should be pg_str[n]casecmp.

Fix some uses of str[n]casecmp that should be pg_str[n]casecmp,
per Magnus.

Remove WINLDAPAPI decoration from ldap_start_tls_sA typedef, per Magnus.

Improve confusing comment for HeapTupleSatisfiesNow, per gripe from Greg Stark.

Make postgresql.conf.sample match the initdb defaults.  This fixes
comment alignment on most systems.

Add a couple of information functions to support direct checks on whether
a schema is our own temp schema or another backend's temp schema, and use
these in place of some former kluges in information_schema.  Per my
proposal of yesterday.

Seems no one wants this:

< * -Allow PREPARE to automatically determine parameter types based on the SQL
<   statement (Neil)

Done:

< * -Allow protocol-level BIND parameter values to be logged, if text mode
> * -Allow protocol-level BIND parameter values to be logged

Not done yet:

<  o -Allow commenting of variables in postgresql.conf to restore them
>  o Allow commenting of variables in postgresql.conf to restore them

Fix missing markup.

Remove sslinfo copyright with author permission, keep author attribution.

Victor Wagner

Remove:

< * Research storing only active XIDs in subtransaction cache

Add:

> * Research storing only active XIDs in subtransaction cache

Add XML item.
>
> * Improve XML support
>
>   http://developer.postgresql.org/index.php/XML_Support

Add description for variable-length header:

>    One idea is to store the header in network byte order (high bits
>    first), and read the high bits to determine the header length.

Add recent ICU URL:

>   http://archives.postgresql.org/pgsql-hackers/2006-09/msg00662.php

Update TODO items:

<  o Store disk pages with no alignment/padding?
>
>    This involves having the user-specified order of columns
>    be different from the physical order.  SELECT * would
>    need to reorder the physical values to match the
>    user-specified ordering.
>
>  o Store disk pages with no alignment/padding?
>
>    This necessitates adding CPU-required padding when moving
>    rows from disk to memory.
>
>
>    http://archives.postgresql.org/pgsql-hackers/2006-09/msg00848.php
>

For release notes checklist, add UTF8 URL.

In release notes checklist, add URL for HTML non-ASCII escapes, mention
UTF8 escapes.

Fix SGML markup

Improve error message. Per discussion
http://archives.postgresql.org/pgsql-general/2006-09/msg00186.php

GIN documentation and slightly improving GiST docs.
Thanks to  Christopher Kings-Lynne <chris.kingslynne@gmail.com> for
initial version and Jeff Davis <pgsql@j-davis.com> for inspection

Cygwin systems seem to have a different error message when trying to connect to port 0. Let's try port 20 instead.

Added missing constuctor/destructor for interval and date.

Some small editorialization on the description of CREATE INDEX
CONCURRENTLY.  Greg Stark, some further tweaks by me.

Make logging of extended-protocol commands a bit more consistent, per
discussion with Guillaume Smet.

Code review for patch to avoid second scan when vacuuming index-less
table: avoid invoking LockBufferForCleanup without need, put out the
same log message we would have before, minor code beautification.

Change "tps" to "scale" to avoid confusion

Fix bug with handling default scaling factor in the default
scenarios

Add David Fetter:

> * David is David Fetter <david@fetter.org>

Add uninstall script

Spell readme file's name correctly, per buildfarm reports.

Small fix for the README install for contrib/sslinfo.

Dave Page

Seems this wasn't committed originally

Move set_pglocale_pgservice() from path.c to exec.c, so that pulling in
path.c does not in itself force linking of both exec.c and libintl.
Should fix current ecpglib build failure on pickier platforms.

Move xml2's PG_MODULE_MAGIC block to the right source file, per Michael Fuhr.

Rename the uninstall scripts for contrib/lo and contrib/tsearch2 to
match the convention that foo's uninstall script is uninstall_foo.sql.
Also, stop installing lo_test.sql, which really ought to be made into
a regression test anyway (though it's unclear how to avoid a dependency
on the current OID counter...)

Add LDAP_LIBS_FE to $(libpq) for static or AIX linking, per Albe Laurenz.

Ooops, forgot to update ecpg comparison files to match
source-code change.

Fix omissions in contrib uninstall scripts.  Michael Fuhr

Fix unportable usage complained of by icc.  Per buildfarm results.

Remove unnecessary brace pair.

Back out patch added during 8.2.X development:

Allow to_char() "D" format specifiers for interval/time.

It doesn't work, and I doubt it is useful enough to fix ("D" = day of
week).

Add items:

< * Research storing disk pages with no alignment/padding
> * Consider ways of storing rows more compactly on disk
>
>  o Store disk pages with no alignment/padding?
>  o Reorder physical storage order to reduce padding?
>  o Support a smaller header for short variable-length fields?
>  o Reduce the row header size?

Install a cleaner solution to the AIX libpq linking problem, as per
an earlier discussion.  Centralize assumptions about what libpq depends
on in one place in Makefile.global.  I am unconvinced that this list
is complete, but since ecpg seems to have gotten along with just these
entries, we'll try it this way and see what happens.

Fix dblink build for --enable-nls or --enable-openssl on AIX.
Per Chris Browne.

Remove unhelpful/misleading advice about how to use SPI_saveplan().
Per gripe from Jack Orenstein.

contrib/isn updates from Jeremy Kronuz.

If we're going to advertise the array overlap/containment operators,
we probably should make them work reliably for all arrays.  Fix code
to handle NULLs and multidimensional arrays, move it into arrayfuncs.c.
GIN is still restricted to indexing arrays with no null elements, however.

Add missing documentation for new anyarray-overlap operator.

Rename contrib contains/contained-by operators to @> and <@, per discussion.

Rename contains/contained-by operators to @> and <@, per discussion that
agreed these symbols are less easily confused.  I made new pg_operator
entries (with new OIDs) for the old names, so as to provide backward
compatibility while making it pretty easy to remove the old names in
some future release cycle.  This commit only touches the core datatypes,
contrib will be fixed separately.

Remove reference to contrib/mac, which isn't there anymore.

Add contrib/isn module for ISBN/ISSN/EAN13/etc product numbers, and
remove the old isbn_issn module which is about to be obsoleted by EAN13.
contrib/isn is by Germán Méndez Bravo.  Our thanks to Garrett A. Wollman
for having written the original isbn_issn module.

Revise OpenLDAP configuration and linking to work on more platforms
than before.  Albe Laurenz (but editorialized heavily by me, so if it
doesn't work it's my fault).

Fix pg_dump for recent change removing separate RULE privilege.
I had thought this code could be left alone, but I was wrong: as-is
it's failing to recognize when to use ALL for table privileges in 8.2.

Put back plan-time check for trying to apply SELECT FOR UPDATE/SHARE
to a relation on the nullable side of an outer join.  I had removed
this during the outer join planning rewrite a few months ago ... I think
I intended to put it somewhere else, but forgot ...

Tweak the behavior of log_duration as proposed by Guillaume Smet: rather
than being equivalent to setting log_min_duration_statement to zero, this
option now forces logging of all query durations, but doesn't force logging
of query text.  Also, add duration logging coverage for fastpath function
calls.

Not all C compilers understand the "//" comment in this test case, so I removed it.

Replaced complex tests with small ones.

More logic from complex/test* moved to their own testcases.

Clean up logging for extended-query-protocol operations, as per my recent
proposal.  Parameter logging works even for binary-format parameters, and
logging overhead is avoided when disabled.

log_statement = all output for the src/test/examples/testlibpq3.c example
now looks like

LOG:  statement: execute <unnamed>: SELECT * FROM test1 WHERE t = $1
DETAIL:  parameters: $1 = 'joe''s place'
LOG:  statement: execute <unnamed>: SELECT * FROM test1 WHERE i = $1::int4
DETAIL:  parameters: $1 = '2'

and log_min_duration_statement = 0 results in

LOG:  duration: 2.431 ms  parse <unnamed>: SELECT * FROM test1 WHERE t = $1
LOG:  duration: 2.335 ms  bind <unnamed> to <unnamed>: SELECT * FROM test1 WHERE t = $1
DETAIL:  parameters: $1 = 'joe''s place'
LOG:  duration: 0.394 ms  execute <unnamed>: SELECT * FROM test1 WHERE t = $1
DETAIL:  parameters: $1 = 'joe''s place'
LOG:  duration: 1.251 ms  parse <unnamed>: SELECT * FROM test1 WHERE i = $1::int4
LOG:  duration: 0.566 ms  bind <unnamed> to <unnamed>: SELECT * FROM test1 WHERE i = $1::int4
DETAIL:  parameters: $1 = '2'
LOG:  duration: 0.173 ms  execute <unnamed>: SELECT * FROM test1 WHERE i = $1::int4
DETAIL:  parameters: $1 = '2'

(This example demonstrates the folly of ignoring parse/bind steps for duration
logging purposes, BTW.)

Along the way, create a less ad-hoc mechanism for determining which commands
are logged by log_statement = mod and log_statement = ddl.  The former coding
was actually missing quite a few things that look like ddl to me, and it
did not handle EXECUTE or extended query protocol correctly at all.

This commit does not do anything about the question of whether log_duration
should be removed or made less redundant with log_min_duration_statement.

Update pgcvslog comments.

Specify lo_write() to take a _const_ buffer, to match documentation.

Changing a test case also changes the output that is expected.
Float/Long aliasing doesn't work on all architecures.

Removed one output because integer/double usage differ

HTLM cleanup.

Update Emacs/vim editor info.

Update emacs info for FAQ_DEV.

Andrew Dunstan

Add XML documentation.

Update tools directory name.

Change processing of extended-Query mode so that an unnamed statement
that has parameters is always planned afresh for each Bind command,
treating the parameter values as constants in the planner.  This removes
the performance penalty formerly often paid for using out-of-line
parameters --- with this definition, the planner can do constant folding,
LIKE optimization, etc.  After a suggestion by Andrew@supernews.

Be more wary about which LIBS elements to pull into sslinfo link.
Per buildfarm results.