Fix some portability issues (reliance on gcc-isms).

Fix portability issues with functions that don't match their declaration.

Add missing casts to unsigned char in recently-added isspace() calls.

Adjust expected regression outputs for plan changes caused by recent
'fuzzy cost comparison' patch.  Mea culpa for not having noticed this
when I committed the patch.

Replace TupleTableSlot convention for whole-row variables and function
results with tuples as ordinary varlena Datums.  This commit does not
in itself do much for us, except eliminate the horrid memory leak
associated with evaluation of whole-row variables.  However, it lays the
groundwork for allowing composite types as table columns, and perhaps
some other useful features as well.  Per my proposal of a few days ago.

Improve comments on USERLIMIT GUC processing.

Add description of log_statement_stats's function (total statement stats
rather than per-stage stats).

Back out tutorial changes:

---------------------------------------------------------------------------

1.  In keeping with the recent discussion that there should be more
said about views, stored procedures, and triggers, in the tutorial, I
have added a bit of verbiage to that end.

2.  Some formatting changes to the datetime discussion, as well as
addition of a citation of a relevant book on calendars.

Christopher Browne

Add psql backslash command discussion.

Add:

* Move psql backslash database information into the backend, use
nmumonic commands? [psql]

Remove:

* Prevent unneeded quoting in psql \d output using fmtId()

Fix markup errors from recent patch.

1.  In keeping with the recent discussion that there should be more
said about views, stored procedures, and triggers, in the tutorial, I
have added a bit of verbiage to that end.

2.  Some formatting changes to the datetime discussion, as well as
addition of a citation of a relevant book on calendars.

Christopher Browne

Add:

> * Add GUC variable to allow output of interval values in ISO8601 format

Add mention of how to use \df to find functions using/returning specific
data types.

Fix to_char for 1 BC.  Previously it returned 1 AD.

Fix to_char(year) for BC dates.  Previously it returned one less than
the current year.

Add documentation mentioning that there is no 0 AD.

Cleanup vectors of GISTENTRY and eliminate problem with 64-bit strict-aligned
boxes. Change interface to user-defined GiST support methods union and
picksplit. Now instead of bytea struct it used special GistEntryVector
structure.

Use fuzzy comparison of path costs in add_path(), so that paths with the
same path keys and nearly equivalent costs will be considered redundant.
The exact nature of the fuzziness may get adjusted later based on current
discussions, but no one has shot a hole in the basic idea yet ...

Fixed possible segfault in type.c (by Juergen Cappel)

Update FAQ from Robert Treat

Some further editorializing on README.CVS.

Clean up thread test program.

Add test for thread-safe errno to thread test program.

Add comment about psql \s (history display):

        /* This scrolls off the screen when using /dev/tty */

Clean up function call arg appearance.

Change small 'if/else' test to use Max().

Fix brain-dead placement of global variable declaration.

Now that we are allowing index opclasses to contain operators that are
only stable and not immutable, pred_test_simple_clause has to guard
against making invalid deductions.  Add a test for immutability of
the selected test_op.

Add -D_REENTRANT for Solaris threading.

More cross-reference work, other minor SGML fixes and improvements.

Update gcc solaris flags.

Fix for SSL compile.

Convert some GUC variable references to links.

1 Minimize memory allocation for void (but not null) value.
2 Add silly ordering for ts_vector to aim grouping, union, except etc. Don't use BTree opclass (tsvector_ops).

Add a more useful error message for the case where someone tries to pass
a whole row or record variable into a SQL function.  Eventually this case
should be made to actually work, but for now this is better than what it
did before.

Replace max_expr_depth parameter with a max_stack_depth parameter that
is measured in kilobytes and checked against actual physical execution
stack depth, as per my proposal of 30-Dec.  This gives us a fairly
bulletproof defense against crashing due to runaway recursive functions.

Add:

* Have pg_dump use multi-statement transactions for INSERT dumps

Adjust error message wording per Andrew Dunstan's advice.

Make a cross-reference in the ALTER TABLE ref page into a <xref>.

Update description error text:

        (errmsg("no socket configured for listening")));

Here's a patch implementing the "thread method" to workaround the bug
with socket calls in signal handlers (APC) on Win32. See details in mail
to pgsql-hackers-win32 a couple of minutes ago.

Magnus Hagander

Allow unlink/rename of files open by another process on Win32, using a
special Win32 open flag FILE_SHARE_DELETE.

Claudio Natoli

Fix a minor typo in the log_line_prefix docs and make a couple other
tiny improvements.

Add thread locking to SSL and Kerberos connections.

I have removed the docs mentioning that SSL and Kerberos are not
thread-safe.

Manfred Spraul

Document fallback to tcp/ip localhost connection from psql on ports
without unix domain sockets (win32).

Fix some whitespace formatting, and remove an overly-verbose
parameter description: postgresql.conf is not the place for
documentating the functionality of a GUC var.

>>Also, what is the default connection mode of psql? It should probably be
>>equivalent to "-h localhost", shouldn't it?
>>
>>
>
>Now that is something I had not thought of.  Seems we can assume a Win32
>psql can never use unix domain sockets, so defaulting that to localhost
>is a good solution too.

Andrew Dunstan

Use the new GUC variable default_with_oids in pg_dump, rather than using
WITH/WITHOUT OIDS in dump files.  This makes dump files more portable.

I have updated the pg_dump version so old binary dumps will load fine.

Pre-7.5 dumps use WITHOUT OIDS in SQL were needed, so they should be
fine.

Update the float8 regression tests: they should now pass on FreeBSD 4.x
and (in theory) QNX. Thanks to Chris KL for the report.

PQresult should be PGresult.

Fix small typo.

Repair misleading description of MOVE's command tag result.

Use dollar-quoting for function bodies, unless disabled with
--disable-dollar-quoting.

Andrew Dunstan

When changing select() calls for delays into pg_usleep(), two comments
in s_lock.c were not updated, and still refers to select. Made my grep
hit the wrong files, so I figured a simple patch was in order.. (other
refs in the same comment block was changed..)

Magnus Hagander

Upgrade ALTER TABLE DROP COLUMN so that it can drop an OID column, and
remove separate implementation of ALTER TABLE SET WITHOUT OIDS in favor
of doing a regular DROP.  Also, cause CREATE TABLE to account completely
correctly for the inheritance status of the OID column.  This fixes
problems with dropping OID columns that have dependencies, as noted by
Christopher Kings-Lynne, as well as making sure that you can't drop an
OID column that was inherited from a parent.

Minor SGML improvements.

Fix typo in postmaster reference page: from Tom.

fflush() FILE buffer to descriptor so stat call gets proper size in fseeko.c.

Fixed problem with pg_dump tar backups.  Only happens on platforms that
use our port/fseeko.c, which is currently BSD/OS and NetBSD.

Improve the locale and character set docs, add some <xref>s pointing
to the character set docs where appropriate, and improve the postmaster
reference page. Character set cross-refs suggested by Gavin Kistner.

Fix getpwuid_r call:

    *result = getpwuid_r(uid, resultbuf, buffer, buflen);

Replace the virtual_host and tcpip_socket parameters with a unified
listen_addresses parameter, as per recent discussion.  The default behavior
is now to listen on localhost, which eliminates the need for the -i
postmaster switch in many scenarios.

Andrew Dunstan

Before deciding we can use a socket for statistics collection, test to
ensure that it actually passes data.  This catches cases such as a kernel
packet filter rule that makes the socket useless.

Andrew Dunstan

Add missing extern for optind.

Clean up rather poor description of the difference between INHERITS and
LIKE.  Per gripe from Patrick Samson.

Standardize output buffer size and display format for strftime;
followup to complaint from Korean User's Group.

Disasble threads on unixware until other platforms report a problem.

Increase xlog str_time() static string variable, per Korean User's Group.

In working through a pg_autovacuum problem with Joe Conway (which turned
out to be the same problem reported by Cott Lang which the previous
patch resolved) a new bug was uncovered when running with a debug level
of greater than 1.

This patch resolves this new found bug and fixes some of the other
debugging output to be more consistent.

Please apply to both HEAD and the 7.4 branch.

Matthew T. O'Connor

I just noticed that \dp outputs "Table" to indicate relations (tables,
sequences and views).  This patch allows it to handle views and
sequences.

Euler Taveira de Oliveira

Adds DLLIMPORT modifier to check_function_bodies

Claudio Natoli

Add mention of "-D_POSIX_PTHREAD_SEMANTICS" enabling 5-arg getpwuid_r().

Add timestamp-versus-timestamptz cross-type comparison functions,
flesh out the index operator classes to include these.  In passing,
fix erroneous volatility marking of ACL functions.

Revise syntax-error reporting behavior to give pleasant results for
errors in internally-generated queries, such as those submitted by
plpgsql functions.  Per recent discussions with Fabien Coelho.

Translation updates

Translation updates

Rewrite pg_dump's comment-dumping code to pull over all the comments
in one query, rather than making a separate query for each object that
could have a comment.  This costs relatively little space (a few tens of
K typically) and saves substantial time in databases with many objects.
I find it reduces the runtime of 'pg_dump -s regression' by about a
third.

Remove suggestiong for using test for wal_sync_method.

Always test getpwuid_r because HEAD always uses *_r if it finds them.
Not an issue for 7.4.X.

Handle draft version of getpwuid_r() that accepts only four arguments.

Backpatch to 7.4.X.  Required for Solaris 7 & 8.

Add a usage example for has_function_privilege().

Create a validator for plpgsql, so that some minimal syntax checking
is done at creation time for plpgsql functions.  Improve createlang and
droplang to support adding/dropping validators for PLs.  Initial steps
towards producing a syntax error position from plpgsql syntax errors
(this part is a work in progress, and will change depending on outcome
of current discussions).

Code review for log_line_prefix patch.  Cooperate with StringInfo instead
of fighting it, avoid hard-wired (and wrong) assumption about max length
of prefix, cause %l to actually work as documented, don't compute data
we may not need.

Make test_fsync use optimizer flag.

Fix log_executor_stats if() test.

Add Unixware thread support from Larry Rosenman.

Allocate 16mb file for testing.

Have program default to 1000 loops, and add file name and loop option.

Make open/close loop testings the same.  Add descriptions for certain tests.

Add comment about using this testing for wal_sync_method.

Add fsync tests:

Compare fsync before and after write's close:
Compare one o_sync write to two:
Compare file sync methods with one 8k write:
Compare file sync methods with 2 8k writes:

Mention which fsync methods are unavailable.

Add permission mode to opens().

Add missing include

Remove acccidental object/binary files.

Add fsync test program.  It tests fsync to see if times for fsync are the
same when done on the write() ile descriptor and a new descriptor.

it also times various fsync methods.

Done:

> * -Order duplicate index entries on creation by tid for faster heap lookups

During btree index build, sort equal-keyed tuples according to their
TID (heap position).  This doesn't do anything to the validity of the
finished index, but by pretending to qsort() that there are no really
equal keys in the sort, we can avoid performance problems with qsort
implementations that have trouble with large numbers of equal keys.
Patch from Manfred Koizar.

Reimplement CASE val WHEN compval1 THEN ... WHEN compval2 THEN ... END
so that the 'val' is computed only once, per recent discussion.  The
speedup is not much when 'val' is just a simple variable, but could be
significant for larger expressions.  More importantly this avoids issues
with multiple evaluations of a volatile 'val', and it allows the CASE
expression to be reverse-listed in its original form by ruleutils.c.

Modify:

< * Order duplicate index entries by tid for faster heap lookups
> * Order duplicate index entries on creation by tid for faster heap lookups

Document SPI_push() and SPI_pop().

Replace the switching function ExecEvalExpr() with a macro that jumps
directly to the appropriate per-node execution function, using a function
pointer stored by ExecInitExpr.  This speeds things up by eliminating one
level of function call.  The function-pointer technique also enables further
small improvements such as only making one-time tests once (and then
changing the function pointer).  Overall this seems to gain about 10%
on evaluation of simple expressions, which isn't earthshaking but seems
a worthwhile gain for a relatively small hack.  Per recent discussion
on pghackers.