More table markup fixes.

Rework "aliases" column for encodings.

Fix markup typo.

Add missing conversion documentation for certain encodings.

Reorder documented encodings to be alphabetical.

Remove warning about pre-7.2 LATIN5 usage.

Document aliases for our supported encodings.

Add a few encodings that were not documented.

When cloning template0 (or other fully-frozen databases), set the new
database's datallowconn and datfrozenxid to the current transaction ID
instead of copying the source database's values.  This is OK because we
assume the source DB contains no normal transaction IDs whatsoever.
This keeps VACUUM from immediately starting to complain about unvacuumed
databases in the situation where we are more than 2 billion transactions
out from the XID stamp of template0.  Per discussion with Milen Radev
(although his complaint turned out to be due to something else, but the
problem is real anyway).

Fix ALTER DATABASE RENAME to allow the operation if user is a superuser
who for some reason isn't marked usecreatedb.  Per report from Alexander
Pravking.  Also fix sloppy coding in have_createdb_privilege().

Adjust the API for aggregate function calls so that a C-coded function
can tell whether it is being used as an aggregate or not.  This allows
such a function to avoid re-pallocing a pass-by-reference transition
value; normally it would be unsafe for a function to scribble on an input,
but in the aggregate case it's safe to reuse the old transition value.
Make int8inc() do this.  This gets a useful improvement in the speed of
COUNT(*), at least on narrow tables (it seems to be swamped by I/O when
the table rows are wide).  Per a discussion in early December with
Neil Conway.  I also fixed int_aggregate.c to check this, thereby
turning it into something approaching a supportable technique instead
of being a crude hack.

Add pg_buffercache contrib module, from Mark Kirkwood.

Some builds (depends on crypto engine support?) of OpenSSL
0.9.7x have EVP_DigestFinal function which which clears all of
EVP_MD_CTX.  This makes pgcrypto crash in functions which
re-use one digest context several times: hmac() and crypt()
with md5 algorithm.

Following patch fixes it by carring the digest info around
EVP_DigestFinal and re-initializing cipher.

Marko Kreen.

Documention all our supported encodings.

Done:

<  o Allow COPY FROM ... CSV to interpret newlines and carriage
>  o -Allow COPY FROM ... CSV to interpret newlines and carriage

Handle carriage returns and line feeds in COPY CSV mode.

Andrew Dunstan

Done:

* -Add a warning when the free space map is too small

Add warning about the need to increase "max_fsm_relations" and
"max_fsm_relations" for vacuums.  Also improve VACUUM VERBOSE final
message text.

Ron Mayer

Fix snprintf() to properly handle precision specification for %f.

Fix problem with infinite recursion between write_syslogger_file and
elog if the former has trouble writing its file.  Code review for
Magnus' patch to redirect stderr to syslog on Windows (Bruce's version
seems right, but did some minor prettification).

Backpatch both changes to 8.0 branch.

Here's the patch to fix a lot of markup errors in the HTML FAQs. Doesn't
change content (at least not supposed to).

Magnus Hagander

Add fprintf() custom version to libpgport.

Document use of macros for pg_printf functions.

Bump major versions of all interfaces to handle movement of get_progname
from libpq to libpgport in 8.0, and probably other libpgport changes in 8.1.

Define snprintf() to call pg_snprintf() so our own snprintf-like
implementation doesn't export out via libpq and get used by a user
application.

Add 'static' to initdb.c file-global variables.

Mark file-global function and variables as static.

Fix typos.

Hashem Masoud

Fix typos.

Robert Treat

Fix typos.

Robert Treat

Fix typo.

Slight refactoring and optimization of some code in WaitOnLock().

Add:

> * Fix cross-compiling of time zone database via 'zic'

Make the behavior of HAVING without GROUP BY conform to the SQL spec.
Formerly, if such a clause contained no aggregate functions we mistakenly
treated it as equivalent to WHERE.  Per spec it must cause the query to
be treated as a grouped query of a single group, the same as appearance
of aggregate functions would do.  Also, the HAVING filter must execute
after aggregate function computation even if it itself contains no
aggregate functions.

Add spinlock support for Itanium processor with Intel compiler.

Vikram Kalsi

Refactor fork()-related code. We need to do various housekeeping tasks
before we can invoke fork() -- flush stdio buffers, save and restore the
profiling timer on Linux with LINUX_PROFILE, and handle BeOS stuff. This
patch moves that code into a single function, fork_process(), instead of
duplicating it at the various callsites of fork().

This patch doesn't address the EXEC_BACKEND case; there is room for
further cleanup there.

Add comma.

Fix markup of URL.

Add URL for patches queues.

Unbreak out-of-tree builds, by fixing a typo.

Replace an instance of $Id$ and an instance of $Header$ with $PostgreSQL$

Properly implement "Response files" for bcc.  Add URL's to describe the
feature for Win32 and bcc.

Adjust creation/destruction of TupleDesc data structure to reduce the
number of palloc calls.  This has a salutory impact on plpgsql operations
with record variables (which create and destroy tupdescs constantly)
and probably helps a bit in some other cases too.

Rename canonical encodings, per Peter:

UNICODE => UTF8
ALT => WIN866
WIN => WIN1251
TCVN => WIN1258

The old codes continue to work.

Here's a tiny fix for a harmless typo in catalog.c:

Too much space is allocated for tablespace file path, I guess the
directory name used to be "pg_tablespaces" instead of "pg_tblspc" at
some point.

Heikki Linnakangas

Add a note that regression test row ordering differences can be caused
by non-default planner parameter settings.

Revise hash join code so that we can increase the number of batches
on-the-fly, and thereby avoid blowing out memory when the planner has
underestimated the hash table size.  Hash join will now obey the
work_mem limit with some faithfulness.  Per my recent proposal
(hash aggregate part isn't done yet though).

Prevent rank change in case of duplicate search terms

Replace the BufMgrLock with separate locks on the lookup hashtable and
the freelist, plus per-buffer spinlocks that protect access to individual
shared buffer headers.  This requires abandoning a global freelist (since
the freelist is a global contention point), which shoots down ARC and 2Q
as well as plain LRU management.  Adopt a clock sweep algorithm instead.
Preliminary results show substantial improvement in multi-backend situations.

Move snprintf int64 compatibility letters into a NOT_USED block.

Fix misspelling of sysctl, per Stefan Hans.

Use test && test rather than test -a, per Peter.

Fix for %I64d snprintf.

snprintf() %I64d code fix.

Use our own snprintf() only if NLS is enabled, and support %qd and %I64d.

Prevent large allocation in snprintf to hold positional parameters.
Allocated size based on format string.

Another go at making pred_test() handle all reasonable combinations
of AND and OR clauses.  The key point here is that an OR on the
predicate side has to be treated gingerly: we may be able to prove
that the OR is implied even when no one of its components is implied.
For example (x OR y) implies (x OR y OR z) even though no one of x,
y, or z can be individually proven.  This code handles both the
example shown recently by Sergey Koshcheyev and the one shown last
October by Dawid Kuroczko.

Fix snprintf on Win32:

 *  If vsnprintf() is not before snprintf() in this file, snprintf()
 *  will call the system vsnprintf() on MinGW.

Make port snprintf.c finally thread-safe.

SGML improvements to the DML chapter.

Release proclock immediately in RemoveFromWaitQueue() if it represents
no held locks.  This maintains the invariant that proclocks are present
only for procs that are holding or awaiting a lock; when this is not
true, LockRelease will fail.  Per report from Stephen Clouse.

Allow Trace_lock_oidmin to be set to zero; this is a reasonable
representation of not wanting tracing to be limited by object OID.

Simplify/clean up code for varlena types

Make snprintf() use already-defined int64/uint64 typedefs rather than
defining its own.

Done:

> * -Make src/port/snprintf.c thread-safe

Adjust OR indexscan logic to not generate redundant condition-free OR
indexscans involving partial indexes.  These would always be dominated
by a simple indexscan on such an index, so there's no point in considering
them.  Fixes overoptimism in a patch I applied last October.

And while we are on it, I would like to submit minor
changes to make snprintf() vsnprintf() and printf()
functions in src/port/snprintf.c thread-safe.

Nicolai Tufar

Revert the logic for expanding AND/OR conditions in pred_test() to what
it was in 7.4, and add some comments explaining why it has to be this way.
I broke it for OR'd index predicates in a fit of code cleanup last summer.
Per example from Sergey Koshcheyev.

Un-break configure snprintf tests (partly my fault, partly Bruce's).

Move PGAC_FUNC_PRINTF_ARG_CONTROL to just above snprintf 64-bit tests so
its output can be used to select the proper printf outputs.

snprintf.c has no sys/ioctl.h. Trivial patch below:

Magnus Hagander

Implement max() and min() aggregates for array types. Patch from Koju
Iijima, reviewed by Neil Conway. Catalog version number bumped,
regression tests updated.

Because the change to gettext. Needs a bunch of new includes. Patch
follows:

Magnus Hagander

Tab indent all actions in bcc32.mak, and do it on win32.mak too for
consistency.  Backpatch only bcc32.mak to 8.0.X.

Add explicit casts between int4 and boolean. Patch from Sean Chittenden,
editorializing by Neil Conway. Catalog version bumped.

Rename planner stats detail to avoid label confict.

Cause Win32 to output to the event log rather than stderr by default.

Magnus Hagander

Allow Win32 to support the O_SYNC open flag as an wal_sync_method method.

Magnus Hagander

Add documentation on how statistics are used by the planner.

Mark Kirkwood

Add routines to dbsize to return the index size and total relation size.
Improve documentation.

Ed L.

Adjust OS-specific kernel settings to mention old and new BSD methods of
adjusting values:

> But to be on the safe side, it would make sense to do something similar
> to the BSD section, and comment about older distributions maybe needing
> to manipulate /proc/kernel/* directly.

Mark Kirkwood

Finish up the flat-files project: get rid of GetRawDatabaseInfo() hack
in favor of looking at the flat file copy of pg_database during backend
startup.  This should finally eliminate the various corner cases in which
backend startup fails unexpectedly because it isn't able to distinguish
live and dead tuples in pg_database.  Simplify locking on pg_database
to be similar to the rules used with pg_shadow and pg_group, and eliminate
FlushRelationBuffers operations that were used only to reduce the odds
of failure of GetRawDatabaseInfo.
initdb forced due to addition of a trigger to pg_database.

Clarify PGPASSWORD usage:

! authentication.  Use of this environment variable is not
! recommended for security reasons (some operating systems
! allow non-root users to see process environment variables via
! <application>ps</>); instead consider using  the
! <filename>~/.pgpass</> file (see <xref linkend="libpq-pgpass">).

Correct overstatement in locking docs: we said ExclusiveLock is never
taken automatically, but this is only true with respect to user tables.

State PGPASSWORD is "not recommended" rather than "depricated".

In accordance to
http://www.pgsql.ru/db/mw/msg.html?mid=2045361

change TimeATD to/from Datum macros.

Re-initdb is needed.

Add linking from /port to bcc makefile.

Fix float8->int64 transformation

Please find enclosed a patch, per Dennis Bj<C3><B6>rklund, that uses -f
for input files rather than <.  This makes error messages, &c. more
expressive.

David Fetter

Fix plpgsql error message for dropped temporary tables.

Clarify object ownership language to prevent confusion with database
ownership.

Updatge wording.

Fix markup.

Fix HTML markup and add NULL sorting item to existing NULL FAQ item.

Greg Sabino Mullan

Add:

> * Improve psql's handling of multi-line queries

Add wording about UTF8:

<    like towupper().
>    like towupper().  However, UTF8 already works with normal
>    locales but provides no ordering.

Done:

>  o -Change PL/PgSQL to use palloc() instead of malloc()

We aren't supposed to try to run test programs until after we've
verified that AC_TRY_RUN works.

Clean up printf arg-control test, per Kurt Roeckx.

My patch this morning was overly hasty; revert code to original state.

Minor code cleanup: remove a variable that was assigned to but never
subsequently referenced.

Found by: Coverity
Fixed by: Sean Chittenden

This patch optimizes the md5_text() function (which is used to
implement the md5() SQL-level function). The old code did the
following:

1. de-toast the datum
2. convert it to a cstring via textout()
3. get the length of the cstring via strlen()

Since we are treating the datum context as a blob of binary data,
the latter two steps are unnecessary. Once the data has been
detoasted, we can just use it as-is, and derive its length from
the varlena metadata.

This patch improves some run-of-the-mill md5() computations by
just under 10% in my limited tests, and passes the regression tests.

I also noticed that md5_text() wasn't checking the return value
of md5_hash(); encountering OOM at precisely the right moment
could result in returning a random md5 hash. This patch corrects
that. A better fix would be to make md5_hash() only return on
success (and/or allocate via palloc()), but since it's used in
the frontend as well I don't see an easy way to do that.

Un-break plpgsql build by removing unwanted _() usage.
This would be a completely inappropriate place to apply localization
anyway.

Properly undef _(x) gettext macro.