Eliminate a gratuitously different wording of the 'cannot use aggregate function in check constraint' error message.

- During dblink_open, if transaction state was IDLE, force cursor count to
  initially be 0. This is needed as a previous ABORT might have wiped out
  an automatically opened transaction without maintaining the cursor count.
- Fix regression test expected file for the correct ERROR message, which
  we now get given the above bug fix.

Clean up psql variable code a little: eliminate unnecessary tests in
GetVariable() and be consistent about treatment of the list header.
Motivated by noticing strspn() taking an unreasonable percentage of
runtime --- the call removed from GetVariable() was the only one that
could be in a high-usage path ...

Added some more coverity report patches send in by Martijn van Oosterhout <kleptog@svana.org>.

Fixed small typo in changelog

Added fixed from the coverity report send in by Joachim Wieland <joe@mcknight.de>
Added missing error handling in a few functions in ecpglib

Remove redundant gettimeofday() calls to the extent practical without
changing semantics too much.  statement_timestamp is now set immediately
upon receipt of a client command message, and the various places that used
to do their own gettimeofday() calls to mark command startup are referenced
to that instead.  I have also made stats_command_string use that same
value for pg_stat_activity.query_start for both the command itself and
its eventual replacement by <IDLE> or <idle in transaction>.  There was
some debate about that, but no argument that seemed convincing enough to
justify an extra gettimeofday() call.

Split definitions for md5.c out of crypt.h and into their own header
libpq/md5.h, so that there's a clear separation between backend-only
definitions and shared frontend/backend definitions.  (Turns out this
is reversing a bad decision from some years ago...)  Fix up references
to crypt.h as needed.  I looked into moving the code into src/port, but
the headers in src/include/libpq are sufficiently intertwined that it
seems more work than it's worth to do that.

Some editorial work on the documentation of the current-date/time
functions.

Do not use already free'ed errmsg, bug found by Joachim Wieland
<joachim.wieland@credativ.de>

Take the statistics collector out of the loop for monitoring backends'
current commands; instead, store current-status information in shared
memory.  This substantially reduces the overhead of stats_command_string
and also ensures that pg_stat_activity is fully up to date at all times.
Per my recent proposal.

Update my email address.

Don't try to call posix_fadvise() unless <fcntl.h> supplies a declaration
for it.  Hopefully will fix core dump evidenced by some buildfarm members
since fadvise patch went in.  The actual definition of the function is not
ABI-compatible with compiler's default assumption in the absence of any
declaration, so it's clearly unsafe to try to call it without seeing a
declaration.

Fix saveHistory() to not emit bogus complaint during psql exit when using
Darwin's libedit; per my proposal last August.  Also, suppress cast-away-
const warnings.

Increase timeout in statement_timeout test from 1 second to 2 seconds.
We have once or twice seen failures suggesting that control didn't get
to the exception block before the timeout elapsed, which is unlikely
but not impossible in a parallel regression test (with a dozen other
backends competing for cycles).  This change doesn't completely prevent
the problem of course, but it should reduce the probability enough that
we don't see it anymore.  Per buildfarm results.

Merge postmaster and postgres command into just postgres.  postmaster
symlink is kept for now for compatibility.  To call single-user mode, use
postgres --single.

Fix a couple of obvious problems in DROP IF EXISTS patch.

Code review for SELECT INTO STRICT patch: use saner choices of error
SQLSTATEs, fix some documentation problems.

Clean up after someone's curious idea that it'd be good to strip
leading zeroes from the SQLSTATE codes.  They're strings, people,
not numbers.

docs for DROP ... IF EXISTS for the following cases:
  language, tablespace, trigger, rule, opclass, function, aggregate. operator, and cast.

Add:

>  o Allow PL/python to composite types and result sets
>    once buggy assert-enabled versions of python can be detected
>
>    http://archives.postgresql.org/pgsql-patches/2006-04/msg00087.php

Add:

>
> * Fix CREATE CAST on DOMAINs
>
>   http://archives.postgresql.org/pgsql-hackers/2006-05/msg00072.php

Document issues with non-default tablespaces and pg_dumpall restores.

Backpatch documentation addition to 8.1.X.

Add URL for ISO dates:

>    http://archives.postgresql.org/pgsql-bugs/2006-04/msg00248.php

DROP ... IF EXISTS for the following cases:
  language, tablespace, trigger, rule, opclass, function, aggregate. operator, and cast.

Add URL for currency type:

>   http://archives.postgresql.org/pgsql-general/2005-08/msg01432.php

Add URL for updatable views:

>   http://archives.postgresql.org/pgsql-hackers/2006-03/msg00586.php

Add:

> * Consider shrinking expired tuples to just their headers
>   http://archives.postgresql.org/pgsql-patches/2006-03/msg00142.php

Fix problems with cached tuple descriptors disappearing while still in use
by creating a reference-count mechanism, similar to what we did a long time
ago for catcache entries.  The back branches have an ugly solution involving
lots of extra copies, but this way is more efficient.  Reference counting is
only applied to tupdescs that are actually in caches --- there seems no need
to use it for tupdescs that are generated in the executor, since they'll go
away during plan shutdown by virtue of being in the per-query memory context.
Neil Conway and Tom Lane

Add URL to:

* Auto-fill the free space map by scanning the buffer cache or by
  checking pages written by the background writer

Add URL for freespace map filling.

>
>   http://archives.postgresql.org/pgsql-hackers/2006-02/msg01125.php
>

>
> * Add support for public SYNONYMS
>
>   http://archives.postgresql.org/pgsql-hackers/2006-03/msg00519.php

>               o Add PL/Python tracebacks
>
>                http://archives.postgresql.org/pgsql-patches/2006-02/msg00288.php

Add:

>  o Fix problems with RETURN NEXT on tables with
>    dropped/added columns after function creation
>
>    http://archives.postgresql.org/pgsql-patches/2006-02/msg00165.php

Add URL for:

> * Allow protocol-level BIND parameter values to be logged
>
>   http://archives.postgresql.org/pgsql-hackers/2006-02/msg00165.php

Add to locale TODO.detail.

Add URL for ISO date/time syntax:
>
>    http://archives.postgresql.org/pgsql-hackers/2006-01/msg00250.php
>

Add LDAP documentation missed in code patch.

Magnus Hagander

Update preload_library magic block wording.

Test for POSIX_FADV_DONTNEED to use posix_fadvise().

Fix:

<    get_employee_salary(emp_id AS 12345, tax_year AS 2001)
>    get_employee_salary(12345 AS emp_id, 2001 AS tax_year)

Update magic block wording for preloaded libraries.

Use posix_fadvise() to avoid kernel caching of WAL contents on WAL file
close.

ITAGAKI Takahiro

Add STRICT to PL/pgSQL SELECT INTO, so exceptions are thrown if more or
less than one row is returned by the SELECT, for Oracle PL/SQL
compatibility.

Improve SELECT INTO documentation.

Matt Miller

Fix to_ascii() markup error.

Add mention that preload_libraries check the magic block, so
non-PostgreSQL libraries cannot be loaded using this capability.

Update syntax suggestion:

<    get_employee_salary(emp_id => 12345, tax_year => 2001)
>    get_employee_salary(emp_id AS 12345, tax_year AS 2001)

Move list of supported to_ascii() encodings from footnote to main
description.

Nis Jorgensen

Add:

>
>  o Fix memory leak from exceptions
>
>    http://archives.postgresql.org/pgsql-performance/2006-06/msg00305.php

Clarify dynamic pl/pgsql item and add URLs.  Restructure server-side
section into PL/pgSQL and non-PL/pgSQL sections:

<  o Fix PL/pgSQL RENAME to work on variables other than OLD/NEW
<  o Allow function parameters to be passed by name,
<    get_employee_salary(emp_id => 12345, tax_year => 2001)
<  o Add Oracle-style packages
<  o Add table function support to pltcl, plpython
<  o Add capability to create and call PROCEDURES
<  o Allow PL/pgSQL to handle %TYPE arrays, e.g. tab.col%TYPE[]
<  o Allow function argument names to be statements from PL/PgSQL
<  o Add MOVE to PL/pgSQL
<  o Add support for polymorphic arguments and return types to
<    languages other than PL/PgSQL
<  o Add support for OUT and INOUT parameters to languages other
<    than PL/PgSQL
<  o Add single-step debugging of PL/PgSQL functions
<  o Allow PL/PgSQL to support WITH HOLD cursors
<  o Allow PL/PgSQL RETURN to return row or record functions
<
<    http://archives.postgresql.org/pgsql-patches/2005-11/msg00045.php
>  o PL/pgSQL
>  o Fix RENAME to work on variables other than OLD/NEW
>  o Allow function parameters to be passed by name,
>    get_employee_salary(emp_id => 12345, tax_year => 2001)
>  o Add Oracle-style packages
>  o Allow handling of %TYPE arrays, e.g. tab.col%TYPE[]
>  o Allow listing of record column names, and access to
>    record columns via variables, e.g. columns := r.(*),
>    tval2 := r.(colname)
>
>    http://archives.postgresql.org/pgsql-patches/2005-07/msg00458.php
>    http://archives.postgresql.org/pgsql-patches/2006-05/msg00302.php
>    http://archives.postgresql.org/pgsql-patches/2006-06/msg00031.php
>
>  o Add MOVE
>  o Add single-step debugging of functions
>  o Add support for WITH HOLD cursors
>  o Allow PL/RETURN to return row or record functions
>
>    http://archives.postgresql.org/pgsql-patches/2005-11/msg00045.php
>
>
>  o Other
>  o Add table function support to pltcl, plpython
>  o Add support for polymorphic arguments and return types to
>    languages other than PL/PgSQL
>  o Add capability to create and call PROCEDURES
>  o Add support for OUT and INOUT parameters to languages other
>    than PL/PgSQL

Add:

>  o Allow PL/PgSQL RETURN to return row or record functions
>
>    http://archives.postgresql.org/pgsql-patches/2005-11/msg00045.php

Done, removed:

<
< * Consider padding the spinlock structure to have spinlock registers
<   on different CPU cache line
<
<   http://archives.postgresql.org/pgsql-hackers/2005-10/msg00498.php

Remove the limit on the number of entries allowed in catcaches, and
remove the infrastructure needed to enforce the limit, ie, the global
LRU list of cache entries.  On small-to-middling databases this wins
because maintaining the LRU list is a waste of time.  On large databases
this wins because it's better to keep more cache entries (we assume
such users can afford to use some more per-backend memory than was
contemplated in the Berkeley-era catcache design).  This provides a
noticeable improvement in the speed of psql \d on a 10000-table
database, though it doesn't make it instantaneous.

While at it, use per-catcache settings for the number of hash buckets
per catcache, rather than the former one-size-fits-all value.  It's a
bit silly to be using the same number of hash buckets for, eg, pg_am
and pg_attribute.  The specific values I used might need some tuning,
but they seem to be in the right ballpark based on CATCACHE_STATS
results from the standard regression tests.

Add to locale TODO.detail.

Remove, done:

<
< * Add support for WIN1253 (Greek) encoding
<
<   http://archives.postgresql.org/pgsql-bugs/2005-12/msg00000.php

Add:

>
> * Add support for WIN1253 (Greek) encoding
>
>   http://archives.postgresql.org/pgsql-bugs/2005-12/msg00000.php

Add URL to ILIKE encoding fix:

>   http://archives.postgresql.org/pgsql-patches/2005-11/msg00173.php

Add:

>
> * Consider increasing internal areas when shared buffers is increased
>
>   http://archives.postgresql.org/pgsql-hackers/2005-10/msg01419.php

Already done:

<
<  o Properly mark all libpq-exported functions with "PQ"
<
<    http://archives.postgresql.org/pgsql-hackers/2005-10/msg00696.php

Done by Peter:

< * Improve the information schema to be more standards-compliant:
<
<   http://archives.postgresql.org/pgsql-bugs/2005-10/msg00028.php

Add:

>
>  o Properly mark all libpq-exported functions with "PQ"
>
>    http://archives.postgresql.org/pgsql-hackers/2005-10/msg00696.php

Add:

>
> * Consider padding the spinlock structure to have spinlock registers
>   on different CPU cache line
>
>   http://archives.postgresql.org/pgsql-hackers/2005-10/msg00498.php

Add item, not fixed:

>
> * Fix ILIKE and regular expressions to handle case insensitivity
>   properly in multibyte encodings
>
>   http://archives.postgresql.org/pgsql-bugs/2005-10/msg00001.php

Remove, already in back branches:

<
<  o Add new version of PQescapeString() that doesn't double backslashes
<    that are part of a client-only multibyte sequence
<
<    Single-quote is not a valid byte in any supported client-only
<    encoding.  This requires using mblen() to determine if the
<    backslash is inside or outside a multi-byte sequence.
<
<  o Add new version of PQescapeString() that doesn't double
<    backslashes when standard_conforming_strings is true and
<    non-E strings are used

Done:

>  o -Port contrib/xml2

Add information schema item:

> * Improve the information schema to be more standards-compliant:
>
>   http://archives.postgresql.org/pgsql-bugs/2005-10/msg00028.php
> * Add URL to more complete multi-byte regression tests
>
>   http://archives.postgresql.org/pgsql-hackers/2005-07/msg00272.php
< * Add URL to more complete multi-byte regression tests
<
<   http://archives.postgresql.org/pgsql-hackers/2005-07/msg00272.php

Add COPY views patch URL:

>    http://archives.postgresql.org/pgsql-patches/2005-09/msg00148.php

Add:

>
>  o Add API for 64-bit large object access
>
>    http://archives.postgresql.org/pgsql-hackers/2005-09/msg00781.php

Keep TODO.detail until item is done:

<  o %Allow pg_dump to use multiple -t and -n switches
>  o %Allow pg_dump to use multiple -t and -n switches  [pg_dump]

Add URL, remove TODO.detail:

<  o %Allow pg_dump to use multiple -t and -n switches [pg_dump]
>  o %Allow pg_dump to use multiple -t and -n switches
>
>    http://archives.postgresql.org/pgsql-patches/2006-01/msg00190.php
>

Add locale/encoding references:

<   Right now only one encoding is allowed per database.
>   Right now only one encoding is allowed per database.  [locale]
> * Add CREATE COLLATE?  [locale]

Add locale TODO.detail emails.

Update link to TODO.detail:

<  o Check WSACancelBlockingCall() for interrupts (win32intr)
>  o Check WSACancelBlockingCall() for interrupts [win32intr]

Add:

>  o Fix problem when cascading referential triggers make changes on
>    cascaded tables, seeing the tables in an intermediate state
>
>    http://archives.postgresql.org/pgsql-hackers/2005-09/msg00174.php
>    http://archives.postgresql.org/pgsql-hackers/2005-09/msg00174.php

Added:

>
> * Add URL to more complete multi-byte regression tests
>
>   http://archives.postgresql.org/pgsql-hackers/2005-07/msg00272.php

Add entry:

> * Allow user-defined types to accept 'typmod' parameters
>   http://archives.postgresql.org/pgsql-hackers/2005-08/msg01142.php
>   http://archives.postgresql.org/pgsql-hackers/2005-09/msg00012.php

Further thoughts about lo_export/lo_import error handling: if one of
the lower-level large object functions fails, it will have already set
a suitable error message --- probably something from the backend ---
and it is not useful to overwrite that with a generic 'error while
reading large object' message.  So remove redundant messages.

Clean up psql's control-C handling to avoid longjmp'ing out of random
places --- that risks corrupting data structures, losing sync with the
backend, etc.  We now longjmp only from calls to readline, fgets, and
fread, which we assume are coded to protect themselves against interrupts
at undesirable times.  This requires adding explicit tests for
cancel_pressed in long-running loops, but on the whole it's far cleaner.
Martijn van Oosterhout and Tom Lane.

Improve error detection and recovery in lo_import()/lo_export().
Problems noted while experimenting with new control-C code for psql.

Do a CHECK_FOR_INTERRUPTS immediately before terminating a fastpath
function call.  Previously, there may have been no CHECK_FOR_INTERRUPTS
at all in the fastpath code path, making it impossible to cancel an
operation such as \lo_import externally.  This addition doesn't ensure
you can cancel, since your SIGINT may arrive while the backend is idle
waiting for the client, but it gives the largest window we can easily
provide.  Noted while experimenting with new control-C code for psql.

Remove:

<  o %Add CSV output format

Add BY clause to PL/PgSQL FOR loop, to control the iteration increment.

Jaime Casanova

Avoid use of C commment inside C comment from recent Win32 int overflow patch.

Prevent Win32 from displaying a popup box on backend crash.  Instead let
the postmaster deal with it.

Magnus Hagander

Add URL for bitmap AM:

>   http://archives.postgresql.org/pgsql-patches/2005-07/msg00512.php

Win32 can't catch the exception thrown by INT_MIN / -1 or INT_MIN * -1,
so on that platform we test for those before the computation and throw
an "out of range" error.

Backpatch to 8.1.X.

Minor code cleanup: make the WIN32 case less gratuitously different from
the other platform-specific cases in ps_status.

Code review for psql multiline history patch(es).  Fix memory leak,
failure to enter commands in history if canceled by control-C, other
infelicities.

Fix Assert failure when a fastpath function call is attempted inside an
already-aborted transaction block.  GetSnapshotData throws an Assert if
not in a valid transaction; hence we mustn't attempt to set a snapshot
for the function until after checking for aborted transaction.  This is
harmless AFAICT if Asserts aren't enabled (GetSnapshotData will compute
a bogus snapshot, but it doesn't matter since HandleFunctionRequest will
throw an error shortly anywy).  Hence, not a major bug.

Along the way, add some ability to log fastpath calls when statement
logging is turned on.  This could probably stand to be improved further,
but not logging anything is clearly undesirable.

Backpatched as far as 8.0; bug doesn't exist before that.

Remove pg_dump wording:

<   pg_get_tabledef(), pg_get_domaindef(), pg_get_functiondef(), and
<   make use of them in pg_dump
>   pg_get_tabledef(), pg_get_domaindef(), pg_get_functiondef()

Add:

<   pg_get_tabledef(), pg_get_domaindef(), pg_get_functiondef()
>   pg_get_tabledef(), pg_get_domaindef(), pg_get_functiondef(), and
>   make use of them in pg_dump

Repair remarkably-inefficient query for dumping large object comments: it
was invoking obj_description() for each large object chunk, instead of once
per large object.  This code is new as of 8.1, which may explain why the
problem hadn't been noticed already.

Revert sampling patch for EXPLAIN ANALYZE; it turns out to be too unreliable
because node timing is much less predictable than the patch expects.  I kept
the API change for InstrStopNode, however.

Cleanups:

<  * Allow a database in tablespace t1 with tables created in
>  o Allow a database in tablespace t1 with tables created in
125c125
<  * Allow reporting of which objects are in which tablespaces
>  o Allow reporting of which objects are in which tablespaces
593c593
<  * Allow GRANT/REVOKE permissions to be inherited by objects based on
>  o Allow GRANT/REVOKE permissions to be inherited by objects based on
596c596
<  * Allow SERIAL sequences to inherit permissions from the base table?
>  o Allow SERIAL sequences to inherit permissions from the base table?

Cleanup:

< * -%Allow pooled connections to list all prepared statements
> * -Allow pooled connections to list all prepared statements

Now ispell dictionary can eat dictionaries in MySpell format,
used by OpenOffice. Dictionaries are placed at
http://lingucomponent.openoffice.org/spell_dic.html
Dictionary automatically recognizes format of files.

Warning. MySpell's format has limitation with compound
word support: it's impossible to mark affix as
compound-only affix. So for norwegian, german etc
languages it's recommended to use original ispell format.
For that reason I don't want to remove my2ispell
scripts, it's has workaround at least for norwegian language.

Add URL.

<    the insert.
>    the insert.
>    http://archives.postgresql.org/pgsql-patches/2005-07/msg00568.php

Update:

< * Add SQL99 WITH RECURSIVE (hierarchical) queries to SELECT
> * Add SQL:2003 WITH RECURSIVE (hierarchical) queries to SELECT

Remove:

< * Rename some /contrib modules from pg* to pg_*

Fix bootstrap.c so that database startup process and bgwriter properly release
LWLocks during a panic exit.  This avoids the possible self-deadlock pointed
out by Qingqing Zhou.  Also, I noted that an error during LoadFreeSpaceMap()
or BuildFlatFiles() would result in exit(0) which would leave the postmaster
thinking all is well.  Added a critical section to ensure such errors don't
allow startup to proceed.

Backpatched to 8.1.  The 8.0 code is a bit different and I'm not sure if the
problem exists there; given we've not seen this reported from the field, I'm
going to be conservative about backpatching any further.