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.

Document that only a table's owner may TRUNCATE it.  Per Keith Worthington.

This patch changes makes some significant changes to how compilation
and parsing work in PL/PgSQL:

- memory management is now done via palloc(). The compiled representation
  of each function now has its own memory context. Therefore, the storage
  consumed by a function can be reclaimed via MemoryContextDelete().

  During compilation, the CurrentMemoryContext is the function's memory
  context. This means that a palloc() is sufficient to allocate memory
  that will have the same lifetime as the function itself. As a result,
  code invoked during compilation should be careful to pfree() temporary
  allocations to avoid leaking memory. Since a lot of the code in the
  backend is not careful about releasing palloc'ed memory, that means
  we should switch into a temporary memory context before invoking
  backend functions. A temporary context appropriate for such allocations
  is `compile_tmp_cxt'.

- The ability to use palloc() allows us to simply a lot of the code in
  the parser. Rather than representing lists of elements via ad hoc
  linked lists or arrays, we can use the List type. Rather than doing
  malloc followed by memset(0), we can just use palloc0().

- We now check that the user has supplied the right number of parameters
  to a RAISE statement. Supplying either too few or too many results in
  an error (at runtime).

- PL/PgSQL's parser needs to accept arbitrary SQL statements. Since we
  do not want to duplicate the SQL grammar in the PL/PgSQL grammar, this
  means we need to be quite lax in what the PL/PgSQL grammar considers
  a "SQL statement". This can lead to misleading behavior if there is a
  syntax error in the function definition, since we assume a malformed
  PL/PgSQL construct is a SQL statement. Furthermore, these errors were
  only detected at runtime (when we tried to execute the alleged "SQL
  statement" via SPI).

  To rectify this, the patch changes the parser to invoke the main SQL
  parser when it sees a string it believes to be a SQL expression. This
  means that synctically-invalid SQL will be rejected during the
  compilation of the PL/PgSQL function. This is only done when compiling
  for "validation" purposes (i.e. at CREATE FUNCTION time), so it should
  not impose a runtime overhead.

- Fixes for the various buffer overruns I've patched in stable branches
  in the past few weeks. I've rewritten code where I thought it was
  warranted (unlike the patches applied to older branches, which were
  minimally invasive).

- Various other minor changes and cleanups.

- Updates to the regression tests.

Add semicolon so snprintf.c goto has a statement to attach to:

nochar:
    /* nothing */
    ; /* semicolon required because a goto has to be attached to a statement */

Use _() macro consistently rather than gettext().  Add translation
macros around strings that were missing them.

Add:

>  * Add internationalized message strings

Add support to port/snprintf.c for position parameter specification:

+ # Determine if printf supports %1$ argument selection, e.g. %5$ selects
+ # the fifth argument after the printf print string.
+ # This is not in the C99 standard, but in the Single Unix Specification (SUS).
+ # It is used in our langauge translation strings.

Nicolai Tufar with configure changes by Bruce.

Update initdb locale/encoding documentation description.  Backpatch to
8.0.X.

Try to get Borland CC to compile.

Backpatch to 8.0.X which doesn't work right now.

Add port mention:

< * Add the client IP address to pg_stat_activity
> * Add the client IP address and port to pg_stat_activity

Update Russian FAQ.

Viktor Vislobokov

Clarify item:

< * Allow server configuration parameters to be remotely modified
> * Allow pg_hba.conf settings to be controlled via SQL
>
>   This would require a new global table that is dumped to flat file for
>   use by the postmaster.  We do a similar thing for pg_shadow currently.
>

Simplify defines

Fix memory leak for timestamp(with and w/o tz) and indexes

Trivial fix: change the reference to further documentation of pathkeys to
point to its new location.

Fix two typos, per report from Hashem Masoud.

Clarify item:

< * Consider use of open/fcntl(O_DIRECT) to minimize OS caching
> * Consider use of open/fcntl(O_DIRECT) to minimize OS caching,
>   especially for WAL writes

Updates to the bibliography. Patch from Michael Fuhr.

Document the "register" and "unregister" pg_ctl subcommands, for use on
Windows. Patch from Magnus Hagander.

Use SnapshotNow instead of SnapshotSelf for reading the catalogs
during flat-file writing.  The only difference is that SnapshotSelf
would consider tuples of the 'current command' within the current
transaction as valid, where SnapshotNow wouldn't.  We can eliminate
the need for this with one extra CommandCounterIncrement call before
we start reading the catalogs.

Remove some no-longer-needed kluges for bootstrapping, in particular
the AMI_OVERRIDE flag.  The fact that TransactionLogFetch treats
BootstrapTransactionId as always committed is sufficient to make
bootstrap work, and getting rid of extra tests in heavily used code
paths seems like a win.  The files produced by initdb are demonstrably
the same after this change.

Rename macro to MAKE_EXPIRED_TUPLES_VISIBLE.

Fix MAKE_ALL_TUPLES_VISIBLE define.

Move define MAKE_ALL_TUPLES_VISIBLE to a more logical place.

I have added a define, MAKE_ALL_TUPLES_VISIBLE, to help people recover
deleted tuples. Of course it is only to be used for disaster recovery.

Flat file cleanup phase 2: make it work for pg_group.  The flat group
file now identifies group members by usesysid not name; this avoids
needing to depend on SearchSysCache which we can't use during startup.
(The old representation was entirely broken anyway, since we did not
regenerate the file following RENAME USER.)  It's only a 95% solution
because if the group membership list is big enough to be toasted out
of line, we cannot read it during startup.  I think this will do for
the moment, until we have time to implement the planned pg_role
replacement for pg_group.