Make Solaris non-gcc thread flag "-mt" a compile flag, not a link flag.

Adjust the definition of RestrictInfo's left_relids and right_relids
fields: now they are valid whenever the clause is a binary opclause,
not only when it is a potential join clause (there is a new boolean
field canjoin to signal the latter condition).  This lets us avoid
recomputing the relid sets over and over while examining indexes.
Still more work to do to make this as useful as it could be, because
there are places that could use the info but don't have access to the
RestrictInfo node.

Recommend SHOW, instead of pg_controldata, for checking LC_COLLATE and
LC_CTYPE settings of a database.

Add "-mt" link flag for Solaris non-gcc compiles when using threads ---
required.

Instead of trying to force WHERE clauses into CNF or DNF normal form,
just look for common clauses that can be pulled out of ORs.  Per recent
discussion, extracting common clauses seems to be the only really useful
effect of normalization, and if we do it explicitly then we can avoid
cluttering the qual with partially-redundant duplicated expressions, which
was an unpleasant side-effect of the old approach.

Avoid running out of memory during hash_create, by not passing a
number-of-buckets that exceeds the size we actually plan to allow
the hash table to grow to.  Per trouble report from Sean Shanny.

ShmemInitHash forgot to specify HASH_ALLOC flag bit in its hash_create
call.  You'd think this would cause some problems, but because of the
way hash_create is coded, the only side-effect was creation of a useless
memory context for the hashtable.

Use hash table name, not one-size-fits-all 'DynaHashTable', to identify
memory contexts belonging to hash tables.  Makes the memory stats printout
a little more useful.

Using canonicalize_qual() to get rid of duplicate index predicate
conditions is overkill; set_union() does the job about as well, and
much more efficiently.  Furthermore this avoids assuming that
canonicalize_qual() will check for duplicate clauses at all, which
it may not always do.

Improve comment.

Tweak OpernameGetCandidates() to reduce palloc overhead --- profiling
showed that for common operator names such as '=', the pallocs done by
this routine occupied a surprisingly large fraction of the total time
for the parser to process an operator.

Added missing whitespaces to array argument parsing.

Clean up the usage of canonicalize_qual(): in particular, be consistent
about whether it is applied before or after eval_const_expressions().
I believe there were some corner cases where the system would fail to
recognize that a partial index is applicable because of the previous
inconsistency.  Store normal rather than 'implicit AND' representations
of constraints and index predicates in the catalogs.
initdb forced due to representation change of constraints/predicates.

Fix sanity-check code that mistakenly assumed error and notice messages
could never exceed 30K.  Per report from Andreas Pflug.

Avoid infinite loop if connection is lost during PQexecStart() or
PQexecFinish().  Per report from Andreas Pflug.

Improve spinlock code for recent x86 processors: insert a PAUSE
instruction in the s_lock() wait loop, and use test before test-and-set
in TAS() macro to avoid unnecessary bus traffic.  Patch from Manfred
Spraul, reworked a bit by Tom.

Remove incorrect remark about having to restart backend to load a new
version of a shared library.

Be a little more consistent about which terms are uppercased in
index entries.

Fixed docs to clearly note that ecpg is originally a C precompiler and C++ stuff is not 100%.

Continued rearrangement to permit pgstat + BootstrapMain processes to be
fork/exec'd, in the same mode as the previous patch for backends.

Claudio Natoli

Patch that makes quoting "sameuser", "samegroup", and "all" remove
special meaning of these terms in pg_hba.conf.

Also changes ugly pg_hba.conf IPv6 netmask of
ffff:ffff:ffff:ffff:ffff:ffff:ffff:ffff to ::1/128.

Andrew Dunstan

> > I have no idea if this in Oracle or not.  But it's something I
> > needed, and other people in the past asked about it too.
>
> It is in Oracle, but you aren't exactly on the spot.  It should be
>
> IYYY - 4 digits  ('2003')
> IYY  - 3 digits  ('003')
> IY   - 2 digits  ('03')
> I    - 1 digit   ('3')

Here is an updated patch that does that.

Kurt Roeckx

Fixed segfault in parsing of EXEC SQL SELECT * FROM foo() AS TBL( c int, i int);

Supress non-temp schemas from psql \dn display.

Use inlined TAS() on PA-RISC, if we are compiling with gcc.
Patch inspired by original submission from ViSolve.

Fix a number of places where reconfiguring with a different installation
prefix would fail, because the new path did not get propagated to where
it needed to be.  Note this would fail even with --enable-depend.

Fix compile warning.

More bogosity in alter_table test: sometimes causes prepare test to fail
by means of arbitrarily renaming tables the other test depends on.

alter_table test sometimes failed in parallel mode, because of transient
table name conflict against rangefuncs test.

Push responsibility for selecting out-of-line-assembler TAS code out to
the platform template files, instead of doing it directly in configure.in.
This seems cleaner, and also opens the door to making the choice be
dependent on the compiler being used.

Mop-up for HAS_TEST_AND_SET refactoring.  Un-break two or three platforms
that were broken, try to make layout of s_lock.h entries consistent,
use HAVE_SPINLOCKS in preference to HAS_TEST_AND_SET everywhere outside
s_lock.h itself.

Have configure --without-spinlocks actually not use spinlock code, even
if supported by the cpu.

Move slock_t typdefs into s_lock.h from include/port files for
centralization and easier maintanence.

Remove __alpha__ additions in main.c but document that they are missing.

Remove NEED_I386_TAS_ASM and just test for compiler defines.

Allow plpgsql variables' default value expressions to reference
existing variables (such as function parameters).  Per gripe from
David Fetter.

This applied patch remove NEED_SPARC_TAS_ASM and instead uses __sparc ||
__sparc__.

Test for __alpha and __alpha__.

Clarification:

< * Have psql \dn show only visible schemas using current_schemas()
> * Have psql \dn show only visible temp schemas using current_schemas()

Regenerate HISTORY for release.

Fix broken markup, adjust 7.4.1 release date, re-sync HEAD with branch.

Further cleanup in _bt_first: eliminate duplicate code paths.

Update 7.4.1 release notes in preparation for release.

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.