Add Win32 semaphore implementation, rather than mimicking SysV
semaphores.

Qingqing Zhou

Solaris tas() uses 'int' now.

Theo Schlossnagle

Intel compiler has a bug/misoptimization in checking for division by NAN
(NaN == 0), -mp1 fixes it, so add it to the CFLAGS.  Autoconf run.

Jeremy Drake

Fix s_lock_test to use tas.o file, if needed.

Remove the restriction originally coded into optimize_minmax_aggregates() that
MIN/MAX not be converted to use an index if the query WHERE clause contains
any volatile functions or subplans.

I had originally feared that the conversion might alter the behavior of such a
query with respect to a volatile function.  Well, so it might, but only in the
sense that the function would get evaluated at a subset of the table rows
rather than all of them --- and we have never made any such guarantee anyway.
(For instance, we don't refuse to use an index for an ordinary non-aggregate
query when one of the non-indexable filter conditions contains a volatile
function.)

The prohibition against subplans was because of worry that that case wasn't
adequately tested, which it wasn't, but it turns out to be possible to make
8.1 fail anyway:

regression=# select o.ten, (select max(unique2) from tenk1 i where ten = o.ten
or ten = (select f1 from int4_tbl limit 1)) from tenk1 o;
ERROR:  direct correlated subquery unsupported as initplan

This is due to bogus code in SS_make_initplan_from_plan (it's an initplan,
ergo it can't have any parParams).  Having fixed that, we might as well allow
subplans as well as initplans.

Modify Solaris compiler build rules to use the cpp preprocessor, the the
x86 file.

Darin -> Darwin.

Remove extra 'else' in solaris compiler code.

Remove "volatile" from tas function, per TOm.

Update list of platforms that have a list of exported symbols.

Arrange to strip libpq.so of symbols that aren't officially supposed to
be exported on Linux and Darwin.  We already did this on Windows but
that's not enough, as evidenced by the fact that libecpg had an unexpected
dependency on one such symbol.  We should try to do it on more platforms.
Fix ecpg's oversight, and bump libpq's major .so version number to reflect
the unwanted but nonetheless real ABI break.

Add info on pgport linking requirements.

Rewrite Solaris compiler tas() assembly routines, merge i386 and x86_64
assembler files, renamed as solaris_x86.s.

Theo Schlossnagle

Fix ltreeparentsel so it actually works ...

Generalize mcv_selectivity() to support both VAR OP CONST and CONST OP VAR
cases.  This was not needed in the existing uses within selfuncs.c, but if
we're gonna export it for general use, the extra generality seems helpful.
Motivated by looking at ltree example.

Change log message about vacuuming database name from LOG to DEBUG1.
Prevents duplicate meaningless log messsages.

Add to item log_min_error_messages item:

>   Another idea is to allow separate configuration files for each module,
>   or allow arbitrary SET commands to be passed to them.

On second thought, keep SSL CRL as a log, and wait for feedback from 8.2.

Downgrade SSL CRL file missing message from LOG to DEBUG1.

Turn off strict aliasing when using AIX xlc compiler.

autoconf run.

Revert patch, causing plpython regression failues:

> >> >> > 1) named parameters additionally to args[]
> >> >> > 2) return composite-types from plpython as dictionary
> >> >> > 3) return result-set from plpython as list, iterator or generator

Tab alignment cleanup.

Add underscores to SSL CERT macro names, for clarity and consistency
with be-secure.c.

Add SSL include needed for psql, after libpq adjustments.

Add:

> * Invalidate prepared queries, like INSERT, when the table definition
>   is altered

Add support for SSL Certificate Revocation List (CRL) files, root.crl.

Libor Hoho?

plpython improvements:

1) named parameters additionally to args[]
2) return composite-types from plpython as dictionary
3) return result-set from plpython as list, iterator or generator

Hannu Krosing
Sven Suursoho

Un-break ltree.

Change libpq's PQgetssl() to return a void*, rather than SSL *, so that
applications don't need the SSL headers.

Martijn van Oosterhout

If we're going to expose VariableStatData for contrib modules to use,
then we should export a reasonable set of the supporting routines too.

Remove unused function SSL_CTX_set_tmp_dh_callback() from libpq:

In the SSL code in libpq it does some processing with DH parameters:

SSL_CTX_set_tmp_dh_callback()

This function is marked as server use only[1], the client always uses
the DH parameters in the server, so all the code in the client dealing
with the DH parameters is useless. This patch removes it.

It's not clear why the code was added in the first place, it's been
there almost since the beginning[2]. At the time there was a suggestion
of merging the front-end and backend SSL code, but looking at the
changes since, that seems unlikely.

As a further example, the s_server program allows you to specify DH
params, but s_client doesn't. In the GnuTLS documentation under
gnutls_dh_params_generate2() it says[3]:

  Also note that the DH parameters are only useful to servers. Since
  clients use the parameters sent by the server, it's of no use to call
  this in client side.

Use schema search path to find the first matching contraint name for SET
CONSTRAINT, rather than affecting all constraints in all schemas (which
is what we used to do).  Also allow schema specifications.

Kris Jurka

Delay write of pg_stats file to once every five minutes, during
shutdown, or when requested by a backend:

It changes so the file is only written once every 5 minutes (changeable
of course, I just picked something) instead of once every half second.
It's still written when the stats collector shuts down, just as before.
And it is now also written on backend request. A backend requests a
rewrite by simply sending a special stats message. It operates on the
assumption that the backends aren't actually going to read the
statistics file very often, compared to how frequent it's written today.

Magnus Hagander

Add tablespace display to psql \l+.

Philip Yarra

Improve /contrib/pg_buffercache installation to use BEGIN/COMMIT,
improve examples.

Mark Kirkwood

Split out pg_freespace views to one for relations and another for pages,
pg_freespacemap_relations and pg_freespacemap_pages.

Mark Kirkwood

Adjust /contrib/pg_freespace to show index free space as NULL (FSM only
tracks index pages, not free space on pages):

1/ Index free bytes set to NULL
2/ Comment added to the README briefly mentioning the index business
3/ Columns reordered more logically
4/ 'Blockid' column removed
5/ Free bytes column renamed to just 'bytes' instead of 'blockfreebytes'

Mark Kirkwood

Move ltree parentsel() selectivity function into /contrib/ltree.

In pg_resetxlog.c, uint -> uint32, for Win32 port.

Add missing ControlFile.checkPointCopy.ThisTimeLineID line for 'guess'
pg_resetxlog.

Simon

Update catalog version for ltree changes.

Enhanced containment selectivity function for /contrib/ltree

Matteo Beccati

Allow pg_resetxlog -f to reset pg_control counters using xlog
information, and add a -r option to reset pg_control without affecting
xlog.

yuanjia lee

Done:

* -Allow WAL information to recover corrupted pg_controldata

Revise large-object access routines to avoid running with CurrentMemoryContext
set to the large object context ("fscxt"), as this is inevitably a source of
transaction-duration memory leaks.  Not sure why we'd not noticed it before;
maybe people weren't touching a whole lot of LOs in the same transaction
before the 8.1 pg_dump changes.  Per report from Wayne Conrad.

Backpatched as far as 8.1, but the problem doubtless goes all the way back.
I'm disinclined to spend the time to try to verify that the older branches
would still work if patched, seeing that this code was significantly modified
for 8.0 and again for 8.1, and that we don't have any trouble reports before
8.1.  (Maybe the leaks were smaller before?)

Arrange to cache btree metapage data in the relcache entry for the index,
thereby saving a visit to the metapage in most index searches/updates.
This wouldn't actually save any I/O (since in the old regime the metapage
generally stayed in cache anyway), but it does provide a useful decrease
in bufmgr traffic in high-contention scenarios.  Per my recent proposal.

Adjust SGML spacing.

Back out the rest of the RESET CONNECTION patch.

The 8.1 planner removes WHERE quals from the plan when the quals are
implied by the predicate of a partial index being used to scan a table.
However, this optimization is unsafe in an UPDATE, DELETE, or SELECT FOR
UPDATE query, because the quals need to be rechecked by EvalPlanQual if
there's an update conflict.  Per example from Jean-Samuel Reynaud.

Remove equals sign from environment variable with not defaults for
configure.

Make configure environment variable options without defaults not have an
equals sign.

Add documentation about configure environment variabbles used.

 David Wheeler

Call SET varaiables "configuration_parameter"s in the documentation.

Change "parameter" to "varname" in the alter role and alter user
documentation sections.  Also update SET/RESET to be consistent.

Nicolas Barbier

I now see we support RESET SESSION AUTHORIZATION, so it seems we have to
use RESET CONNECTION:

< * Add RESET SESSION command to reset all session state
> * Add RESET CONNECTION command to reset all session state
447c447
<   notify the protocol when a RESET SESSION command is used.
>   notify the protocol when a RESET CONNECTION command is used.

RESET SESSION is more precise:

< * Add RESET CONNECTION command to reset all session state
> * Add RESET SESSION command to reset all session state
447c447
<   notify the protocol when a RESET CONNECTION command is used.
>   notify the protocol when a RESET SESSION command is used.

Back out RESET CONNECTION until there is more discussion.

Add RESET CONNECTION, to reset all aspects of a session.

Hans-J?rgen Sch?nig

Update inheritance constraint items:

<  o %Prevent child tables from altering or dropping constraints
<           like CHECK that were inherited from the parent table
<    like CHECK that are inherited by child tables
<
<    Dropping constraints should only be possible with CASCADE.
<
>    like CHECK that are inherited by child tables unless CASCADE
>    is used
>  o %Prevent child tables from altering or dropping constraints
>           like CHECK that were inherited from the parent table

Fix bug reported by Michael Fuhr on Sun, 23 Apr 2006 09:52:31 -0600
(pgsql-committers@postgresql.org).

Done:

> * -Add transaction_timestamp(), statement_timestamp(), clock_timestamp()

Add statement_timestamp(), clock_timestamp(), and
transaction_timestamp() (just like now()).

Also update statement_timeout() to mention it is statement arrival time
that is measured.

Catalog version updated.

Word wrap entry

Update SQL-standard INTERVAL item:

        o Support ISO INTERVAL syntax if units cannot be determined from
          the string, and are supplied after the string

          The SQL standard states that the units after the string specify
          the units of the string, e.g. INTERVAL '2' MINUTE should
          return '00:02:00'.  The current behavior has the units
          restrict the interval value to the specified unit or unit range,
          INTERVAL '70' SECOND returns '00:00:10'.

          For syntax that isn't uniquely ISO or PG syntax, like '1' or
          '1:30', treat as ISO if there is a range specification clause,
          and as PG if there no clause is present, e.g. interpret
          '1:30' MINUTE TO SECOND as '1 minute 30 seconds', and
          interpret '1:30' as '1 hour, 30 minutes'.

          This makes common cases like SELECT INTERVAL '1' MONTH
          SQL-standard results.  The SQL standard supports a limited
          number of unit combinations and doesn't support unit names
          in the string.  The PostgreSQL syntax is more flexible in
          the range of units supported, e.g. PostgreSQL supports
          '1 year 1 hour', while the SQL standard does not.

Done;

   o -Add support for day-time syntax, INTERVAL '1 2:03:04'DAY TO
      SECOND

Back out patch, unintended.

Done:

     o -Add support for day-time syntax, INTERVAL '1 2:03:04' DAY TO
        SECOND

Suppress more compiler warnings caused by macro tests.

Add one more paren to macro.

Suprress compiler warning in gcc 4.2.

Report by Kris Jurka

Improve our private implementation of cbrt() to give results of the
accuracy expected by the regression tests.  Per suggestion from
Martijn van Oosterhout.

Remove compiler warning by casting SNPRINTF() call to void.

Report from Gevik Babakhani.

Fixed memory leak bugs found by Martijn Oosterhout.

Fixes for BCC 5.5 compile of libpq.  Backpatch to 8.1.X.

Mark Morgan Lloyd

Don't add a shared dependency on the owner of a composite type in pg_class.
We track the owner in pg_type instead, as that is the place where the owner is
changed on ALTER TYPE ... OWNER TO.

Add:

< * -Eventually enable escape_string_warning and standard_conforming_strings
> * -Enable escape_string_warning and standard_conforming_strings
> * Make standard_conforming_strings the default in 8.3?
>
>   When this is done, backslash-quote should be prohibited in non-E''
>   strings because of possible confusion over how such strings treat
>   backslashes.  Basically, '' is always safe for a literal single
>   quote, while \' might or might not be based on the backslash
>   handling rules.
>

Removes or minimizes some documentation mentions of backward
compatibility for release 7.2 and earlier.  I have not altered any
mentions of release 7.3 or later.  The release notes were not modified,
so the changes are still documented, just not in the main docs.

Add pgrowlocks

Add new contrib function pgrowlocks. See README.pgrowlocks for more details.

Update:

<  o Fix psql's backslash commands more consistent
>  o Fix psql's \d commands more consistent

Update:

<  o Fix psql's \dn for various schema combinations (Neil)
>  o Fix psql's backslash commands more consistent
625a626
>    http://archives.postgresql.org/pgsql-hackers/2004-11/msg00014.php

Update:

        o Fix psql's \dn for various schema combinations (Neil)

          http://archives.postgresql.org/pgsql-hackers/2004-11/msg00014.php

Suggest Win32 users user E'' strings and double backslashes used for
patch separators in COPY.

Backpatch doubleing backslashes suggestion to 8.1.

Simplify ParamListInfo data structure to support only numbered parameters,
not named ones, and replace linear searches of the list with array indexing.
The named-parameter support has been dead code for many years anyway,
and recent profiling suggests that the searching was costing a noticeable
amount of performance for complex queries.

Revert Intel compiler bug comment.

Add mentioun of Intel compiler bug:

Intel compiler has a bug in checking for division by NAN
# (NaN == 0), -mp1 fixes it, so someday we might way to add it.

Use special gcc -W flags only if we are using gcc, not if we are using
the Intel compiler.

Jeremy Drake

Remove from TODO ability to edit pg_hba.conf, but add GRANT connection
permission item:

<  o %Allow pg_hba.conf settings to be controlled via SQL
>  o %Allow per-database permissions to be set via GRANT
<    This would add a function to load the SQL table from
<           pg_hba.conf, and one to writes its contents to the flat file.
<    The table should have a line number that is a float so rows
<    can be inserted between existing rows, e.g. row 2.5 goes
<    between row 2 and row 3.
>    Allow database connection checks based on GRANT rules in
>    addition to the existing access checks in pg_hba.conf.

Add some optional code (conditionally compiled under #ifdef LWLOCK_STATS)
to track the number of LWLock acquisitions and the number of times we
block waiting for an LWLock, on a per-process basis.  After having needed
this twice in the past few months, seems like it should go into CVS.

Eliminate some no-longer-needed workarounds for palloc's old behavior
of rejecting palloc(0).  Also, tweak like_selectivity() to avoid assuming
the presented pattern is nonempty; although that assumption is valid,
it doesn't really help much, and the new coding is more correct anyway
since it properly handles redundant wildcards.  In combination these
changes should eliminate a Coverity warning noted by Martijn.

Add "retry another address" log message on statistics collector socket
failure, to reduce confusion in the log file.

Ensure that we validate the page header of the first page of a WAL file
whenever we start to read within that file.  The first page carries
extra identification information that really ought to be checked, but
as the code stood, this was only checked when we switched sequentially
into a new WAL file, or if by chance the starting checkpoint record was
within the first page.  This patch ensures that we will detect bogus
'long header' information before we start replaying the WAL sequence.

Recognize __ppc64__, which seems to be Apple's spelling of the predefined
symbol for PPC64 hardware.  I hadn't known that Apple supported PPC64 at
all, but darn if there aren't 64-bit variant libraries in OS X as well
as support in their gcc.

Add detail:

<    encoding.
>    encoding.  This requires using mblen() to determine if the
>    backslash is inside or outside a multi-byte sequence.

Add:

>
>  o Add new version of PQescapeString() that doesn't double backslashes
>    that are part of a client-only multibyte sequence
>
>    Single-quote is not a valid byte in any supported client-only
>    encoding.
>
>  o Add new version of PQescapeString() that doesn't double
>    backslashes when standard_conforming_strings is true and
>    non-E strings are used

Fix problem that sscanf(buf, "%d", &val) eats leading white space, but
our to_* functions were not handling that.

Remove use of lorder and tsort while building static libraries.  There's
no evidence that any currently-supported platform needs this, and good
reason to think that any platform that did need it couldn't use the static
libraries anyway --- libpq, at least, has circular references.  Removing
the code shuts up tsort warnings about the circular references on some
platforms.

Fix ancient memory leak in PQprintTuples(); our code no longer uses this
routine, but perhaps some applications do.  Found by Martijn van Oosterhout
using Coverity.

Fix a couple of rather-pointless-but-easily-fixed Coverity warnings.
Per Martijn van Oosterhout.

C code whitespace inprovement for formatting.c.