Avoid pulling up sublinks from a subselect's targetlist.  Works around
problems that occur if sublink is referenced via a join alias variable.
Perhaps this can be improved later, but a simple and safe fix is needed
for 7.3.1.

It looks like the problem was introduced when the "SET autocommit" and
"SET search_path"  commands were added to the beginning of the script.

The attatched patch should fix the problem. It probably should be
applied  against the 7.3 and 7.4 branches.

Steven Singer

Add:

> * Add SQL99 WITH clause to SELECT (Tom, Fernando)
> * Add SQL99 WITH RECURSIVE to SELECT (Tom, Fernando)
443a444
> * Fernando Nasser <fnasser@redhat.com>

Clearify variables names so it is clear which variable is the
client-supplied password and which is from pg_shadow.

Allow 'password' encryption even when pg_shadow has MD5 passwords, per
report from Terry Yapt and Hiroshi.

Backpatch to 7.3.

Phase 1 of read-only-plans project: cause executor state nodes to point
to plan nodes, not vice-versa.  All executor state nodes now inherit from
struct PlanState.  Copying of plan trees has been simplified by not
storing a list of SubPlans in Plan nodes (eliminating duplicate links).
The executor still needs such a list, but it can build it during
ExecutorStart since it has to scan the plan tree anyway.
No initdb forced since no stored-on-disk structures changed, but you
will need a full recompile because of node-numbering changes.

Improvements from Neil Conway.

Document get/set bit/byte functions.

More cleanup of userid to be AclId rather than Oid.

changed sort of column names to sort by attnum, not attname

Update docs mention of query string for debug_ commands, from Joseph
Shraibman.

Repair oversight in recent change of dependency extraction code: when
recursing to handle a join alias var, the context had better be set to
be appropriate to the join var's query level.  Per report from Hristo Neshev.

Add:

>  * Add ALTER SEQUENCE to modify min/max/increment/cache/cycle values

Stamp minor version numbers for 7.4 release.

Remove typprtlen from getdescr() as it is not available in 7.3.  Return -1 for
that field so that existing programs don't break.

Make usesysid consistently int4, not oid.

Catalog patch from Alvaro Herrera for same.

catversion updated. initdb required.

Done:

> * -Add OpenBSD's getpeereid() call for local socket authentication

Deal with cases where getpeereid _and_ another creditial method is
supported.

Add OpenBSD local indent credentials, from William Ahern.

Updated:

< * Add sql3 recursive unions
> * Add SQL99 WITH clause to SELECT
> * Add SQL99 WITH RECURSIVE to SELECT

Fix off-by-one bug in usage check.

Add for D'Arcy:

> * Python
>  o Allow users to register their own types with _pg
>  o Allow SELECT to return a dictionary of dictionaries
>  o Allow COPY BINARY FROM

Repair core dump when trying to delete an entry from an already-NULL
datconfig or useconfig field.  Per report from Dustin Sallings.

Fix script name in README.

Fix dependency generation for multicolumn foreign keys.  From Adam Buraczewski.

Done in 7.3, so removed:

<  o Allow psql \copy to specify column names
199d197
<  o Add SET REAL_FORMAT and SET DOUBLE_PRECISION_FORMAT using printf args

Done:

> * -Inline simple SQL functions to avoid overhead (Tom)

Teach planner to expand sufficiently simple SQL-language functions
('SELECT expression') inline, like macros, during the constant-folding
phase of planning.  The actual expansion is not difficult, but checking
that we're not changing the semantics of the call turns out to be more
subtle than one might think; in particular must pay attention to
permissions issues, strictness, and volatility.

Fix ExecMakeTableFunctionResult() to work with generic expressions as
well as function calls.  This is needed for cases where the planner has
constant-folded or inlined the original function call.  Possibly we should
back-patch this change into 7.3 branch as well.

Fix pg_dump failure against 7.1 or older servers, per
report from Philip Warner.

Run COPY OUT in a temporary memory context that's reset once per row,
and eliminate its manual pfree() calls.  This solves the encoding-conversion
bug recently reported, and should be faster and more robust than the
original coding anyway.  For example, we are no longer at risk if
datatype output routines leak memory or choose to return a constant string.

Code review for IS DISTINCT FROM patch.  Fix incorrect constant-folding
logic, dissuade planner from thinking that 'x IS DISTINCT FROM 42' may
be optimized into 'x = 42' (!!), cause dependency on = operator to be
recorded correctly, minor other improvements.

Missed one place that can be simplified after recent Param/Const cleanup.

Be more realistic about plans involving Materialize nodes: take their
cost into account while planning.

Make install and clean targets behave more normally.

Upgrade planner and executor to allow multiple hash keys for a hash join,
instead of only one.  This should speed up planning (only one hash path
to consider for a given pair of relations) as well as allow more effective
hashing, when there are multiple hashable joinclauses.

Tighten selection of equality and ordering operators for grouping
operations: make sure we use operators that are compatible, as determined
by a mergejoin link in pg_operator.  Also, add code to planner to ensure
we don't try to use hashed grouping when the grouping operators aren't
marked hashable.

Repair prehistoric logic error in lseg_eq and lseg_ne.

Suppress compiler warning from newer gcc.

This patch fixes two tiny memory leaks in pg_dump and two in pg_dumpall.

Neil Conway

Add trigger emails.

rm should be rm -f, per comment from Peter Harris.
(Checked with Peter Eisentraut.)

Improve some comments.

Add note that scaling factor should be >= # of clients.

Synced parser with backend

Move to URGENT, for Justin:

< * Create native Win32 port [win32]

Recreate HISTORY file to match release.sgml changes.  Stamp 7.3 final in
configure/configure.in.

Add -cmdTuples to tcl interface.

Update trigger discussion.

Update build rules for /HISTORY.

Add to trigger thread.

Fix typo in release notes.

Allocate proper length for sprintf string, from Tatsuo.

Use Params, rather than run-time-modified Const nodes, to handle
sublink results and COPY's domain constraint checking.  A Const that
isn't really constant is just a Bad Idea(tm).  Remove hacks in
parse_coerce and other places that were needed because of the former
klugery.

Guard against 0 length string encoding conversion case.

Remove check for pg_group file as part of initdb -W password assignment
--- not needed.

Remove unused constisset and constiscast fields of Const nodes.  Clean
up code and documentation associated with Param nodes.

Restructure outfuncs and readfuncs to use macros in the same style as
just done for copyfuncs/equalfuncs.  Read functions in particular get
a lot shorter than before, and it's much easier to compare an out function
with the corresponding read function to make sure they agree.
initdb forced due to small changes in nodestring format (regularizing
a few cases that were formerly idiosyncratic).

Un-break triggers declared for INSERT OR DELETE OR UPDATE.  This worked
okay in 7.3, so I think it must have been busted in the recent triggers
patch.

Restructure the code in copyfuncs and equalfuncs to put much heavier
reliance on macros, in hopes of eliminating silly typos (like copying
to the wrong field) and just generally making it easier to see the forest
instead of the trees.  As an example, here is the new code for A_Indices:

static A_Indices *
_copyAIndices(A_Indices *from)
{
A_Indices  *newnode = makeNode(A_Indices);

COPY_NODE_FIELD(lidx);
COPY_NODE_FIELD(uidx);

return newnode;
}

static bool
_equalAIndices(A_Indices *a, A_Indices *b)
{
COMPARE_NODE_FIELD(lidx);
COMPARE_NODE_FIELD(uidx);

return true;
}

I plan to redo outfuncs/readfuncs in a similar style, but am committing
what I've got.

I'm sending you a small patch to pgdb.py module.  This
raises pgdb.DatabaseError when any of the fetch*
methods was invoked but previous call to execute* did
not produce any result set or no call was issued yet.
Also, raises pgdb.NotSupportedError when .nextset() is
invoked, instead of NameError.

This behaviour complies with DB-API 2.0.

Thanks for your work!

Timur Irmatov.

Change the pkey method so that the caller can optionally set the dictionary
used for the primary key lookup.  This will prevent a database lookup
for each connection object that gets created.  This could be a significant
optimization on a busy system.

Similarly, the get_attnames method allows for the attributes dictionary
to be installed directly.

Change the debug variable to allow better control by the caller over how
debug output is managed.  The user can continue to use the current method
of passing a formatting string to have a replacement done and output will
be sent to the standard output exactly as it did before.  In addition they
can set it to a file object, sys.stderr for example, and the query string
will be printed to it.  Thay can also set it to a method (function) and the
query string will be passed to that method giving them the maximum flexibility
to do whatever they want with the query string.

I will be working with the PyGreSQL documentation shortly and at that time
will properly document this feature.

Restructure planning of nestloop inner indexscans so that the set of usable
joinclauses is determined accurately for each join.  Formerly, the code only
considered joinclauses that used all of the rels from the outer side of the
join; thus for example
FROM (a CROSS JOIN b) JOIN c ON (c.f1 = a.x AND c.f2 = b.y)
could not exploit a two-column index on c(f1,f2), since neither of the
qual clauses would be in the joininfo list it looked in.  The new code does
this correctly, and also is able to eliminate redundant clauses, thus fixing
the problem noted 24-Oct-02 by Hans-Jürgen Schönig.

7.4devel now supports SQL:1999's "enhanced trigger capability" (which
is just FOR EACH STATEMENT triggers, AFAICS); this patch updates the
SQL conformance docs to note that.

Neil Conway

Split:

> * -Support statement-level triggers (Neil)
> * Support triggers on columns (Neil)

Fix inappropriate quoting in dblink.  From Joe Conway.

Fix regression test breakage from triggers patch.

Fix compile warnings from truncate patch.

Update regression for new patches. Bump up catversion.

Heres a patch which will create the sql_help.h file on Win32 if it
doesn't already exist using an installed copy of perl. I've tested it
using perl v5.6.1 from ActiveState and all appears to work.

Al Sutton

Transaction safe Truncate

Rod Taylor

This minor patch corrects an error in the function docs: it's
"version()", not "version".

Neil Conway

This patch implements FOR EACH STATEMENT triggers, per my email to
-hackers a couple days ago.

Notes/caveats:

        - added regression tests for the new functionality, all
          regression tests pass on my machine

        - added pg_dump support

        - updated PL/PgSQL to support per-statement triggers; didn't
          look at the other procedural languages.

        - there's (even) more code duplication in trigger.c than there
          was previously. Any suggestions on how to refactor the
          ExecXXXTriggers() functions to reuse more code would be
          welcome -- I took a brief look at it, but couldn't see an
          easy way to do it (there are several subtly-different
          versions of the code in question)

        - updated the documentation. I also took the liberty of
          removing a big chunk of duplicated syntax documentation in
          the Programmer's Guide on triggers, and moving that
          information to the CREATE TRIGGER reference page.

        - I also included some spelling fixes and similar small
          cleanups I noticed while making the changes. If you'd like
          me to split those into a separate patch, let me know.

Neil Conway

Done:

> * -Support statement-level triggers and triggers on columns (Neil)

When I made the cube(text) function for 7.3, I neglected to add a
matching create cast command. The attached diff adds a create cast as
assignment to cube.sql.in .

Bruno Wolff III

Update ports list from recent regression-test-database entries.

This is a patch to make cube output work like double precision output
with regard to the extra_float_digits setting.

Since builtins.h was already included, I just deleted the extern
statement (and accompaning comments).

 Bruno Wolff III

Release note improvements (Neil, Tom)

Remove inappropriate double-quoting in connectby() code; adjust
regression test to avoid using VALUE as a name.  From Joe Conway.

Redesign internal logic of nodeLimit so that it does not need to fetch
one more row from the subplan than the COUNT would appear to require.
This costs a little more logic but a number of people have complained
about the old implementation.

Fix a dozen or so places that were passing unpredictable data strings
as elog format strings.  Hai-Chen Tu pointed out the problem in
contrib/dbmirror, but it wasn't the only such error.

Fix elog in dbmirror to use %s.

Hai-Chen Tu

Quote database name properly when invoking pg_dump.  Per report from
Christopher Kings-Lynne.

TAS code originally written for s390 (32-bit) does not work for s390x
(64-bit).  Fix it.  Per report from Permaine Cheung.

Fixups for man pages

Regenerate

Fix breakage in new-in-7.3 timetz_zone() function: was giving random
results due to doing arithmetic on uninitialized values.  Add some
documentation about the AT TIME ZONE construct.  Update some other
date/time documentation that seemed out of date for 7.3.

Wups, stuck ORDER BY on wrong query while trying to eliminate random
ordering of GROUP BY output due to hash-based aggregation.

Add:

> * Have sequence dependency track use of DEFAULT sequences, seqname.nextval

Add mention of kern.ipc.shm_use_phys for locking shared memory.

Code review for superuser_reserved_connections patch.  Don't try to do
database access outside a transaction; revert bogus performance improvement
in SIBackendInit(); improve comments; add documentation (this part courtesy
Neil Conway).

NetBSD versions 1.6 and (we hope) later can use the standard geometry
comparison file, instead of geometry-positive-zeros.  Per report from
Patrick Welche.

Done:

> * -Add hash for evaluating GROUP BY aggregates (Tom)

Finish implementation of hashed aggregation.  Add enable_hashagg GUC
parameter to allow it to be forced off for comparison purposes.
Add ORDER BY clauses to a bunch of regression test queries that will
otherwise produce randomly-ordered output in the new regime.

Update ports list.

Fix a dumb cut and paste error from my last commit and update some of the
jdbc3 metadata responses

 Modified Files:
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/jdbc3/AbstractJdbc3DatabaseMetaData.java

Fixed bug reported by Marko Strukelj and Keith Wannamaker.  Using executeBatch
on a preparedStatement would reset the prepared statment causing subsequent
uses of the preparedStatement to fail (i.e. the following series of calls
would fail: addBatch() executeBatch() addBatch() executBatch()).  This is
a regression from 7.2 where this worked correctly.  The regression test has
also been modified to explicitly test for this case.

 Modified Files:
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/test/jdbc2/BatchExecuteTest.java

Add an at-least-marginally-plausible method of estimating the number
of groups produced by GROUP BY.  This improves the accuracy of planning
estimates for grouped subselects, and is needed to check whether a
hashed aggregation plan risks memory overflow.

Add my name:

> * Allow savepoints / nested transactions [transactions] (Bruce)