Back out:

>  Attached is a patch that addressed all the discussed issues
>  that did not break backward compatability, including the
>  ability to output ISO-8601 compliant intervals by setting
>  datestyle to iso8601basic.

[ This description should have been on the earlier fork/exec
  commit, but I am adding it now so it is in CVS.]

The patch basically is a slight rearrangement of the code to allow
fork/exec on Unix, with the ultimate goal of doing CreateProcess on
Win32.  The changes are:

        o  Write out postmaster global variables and per-backend
variables to be read by the exec'ed backend

        o  Mark some static variables as global when exec is used so
then can be dumped from postmaster.c, marked NON_EXEC_STATIC

        o  Remove value passing with -p now that we have per-backend
file

        o  Move some pointer storage out of shared memory for easier
dumping.

        o  Modified pgsql_temp directory cleanup to handle per-database
directories and the backend exec directory under datadir.

Claudio Natoli

Previous change exposed some opportunities for further simplification
in _bt_first().

Improve btree's initial-positioning-strategy code so that we never need
to step more than one entry after descending the search tree to arrive at
the correct place to start the scan.  This can improve the behavior
substantially when there are many entries equal to the chosen boundary
value.  Per suggestion from Dmitry Tkach, 14-Jul-03.

The recent DUMMY_PROCS patch broke accounting for the number of semaphores
needed.  This caused us to fail all the time on Darwin, and we'd fail for
some values of maxBackends on SysV-sema platforms, too.

bufmgr.c failed to compile on Darwin, because it didn't include
<sys/time.h> where struct timeval is defined.

Dept. of third thoughts: in fact, libpq should support SCM_CREDS challenge
even when HAVE_GETPEEREID is defined, else it will be unable to connect to
pre-7.4 backends that are using IDENT authentication.

Fix broken IDENT support for FreeBSD (appears to have been broken by
ill-considered conditional logic in getpeereid patch of 3-Dec-2002).
Per bug #1021.

This patch is the next step towards (re)allowing fork/exec.

Claudio Natoli

Another markup fix for recent patch.

Fix broken markup.

In my mind there were two categories of open issues
  a) ones that are 100% backward (such as the comment about
     outputting this format)
and
  b) ones that aren't (such as deprecating the current
     postgresql shorthand of
         '1Y1M'::interval = 1 year 1 minute
     in favor of the ISO-8601
         'P1Y1M'::interval = 1 year 1 month.

Attached is a patch that addressed all the discussed issues that
did not break backward compatability, including the ability to
output ISO-8601 compliant intervals by setting datestyle to
iso8601basic.

Interval values can now be written as  ISO 8601 time intervals, using
the "Format with time-unit designators". This format always starts with
the character 'P', followed  by a string of values followed
by single character time-unit designators. A 'T' separates the date and
time parts of the interval.

Ron Mayer

Update HISTORY file for 7.4.1.

Supress ecpg thread test if configure didn't enable threads.  Fix
tools/thread to run even if configure didn't enable threads because this
test is used before enabling threads for the OS.

Prevent service dbname from defaulting to service name, per bug report
from Michael Fuhr

7.4.1 release note improvements

Forbid REVOKE on untrusted languages, and don't dump privileges of
untrusted languages (in case they sneak in).

Add missing $(X).

Make to_hex() behave portably on negative input values (treat them as
unsigned integers).  Per report from Jim Crate.

Use a shutdown callback to ensure proper clean up when rescanning
partially-evaluated SRFs. Per report found here:
http://archives.postgresql.org/pgsql-general/2003-12/msg00851.php

Fix memory leak with SSL connections due to missing X509_free() calls.
Per Neil Conway.

Use a shutdown callback to clear setArgsValid in a FuncExprState that is
evaluating a set-valued function.  This fixes some additional problems
with rescanning partially-evaluated SRFs.

Forgot to change one compatlib.h.

Ensure set-returning functions in the targetlist of a plan node will be
shut down cleanly if the plan node is ReScanned before the SRFs are run
to completion.  This fixes the problem for SQL-language functions, but
still need work on functions using the SRF_XXX() macros.

Fix signed char in comparison and check memory allocation

Move Informix compatibility include files out of the way.  compatlib.h
was integrated into ecpg_informix.h, the other ones go into their own
subdirectory that is automatically considered by the embedded preprocessor
when in Informix mode.

patch for new OID74Test

fixed up OID74 test to conform with other tests, by Kris Jurka

Please find enclosed a patch exemplifying typical use of the ARE
Class-Shorthand Escapes<C2><AE>.  I believe it will help intrepid regex
users. :)

David Fetter

Here is the definition of relation_byte_size() in optimizer/path/costsize.c:

----------------------------------------------------------------------
/*
 * relation_byte_size
 *        Estimate the storage space in bytes for a given number of tuples
 *        of a given width (size in bytes).
 */
static double
relation_byte_size(double tuples, int width)
{
        return tuples * (MAXALIGN(width) + MAXALIGN(sizeof(HeapTupleData)));
}

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

Shouldn't this be HeapTupleHeaderData and not HeapTupleData ?

(Of course, from a costing perspective these shouldn't be very different but ...)

Sailesh Krishnamurthy

patch by Kris Jurka to use the correct protocol based upon server information

Adjust rules output for unknown-vs-any change (affects expected contents
of pg_stats view definition).

Be a little smarter in group_clauses_by_indexkey_for_join: detect cases
where a joinclause is redundant with a restriction clause.  Original coding
believed this was impossible and didn't need to be checked for, but that
was a thinko ...

information_schema.constraint_column_usage and key_column_usage should
not discriminate against system columns, since we support constraints on
system columns, and in fact constraints on OID are moderately useful.

Fix DecodeInterval to handle '-0.1' sanely, per gripe from Tilo Schwarz.

Repair badly broken estimation of output buffer size in lquery_out().

Reorder tests in parse_coerce so that ANY/ANYELEMENT/ANYARRAY coercion
does not affect UNKNOWN-type literals or Params.  This fixes the recent
complaint about count('x') being broken, and improves consistency in
a few other respects too.

Remove pg_id.

Repair planner failure when there are multiple IN clauses, each with
a join in its subselect.  In this situation we *must* build a bushy
plan because there are no valid left-sided or right-sided join trees.
Accordingly, hoary sanity check needs an update.  Per report from
Alessandro Depase.

revoked patch from Kris Jurka to fix multiarguments, and changed test to create
a temp table

patch from Kris Jurka to fix large object 7.1 compatible protocol issues
modified test case from Alexey Yudichev to be part of the testsuite

- Added just another patch by Dave that fixes a reversed order in
  variable listing for output variables in cursor definitions
- Fixed incorrect if call in long=>numeric conversion.

added polish translation submitted by Piotr Maj

Restore information schema upgrade instructions of Peter.

Add example for converting epoch back to timestamp.

Fix constraint_column_usage for foreign keys.

Repair name.

Add mention of non-standard extension:

<  o Make SET CONNECTION thread-aware
>  o Make SET CONNECTION thread-aware, non-standard?

Add for ecpg:

>  o Make SET CONNECTION thread-aware

Mention ecpg SET CONNECTION is not thread-aware.

Override some of the changes in DocBook DSSSL stylesheets 1.78 to restore
previous behavior of <literal>, <envar>, <acronym>, and others.

Add mention with might need to use cp -R someday for portability.

Update HISTORY and release notes for 7.4.1.

Fix two typos in the documentation for PREPARE.

I posted some bufmgr cleanup a few weeks ago, but it conflicted with
some concurrent changes Jan was making to the bufmgr. Here's an
updated version of the patch -- it should apply cleanly to CVS
HEAD and passes the regression tests.

This patch makes the following changes:

     - remove the UnlockAndReleaseBuffer() and UnlockAndWriteBuffer()
       macros, and replace uses of them with calls to the appropriate
       functions.

     - remove a bunch of #ifdef BMTRACE code: it is ugly & broken
       (i.e. it doesn't compile)

     - make BufferReplace() return a bool, not an int

     - cleanup some logic in bufmgr.c; should be functionality
       equivalent to the previous code, just cleaner now

     - remove the BM_PRIVATE flag as it is unused

     - improve a few comments, etc.

This patch fixes a few more uppercase GUC vars. I also removed an
example from the RESET reference page because it seemed completely
redundant.

This patch makes some improvements and adds some additional detail
to the documentation on routine database maintainence activities.
I also corrected a bunch of SGML markup.

This patch makes some SGML markup more consistent and makes a small
improvement to the SSL auth docs.

This patch makes some SGML markup more consistent and makes a small
improvement to the SSL auth docs.

Add /usr/local/sgml/docbook-dsssl to the default search patch for
docbook style sheets, as discussed with Peter.

SGML doc build instructions --- no more DTDDECL, fix double dash in path name.

Add fadvise TODO.detail.

Update TODO.detail syntax:

< * Add free-behind capability for large sequential scans (fadvise)
> * Add free-behind capability for large sequential scans [fadvise]

Add Czech language FAQ's

Pavel Stehule

This patch properly sets the prototype for the on_shmem_exit and
on_proc_exit functions, and adjust all other related code to use
the proper types too.

by Kurt Roeckx

add missing SQLState by Patrick Higgins

cancel row updates sets values to null by Kris Jurka

patch to indicate why test cases failed from Oliver Jowett

patch to fix returning he information schema as a system object by Kris Jurka

fix casting pooled connections to PGStatement problem patch by JariP

patch to build under jdk1.1 from Richard Scranton

binary stream patch by Kris Jurka fixes empty stream failure

metadata encoding patch from Kris Jurka

Fix for vpath build

New translation

Fix thinko in comment.

Make some wording consistent with DROP OPERATOR description.

Fix instructions how to shut down postmaster.

build number set to 300 to avoid conflict with 7.4 builds
threadsafe patch from Karl von Randow applied

patch to deal with unique foreign keys in 7.4 from Kris Jurka

tch to make sure PSQLState is Serializable and a test case to prove it

patch to make sure PSQLState is Serializable and a test case to prove it

incremented build to 211

Fix integer types to use definition from c.h. Per bug report by Patrick Boulay <patrick.boulay@medrium.com>

query_tree_mutator should copy RangeTblEntry nodes even when it's not
planning to modify them itself.  Otherwise we end up with shared RTE
substructure, which breaks inheritance_planner because the rte->inh
flag needs to be independent in each copied subquery.  Per bug report
from Chris Piker.

Minor autovacuum fixes from Matthew O'Connor.

Add fflush() before sleeping, per Matthew O'Connor.

Whole-row references were broken for subqueries and functions, because
attr_needed/attr_widths optimization failed to allow for Vars with attno
zero in this case.  Per report from Tatsuo Ishii.

Work around naming conflict between zlib and OpenSSL by tweaking inclusion
order.  Remove some unnecessary #includes (that duplicate c.h).

Done:

< * COMMENT ON [ CAST | CONVERSION | OPERATOR CLASS | LARGE OBJECT | LANGUAGE ]
> * -COMMENT ON [ CAST | CONVERSION | OPERATOR CLASS | LARGE OBJECT | LANGUAGE ]
263c263
< * Allow pg_dump to dump CREATE CONVERSION (Christopher)
> * -Allow pg_dump to dump CREATE CONVERSION (Christopher)

Replace unportable and overflow-prone use of 'long long' with safer
'double' arithmetic, per recent discussion.

Remove test on c.relkind from check_constraints view; unnecessary and
prevents view from showing constraints on domains.  This addresses the
other half of Claus Colloseus' bug report.

Fix typmod interpretation for bit types.  (It was erroneously assumed that
for bit(x), the typmod stores x+4, like for the character types.)

One more pass at reducing the cost of pg_dump's new implementation:
reduce the number of times TopoSort() has to be executed by trying to
extract multiple dependency loops from each pass, instead of only one.
This saves about another factor of ten on the regression database.
This could be considered as another exercise in grokking Fred Brooks'
maxim: Representation *is* the essence of programming.

Repair indexed bytea like operations, and related selectivity
functionality. Per bug report by Alvar Freude:
http://archives.postgresql.org/pgsql-bugs/2003-12/msg00022.php

Speed up findObjectByCatalogId() to get rid of the other salient
bottleneck in the new pg_dump code.

Done:
> * -Use dependency information to dump data in proper order

Documentation update for previously committed changes: read-only GUC variables
and pg_settings view.

Replace not-very-bright implementation of topological sort with a better
one (use a priority heap to keep track of items ready to output, instead
of searching the input array each time).  This brings the runtime of
pg_dump back to about what it was in 7.4.

Massive overhaul of pg_dump: make use of dependency information from
pg_depend to determine a safe dump order.  Defaults and check constraints
can be emitted either as part of a table or domain definition, or
separately if that's needed to break a dependency loop.  Lots of old
half-baked code for controlling dump order removed.