Move resetting of currSchema to a spot where it's not vulnerable to getting
missed in possible future partial-drop scenarios.  Improve comment.
Per report from Zoltan Boszormenyi.

Add regression tests for MONEY type.

Save another little bit of planner overhead on simple queries, by having
clauselist_selectivity skip some analysis that's useless when there's only
one clause in the given list.  Actually this can win even for not-so-simple
queries, because we also apply clauselist_selectivity to sublists such as the
quals matching an index; which are likely to have only a single entry even
when the total query is quite complicated.

If pg_dump drops a schema, make sure 'search_path' it SET the next time
it is needed --- basically don't remember the previous 'search_path'
after a drop schema because it might have failed but be created later.

Clarify how MONEY trims off trailing thousands separator.

Make the MONEY data type have a thousands separator != decimal symbol,
if the locale has the thousands separator as "".  This now matches the
to_char and psql numericlocale behavior.  (Previously this data type was
basically useless for such setups.)

Borland BCC does not support SSPI, per cnliou9@fastmail.fm.

Borland CC 5.5.1 needs ssize_t, per cnliou9@fastmail.fm.

Change fix_scan_expr() to avoid copying the input node tree in the common case
where rtoffset == 0.  In that case there is no need to change Var nodes,
and since filling in unset opfuncid fields is always safe, scribbling on the
input tree to that extent is not objectionable.  This brings the cost of this
operation back down to what it was in 8.2 for simple queries.  Per
investigation of performance gripe from Guillaume Smet.

Avoid uselessly building a duplicate of the original clause in trivial cases
where the EquivalenceClass machinery is unable to deduce anything more from a
simple "var = const" qual clause.  There are probably some more cases where
this could be done, but this seems to take care of most of the added overhead
for simple queries.  Per gripe from Guillaume Smet.

In passing, fix a problem that was exposed by this change:
reconsider_outer_join_clause and friends were passing the wrong relids to
build_implied_join_equality, resulting in RestrictInfos with the wrong
required_relids.  This mistake was masked in typical cases since the bogus
RestrictInfos would never have escaped from the EquivalenceClass machinery,
but I think there might be corner cases involving "broken" ECs where there
would have been a visible failure even without the new optimization.  In any
case the code was certainly not operating as intended.

Fix white space in MONEY type code.  Rename 'comma' to more generic
'ssymbol' as used in previous function.

Update text:

< * Prevent long-lived temporary tables from causing frozen-Xid advancement
> * Prevent long-lived temporary tables from causing frozen-xid advancement
>
>    The problem is that autovacuum cannot vacuum them to set frozen xids;
>    only the session that created them can do that.
>
>
>

Fix buggy usage of vsnprintf in PL/Python by removing it altogether, instead
relying on stringinfo.c.  This fixes a problem reported by Marko Kreen, but I
didn't use his patch, per subsequent discussion.

Fix "Overall Page Layout" table. The second row should be ItemIdData, not
ItemPointerData.

Actually ... it's pretty silly that parse_oper.c doesn't set up the
opfuncid of an OpExpr initially, considering that it has the information
at hand already.  We'll still treat opfuncid as a cache rather than a
guaranteed-valid value, but this change saves one more syscache lookup
in the normal code path.

Save one syscache lookup when examining volatility or strictness of
OpExpr and related nodes.  We're going to have to set the opfuncid of
such nodes eventually (if we haven't already), so we might as well
exploit the opportunity to cache the function OID.  Buys back some
of the extra planner overhead noted by Guillaume Smet, though I still
need to fool with equivclass.c to really respond to that.

Add more comments about thousands separator handling.

Prevent Perl from introducing a possibly-incompatible definition of type
"bool" into plperl.c.  This has always been a hazard since Perl allows a
platform-specific choice to define bool as int rather than char, but
evidently this didn't happen on any platform we support ... until OS X 10.5.
Per report from Brandon Maust.

Back-patch as far as 8.0 --- a bit arbitrary, but it seems unlikely anyone
will be trying to port 7.x onto new platforms.

Add comments about thousands separator logic.

Document that doc builds now build indexes by default, unless "draft" is
used.

WSATYPE_NOT_FOUND was already defined for BCC so don't redefine it
(conflicting values).

Fix build of libpq for Borland CC.

When setting default thousands separator when locale has "", use logic
so new thousands separator doesn't match decimal symbol.

Fix typo in comment.

Clarify the syntax rules for raw tsvector input.

Fix bogus length calculation that could lead to crash if the string
happened to be right up against the end of memory, per report from
Matt Magoffin.  While at it, avoid useless multiple copying of string
by not depending on xmlStrncatNew.

Fix pg_ctl -t to register -t value, per Alvaro.

Split release notes up into smaller sections; easier to read.

Fix some missed usages of 'HTML tag' and 'HTML entity'.

Make "contrib/" tag consistent.

Split out data types and function entries into separate sections in the
release notes.

Release note adjustments suggested by IRC users.

Adjust contrib/tsearch2 regression results to use XML tag and XML entity descriptions, as now used by core text search default parser.

Reorder a few release notes items for consistency.

Trim down release section headings (no need for Changes).

Re-order some items as migration changes.

Change descriptions of entity and tag objects to "XML entity" and "XML tag".
Allow tag and entity names that follow XML rules. Provide for hexadecimal
as well as decimal numeric entities. Adjust code names to coincide with
new descriptions.

Split "server changes" up into smaller sections in release notes.

Markup fix.

More release notes wording cleanups.

Update COMMENT item:

<  o Prevent COMMENT ON dbname from issuing a warning when loading
<    into a database with a different name, perhaps using COMMENT ON
<    CURRENT DATABASE
>  o Change pg_dump so that a comment on the dumped database is
>    applied to the loaded database, even if the database has a
>    different name.  This will require new backend syntax, perhaps
>    COMMENT ON CURRENT DATABASE.

Update wording:

<  o Allow COMMENT ON dbname to work when loading into a database
<    with a different name, perhaps using COMMENT ON CURRENT
<           DATABASE
>  o Prevent COMMENT ON dbname from issuing a warning when loading
>    into a database with a different name, perhaps using COMMENT ON
>    CURRENT DATABASE

Update comment.

Add suggested syntax for COMMENT ON DATABASE:

<    with a different name
>    with a different name, perhaps using COMMENT ON CURRENT
>           DATABASE
>

Add comment about COMMENT ON DATABASE failing for differently-named
databases.

Add:

>  o Allow COMMENT ON dbname to work when loading into a database
>    with a different name

Improve description of implicit-text-casting compatibility issue.
A bunch of other desultory copy-editing, too.

Minor improvement in documentation of temp_tablespaces: avoid grammatical
ambiguity and say what the default value is.

Remove no-longer-accurate claim that REINDEX won't invalidate cached
plans.

Wordsmith improvements to first half of release notes, move TEXT cast
item to #1 imcompatibility slot.

Simplify TEXT cast working;  spelling fixes to release notes.

Improve GIN index build's tracking of memory usage by using
GetMemoryChunkSpace, not just the palloc request size.  This brings the
allocatedMemory counter close enough to reality (as measured by
MemoryContextStats printouts) that I think we can get rid of the arbitrary
factor-of-2 adjustment that was put into the code initially.  Given the
sensitivity of GIN build to work memory size, not using as much of work
memory as we're allowed to seems a pretty bad idea.

GIN index build's allocatedMemory counter needs to be long, not uint32.
Else, in a 64-bit machine with maintenance_work_mem set to above 4Gb,
the counter overflows and we never recognize having reached the
maintenance_work_mem limit.  I believe this explains out-of-memory
failure recently reported by Sean Davis.

This is a bug, so backpatch to 8.2.

Fix inconsistent message wording, and split off a couple of duplicated strings.

Repair still another bug in the btree page split WAL reduction patch:
it failed for splits of non-leaf pages because in such pages the first
data key on a page is suppressed, and so we can't just copy the first
key from the right page to reconstitute the left page's high key.
Problem found by Koichi Suzuki, patch by Heikki.

Fix a few mentions of "Postgres" that slipped into the release notes.

Add release note that "Postgres" is now documented as an acceptable
alias.

Simplify wording.

Add FAQ items about "Postgres" alias usage, remove old reference.

Fix tsvectorout() and tsqueryout() to escape backslesh, add test of that.
Patch by Bruce Momjian <bruce@momjian.us>

Backpatch is needed, but it's impossible to apply it directly

Fix markup that doesn't work in HISTORY generation.

Re-do configure with autoconf 2.59

update files for beta3

Add a couple of notes pointing out that GIN index build time is very
sensitive to maintenance_work_mem (something I just learned the hard
way).

Small comment spacing improvement.

Better guard token used by pgindent.

Run pgindent on remaining files now that LOOPBYTE is a usable macro.

Cleanup for new else/comment handling.

Fix stupid typo in recently-added code :-(

Add snb_ru_init(internal) to list of stub functions in tsearch2
compatibility module.  Needed to support loading of 8.1-era tsearch2
configuration data.

Modify LOOPBYTE/LOOPBIT macros to be more logical;  rather than have the
for() body passed as a parameter, make the macros act as simple headers
to code blocks.

This allows pgindent to be run on these files.

Fix pgindent to properly handle 'else' and single-line comments on the
same line;  previous fix was only partial.  Re-run pgindent on files
that need it.

Re-run pgindent with updated list of typedefs.  (Updated README should
avoid this problem in the future.)

Beef up README instructions, again.

CUpdate README to suggest 'gmake distclean'.  Add library typedefs.

Update pgtools README to be clearer about typdefs.

Update pgindent with current typedefs.

When logging the recovery.conf parameters, show them quoted as they would
appear in the configuration file.

Add blank lines to pgindent.

Use ftruncate() not truncate() in mdunlink.  Seems Windows doesn't
support the latter.

Update timezone data files to release 2007i of the zic database.

pgindent run for 8.3.

Translation updates

Prevent re-use of a deleted relation's relfilenode until after the next
checkpoint.  This guards against an unlikely data-loss scenario in which
we re-use the relfilenode, then crash, then replay the deletion and
recreation of the file.  Even then we'd be OK if all insertions into the
new relation had been WAL-logged ... but that's not guaranteed given all
the no-WAL-logging optimizations that have recently been added.

Patch by Heikki Linnakangas, per a discussion last month.

Update config.guess and config.sub

Add more detail for non-TEXT cast release item.

Message improvements

Add to synopsis that timeout option also applies to stop mode

Make help synopsis consistent

Make synopses of -t option consistent

Document that /contrib/tsearch2 now contains a compatibility interface.

Adjust example to reduce confusion between a tsvector column and
an index, per Simon.

Add a rank/(rank+1) normalization option to ts_rank().  While the usefulness
of this seems a bit marginal, if it's useful enough to be shown in the manual
then we probably ought to support doing it without double evaluation of the
ts_rank function.  Per my proposal earlier today.

Fix incorrect calculation of elapsed_xlogs.  Itagaki Takahiro

Add an Accept parameter to "simple" dictionaries.  The default of true
gives the old behavior; selecting false allows the dictionary to be used
as a filter ahead of other dictionaries, because it will pass on rather
than accept words that aren't in its stopword list.
Jan Urbanski

Mention that GSSAPI is preferred over Kerberos authentication because
GSSAPI is industry-standard.

Add note about deprecating krb5 authentication in favour of GSSAPI,
per discussions (a long time ago). Documentation only, we keep full
support in the code.

Capitalize "Migration to Version".

Properly capitalize "Migrate To Version";  seems we always had that
wrong.

Markup cleanup.

Reference pgfoundry instead of gborg.