Revert some more changes back the way it's in CVS HEAD, now that we
do startup checkpoints again.

Change error message if recovery ends before reaching consistency.

Bring back startup checkpoints. Plus some other small changes

Don't rename recovery.conf out of the way until the first checkpoint, like
it was done in Simon's original . And fix some other comments pointed out
by Simon.

Merge commit 'origin/master' into recoveryinfra

Fix FatalError handling. Bug pointed out by Fujii-san.

Added result checks for calls to gmtime().

Remove minSafeStartPoint, advance minRecoveryPoint instead. Advance it in
XLogFlush, instead of when a WAL file is restored.

Argh, wrong line copied again.

Fixed copy&paste mistake that made library use uninitialized variable.

allow alternative names for tclsh used on Windows

Add a failure check for syslogger's use of _beginthreadex(), and remove
unnecessary thread address output parameter, to make this code look more
like that in pg_restore.

Document the longstanding behavior of LIMIT NULL and OFFSET NULL,
per gripe from David Wheeler that this was mentioned nowhere.
In passing, editorialize a bit on the description of the SQL:2008
equivalent syntax.

Fix plpgsql to not treat INSERT INTO as an INTO-variables clause anywhere
in the string, not just at the start.  Per bug #4629 from Martin Blazek.

Back- to 8.2; prior versions don't have the problem, at least not in
the reported case, because they don't try to recognize INTO in non-SELECT
statements.  (IOW, this is really fallout from the RETURNING .)

Provide for parallel restoration from a custom format archive. Each data and
post-data step is run in a separate worker child (a thread on Windows, a child
process elsewhere) up to the concurrent number specified by the new pg_restore
command-line --multi-thread | -m switch.

Andrew Dunstan, with some editing by Tom Lane.

Allow reloption names to have qualifiers, initially supporting a TOAST
qualifier, and add support for this in pg_dump.

This allows TOAST tables to have user-defined fillfactor, and will also
enable us to move the autovacuum parameters to reloptions without taking
away the possibility of setting values for TOAST tables.

When changing a regression test file one should also change the expected output file.

Set memory segment to '0', just to be sure.

Fixed auto allocation for binary data types.

Fixed bug in %s handling.

Put pg_user_mapping section into the right (alphabetical) order.

Fix unnest() to handle a toasted input array safely.  Per report from
Alvaro.

Check that connection limit is within valid range. IOW, not < -1.

It's missing in older versions too, but it doesn't seem worth
back-porting. All negative are just harmlessly treated as "no limit", and
tightening the check might even brake an application that relies on it.

Comment enhancements

Change the signaling between startup process and post Startup process
now tells postmaster about the start, consistent point, and completion of
WAL redo, using signals. It now returns with exit code 0 when WAL replay is
aborted with SIGTERM. Background writer is launched as soon as we start WAL
redo, so that it can perform restartpoints before reaching consistent state.
Postmaster handles unexpected death of the startup process gracefully. In
case of a fast shutdown, bgwriter performs a "shutdown restartpoint", similar
to a shutdown checkpoint.

Make messages consistent with existing ones

Whitespace fix in message

Update back-branch release notes.

Update time zone data files to tzdata release 2009a: introduces Asia/Kathmandu
as the preferred spelling of that zone name, corrects historical DST
information for Switzerland and Cuba.

Move the check for whether we've reached a safe starting point *before*
checking if we've reached the recovery target. Otherwise, if the first safe
starting point is at the exact same record as the recovery target, we'd
fail to start up.

Replace argument-checking Asserts with regular test-and-elog checks in all
encoding conversion functions.  These are not can't-happen cases because
it's possible to create a conversion with the wrong conversion function
for the specified encoding pair.  That would lead to an Assert crash in
an Assert-enabled build, or incorrect conversion otherwise, neither of
which is desirable.  This would be a DOS issue if production databases
were customarily built with asserts enabled, but fortunately that's not so.
Per an observation by Heikki.

Back- to all supported branches.

Wait for bgwriter to die after startup process ends in PM_RECOVERY. More
needs to be done, but it's a start.

Fix incorrect dereferencing of char* to array's index.
Per Tommy Gildseth <[email protected]> report

Make IsRecoveryProcessingMode() init ThisTimeLineID, to make sure
it's always set before it's used.

Message fix

minor comment & whitespace changes

Remove useless flags argument from LogCheckpointEnd

Remove log_restartpoints, use log_checkpoints instead. Refactor
"restartpoint/checkpoint starting" messages a bit, so that the flags
are printed for restartpoints too.

Fix bug with multiple evaluation of tsearch2 compatibility trigger, trigger
data should be restored.
Backpatch only for 8.3 because previous versions haven't such layer.

Fix issues pointed out by Fujii Masao.

Change warning-silencing code not to cast the pointer type, instead
casting the value of the variable later.

Per comments from Tom.

Go over all OpenSSL return values and make sure we compare them
to the documented API value. The previous code got it right as
it's implemented, but accepted too much/too little compared to
the API documentation.

Per comment from Zdenek Kotala.

Support running as a service on Windows 7, by not specifying
the JOB_OBJECT_UILIMIT_HANDLES flag.

Dave Page & Magnus Hagander

Revert some useless whitespace changes

Fix two bugs: minSafeStartPoint must never move backwards, and
recoveryLogRestartpoints needs to be accessible to bgwriter.

Add logging of startup checkpoint

Remove the signaling to request hurrying a restartpoint. I don't see any
reason for that. Instead, make sure that the bgwriter processes a checkpoint
request as such, and not as a restartoint request, if we've just ended
recovery.

Comment changes. Inline GetRedoLocationForCheckpoint into CreateCheckPoint
again.

rename some variables + other minor cleanup

Disable xlog-triggered checkpoints and progress-tracking during recovery
again. Plus a bunch of other small stuff.

Revert the recovery.conf file renaming change. Remove the (remains of the) code
to pass the redo location of the first checkpoint from startup process to
bgwriter. Silence LOG warning about bgwriter not being started yet when the
startup checkpoint is requested.

Refactor the signaling between bgwriter & startup process

Silence compiler warning on win32.

ITAGAKI Takahiro

Silence compiler warnings in new win32 locale code
when building on mingw.

ITAGAKI Takahiro

Revert updatable views

Remove padding in XLogCtl; might be a good idea, but let's keep it simple
for now.

Also remove the XLogCtl->mode_lck spinlock, which is pretty pointless for
a single boolean that's only written by one process.

comment fixes

Allow extracting and parsing of reloptions from a bare pg_class tuple, and
refactor the relcache code that used to do that.  This allows other callers
(particularly autovacuum) to do the same without necessarily having to open
and lock a table.

Message format tuning

Tweak the existing special case for AIX in pg_getaddrinfo_all() to handle
yet another failure case in AIX's getaddrinfo().  Per report and 
by Andrew Chernow.

Document that SELECT ... ORDER BY .. FOR UPDATE/SHARE might return
results out of order because of locking, per bug report 4593

Remove knownProcessingMode variable.

Message style review

Import Simon's recovery infrastructure  v9

Put back fast-path for the case that there's no backup blocks in
RestoreBkpBlocks. Went missing in my recent refactoring , as pointed
out by Simon's hot standby .

Recreate ecpg_config.h with a stamp file created by config.status, same
way pg_config.h is handled.  This avoids reruns of config.status on every
build, if configure has been rerun.

Support column-level privileges, as required by SQL standard.

Stephen Frost, with help from KaiGai Kohei and others

Only skip pages marked as clean in the visibility map, if the last 32
pages were marked as clean as well. The idea is to avoid defeating OS
readahead by skipping a page here and there, and also makes it less likely
that we miss an opportunity to advance relfrozenxid, for the sake of only
a few skipped pages.

Update comments to reflect that tgenabled is not a boolean anymore.

Jonah Harris, with minor tinkering by me.

Automatic view update rules

Bernd Helmle

Explicitly bind gettext to the correct encoding on Windows.

Original  from Hiroshi Inoue.

Remove stray ' character at end of line.

For some reason it broke OpenBSD compile even when it's inside a
#ifdef WIN32 block.....

Forgot to handle the new SPI_OK_REWRITTEN result code in a couple places.

Add new SPI_OK_REWRITTEN return code to SPI_execute and friends, for the
case that the command is rewritten into another type of command. The old
behavior to return the command tag of the last executed command was
pretty surprising. In PL/pgSQL, for example, it meant that if a command
was rewritten to a utility statement, FOUND wasn't set at all.

Make win32 builds always do SetEnvironmentVariable() when doing putenv().
Also, if linked against other versions than the default MSVCRT library
(for example the MSVC build which links against MSVCRT80), also update
the cache in the default MSVCRT at the same time.

This should fix the issues with setting LC_MESSAGES on the MSVC build.

Original  from Hiroshi Inoue and Hiroshi Saito, much rewritten
by me.

Make some strings translatable again that were accidentally removed in
earlier  to fix "printf-arguments".

Install .mo files for NLS as <program>-<version>.mo when building with MSVC
as well. Same change as made earlier for autoconf builds.

Per Hiroshi Inoue.

Add a new option to RestoreBkpBlocks() to indicate if a cleanup lock should
be used instead of the normal exclusive lock, and make WAL redo functions
responsible for calling RestoreBkpBlocks(). They know better what kind of a
lock they need.

At the moment, this just moves things around with no functional change, but
makes the hot standby  that's under review cleaner.

Do not msgmerge against /dev/null; merge against the pot file itself
instead.

Revise the permission checking on user mapping DDL commands.
CREATE/ALTER/DROP USER MAPPING are now allowed either by the server owner or
by a user with USAGE privileges for his own user name.  This is more or less
what the SQL standard wants anyway (plus "implementation-defined")

Hide information_schema.user_mapping_options.option_value, unless the current
user is the one associated with the user mapping, or is the server owner and
the mapping is for PUBLIC, or is a superuser.  This is to protect passwords.

Also, fix a bug in information_schema._pg_foreign_servers, which hid servers
using wrappers where the current user did not have privileges on the wrapper.
The correct behavior is to hide servers where the current user has no
privileges on the server.

Update \d*S documentation to be more accurate;  system objects are added
to the display, not restricted in the display; new text:

        The letter <literal>S</literal> adds the listing of system
        objects; without <literal>S</literal>, only non-system
        objects are shown.

Make psql \d and \dt consistent for system tables, i.e prevent \d from
showing system tables, make \dS pattern show system table details, and
have \dtS show system and _user_ tables, to be consistent with other \d*
commands.

Document RELKIND_SPECIAL reference in psql, from <= 8.1.X.

Fix accidental (I suppose) introduction of non-ASCII quote marks.

Use the new text domain names ("postgres-8.4" instead of "postgres")

Hiroshi Inoue

Avoid regression test cases that fail in locales that sort "v" and "w"
equally (in glibc: et_EE, sv_SE, tk_TM).  It turns out that this was
already taken care of previously by select_1.out, which I had forgotten to
update for an unrelated change.  But might as well avoid the issue
altogether.

Alter regression test cases that rely on the sort order of "aa".  Some
locales (da_DK, fo_FO, kl_GL, nb_NO, nn_NO in glibc) sort "aa" after "z".

Alter the regression test cases that rely on the sort order of "ch" between
"cg" and "ci".  This eliminates a test failure on the following glibc
locales: br_FR, cs_CZ, cy_GB, es_EC, es_US, hsb_DE, ig_NG, ik_CA, sk_SK.

Message wordsmithing

Fix a pg_dump output ordering problem introduced in 8.3 by the addition of
array types for composite types.  Although pg_dump understood it wasn't
supposed to dump these array types as separate objects, it must include
them in the dependency ordering analysis, and it was improperly assigning them
the same relatively-high sort priority as regular types.  This resulted in
effectively moving composite types and tables up to that same high priority,
which broke any ordering requirements that weren't explicitly enforced by
dependencies.  In particular user-defined operator classes, which should come
out before tables, failed to do so.  Per report from Brendan Jurd.

In passing, also fix an ill-considered decision to give text search objects
the same sort priority as functions and operators --- the sort result looks
a lot nicer if different object types are kept separate.  The recent
foreign-data  had copied that decision, making the sort ordering even
messier :-(

Fix an ungrammatical statement.

Fix typo.

Manual attempt to update this file.

plpython_error.out is for Python 2.4, plpython_error_3.out is for Python 2.5,
as it was previously.

Add vacuum_freeze_table_age GUC option, to control when VACUUM should
ignore the visibility map and scan the whole table, to advance
relfrozenxid.

Wordsmithing

Remove period from primary error message.

Change explanation of pg_switch_xlog()'s return value to match code.

Fix URL generation in headline. Only tag lexeme will be replaced by space.
Per http://archives.postgresql.org/pgsql-bugs/2008-12/msg00013.php

Fix generation too long headline with ShortWords.
Per http://archives.postgresql.org/pgsql-hackers/2008-09/msg01088.php