Reconsider mechanism for marking sub-selects that are at top level of
a qualification clause (and hence can get away with being sloppy about
distinguishing FALSE from UNKNOWN).  We need to know this in subselect.c;
marking the subplans in setrefs.c is too late.

Change Adjust_lo_type() so that it doesn't cause an error
even when cast functions are allowed to be volatile.

Fixes to pgcvslog for last narrive entry.

Cause planner to account for evaluation costs in targetlists and
HAVING quals.  Normally this is an insignificant effect --- but it
will not be insignificant when these clauses contain sub-selects.
The added costs cannot affect the planning of the query containing
them, but they might have an impact when the query is a sub-query
of a larger one.

Recent changes in sublink representation require exprType() to accept
SubPlan nodes, else explaining queries containing sublinks may fail.

Revise cost_qual_eval() to compute both startup (one-time) and per-tuple
costs for expression evaluation, not only per-tuple cost as before.
This extension is needed in order to deal realistically with hashed or
materialized sub-selects.

Fix wrong/misleading comments, be more consistent about where to call
ExecAssignResultTypeFromTL().

Fix some minor grammatical errors.

Update CHAR().

Replace RelidGetNamespaceId() by get_rel_namespace().

Update CHAR() description.

Update date only.

Improve CHAR() description.

First cut at implementing IN (and NOT IN) via hashtables.  There is
more to be done yet, but this is a good start.

SGML build cleanups from Neil Conway.

Fix capitalization.

Back out flockfile change for NetBSD.  Giles Lean reports they are not
supported.

Cause symlinks for shared-library versioning to run in the standard
direction on HP-UX; our former approach has been obsolete since HPUX 9.

Giles Lean

Fix markup problem in link to other SGML file.

Fix typo in an error message

> > This patch improves the documentation for the shared_buffers GUC param.
>
> I'd suggest that the runtime.sgml description explicitly say "values of
> at least a few thousand are recommended for production installations".

Neil Conway

Rewrite for-loop, because this is not the Obfuscated C Code Contest.

Manfred Koizar

A tiny patch to fix a typo in configure.in and another one in
RELEASE_CHANGES.

Manfred Koizar

Add doc links from SET to SET_CONSTRAINTS, SET_SESSION_AUTH,
SET_TRANSACTION.

Fix markup.

Create a new file executor/execGrouping.c to centralize utility routines
shared by nodeGroup, nodeAgg, and soon nodeSubplan.

Add missing <limits.h> for INT64_IS_BUSTED case.

Read-only transactions, as defined in SQL.

Do not shell-quote the name of the editor, so that the editor can be a
command with arguments.

Further tweaking of parsetree & plantree representation of SubLinks.
Simplify SubLink by storing just a List of operator OIDs, instead of
a list of incomplete OpExprs --- that was a bizarre and bulky choice,
with no redeeming social value since we have to build new OpExprs
anyway when forming the plan tree.

Add optional drop behavior clause to REVOKE command, for SQL conformance.
Currently, only RESTRICT is allowed.

Clean ip.c.

Adjust parser so that 'x NOT IN (subselect)' is converted to
'NOT (x IN (subselect))', that is 'NOT (x = ANY (subselect))',
rather than 'x <> ALL (subselect)' as we formerly did.  This
opens the door to optimizing NOT IN the same way as IN, whereas
there's no hope of optimizing the expression using <>.  Also,
convert 'x <> ALL (subselect)' to the NOT(IN) style, so that
the optimization will be available when processing rules dumped
by older Postgres versions.
initdb forced due to small change in SubLink node representation.

Add missing #include <getopt.h>.

Move new typedef AclId into c.h, so as to avoid cluttering namespace
by having to include miscadmin.h into other header files.

Fix IPv6 detection and IPv6 to 4 conversion to use only standard
interfaces.

Call timestamp_in with proper arguments, per gripe from Mark Halliwell.

Fix indenting of log_min_error_statement.

Remove bit.c/h routines.  Not used anymore.

Repair inconsistent rounding behavior for timestamp, time, interval,
per gripe from Csaba Nagy.  There is still potential for platform-specific
behavior for values that are exactly halfway between integers, but at
least we now get the expected answer for all other cases.

add to threads discussion.

Add missing pg_proc entry for interval_scale().  The lack of this entry
causes interval rounding not to work as expected in 7.3, for example
SELECT '18:17:15.6'::interval(0) does not round the value.
I did not force initdb, but one is needed to install the added row.

Remove 'const' from secure_write, to prevent compiler warning.

Guard against stopping when numberTuples=0 and counter wraps around.

The second was that renegotiation was just plain broken.  I can't
believe I didn't notice this before -- once 64k was sent to/from the
server the client would crash.  Basicly, in 7.3 the server SSL code set
the initial state to "about to renegotiate" without actually starting
the renegotiation.  In addition, the server and client didn't properly
handle the SSL_ERROR_WANT_(READ|WRITE) error.  This is fixed in the
second patch.

Nathan Mueller

I was playing around with 7.3.1 and found some more SSL problems.  The
first, that I missed when checking over 7.3.1, was that the client
method was switched to SSLv23 along with the server.  The SSLv23 client
method does SSLv2 by default, but can also understand SSLv3.  In our
situation the SSLv2 backwords compatibility is really only needed on the
server.  This is the first patch.

The last thing is that I found a way for the server to understand SSLv2
HELLO messages (sent by pre-7.3 clients) but then get them to talk
SSLv3.  This is the last one.

Nathan Mueller

Fix dumping of DEFERRABLE/INITIALLY DEFERRED:

> The big problem is that while pg_dump's dump_trigger() looks at
> tginitdeferred and dumps accordingly, pg_get_constraintdef doesn't look
> at tginitdeferred, and therefore doesn't record the requirement as part
> of ALTER TABLE ADD CONSTRAINT.

pg_get_constraintdef should probably be looking at condeferrable and
condeferred in the pg_constraint row it's looking at.  Maybe something
like the attached.

(Added, output only non-default values.)

Stephan Szabo

Updated deferred trigger patch.

Neil Conway

This trivial patch implements disabled, deferred triggers, per my
proposal on -hackers a couple days ago. The 'tgenabled' status of the
trigger is consulted when the trigger is added to the deferred queue
(i.e. when the event occurs that fires the trigger), not when the
deferred event is executed.

No regression tests, as another bug prevents them (the pg_trigger row
for a trigger is only loaded once per session, so any changes to it are
not noticed unless the client disconnects and reconnects).

Neil Conway

Enhanced error/help message for pg_controldata.

Serguei Mokhov

Reconsider old decision to try to constant-fold default and constraint
expressions before they are stored.  This seems like not such a hot idea,
particularly now that the constant-folder will try to inline SQL functions.

Repair bug noticed by Deepak Bhole: a shell type should have a dependency
on its namespace, so that it will go away if the schema is dropped.

Fix pgpass to work with PQconnectDB().

Backpatch to 7.3.X.

Report from ljb.

Fix for bug #866.  7.3 contains new logic for avoiding redundant calls to
the index AM when we know we are fetching a unique row.  However, this
logic did not consider the possibility that it would be asked to fetch
backwards.  Also fix mark/restore to work correctly in this scenario.

Only set the database name if we find a match in the services file.

Correction to last patch.  As per the DB-API spec, we need to return None
here, not -1.

Add:

>  o Allow fastpast to pass values in portable format

Code review for FETCH/MOVE 0 changes.  Improve documentation, do the
right thing with the destination when FETCH 0 can't return a row,
don't try to stuff LONG_MAX into an int value.

SendFunctionResult should take care to detoast varlena results.

Fix various places where global s/NOTICE/WARNING/ was applied with too
much enthusiasm.

Upgrade to my2pg 1.9

Upgrade to even newer ora2pg 2.0.

Upgrade to ora2pg 1.10.  Backpatch to 7.3.X.

Update for new we-always-have-getopt_long approach.

Apply the proper version of Christopher Kings-Lynne's describe patch
(ie, the one with describe-schema support).  Minor code review.
Adjust display of casts to use standard type names.

Add connection service file.

Silence compiler warning caused by removal of netinet/in.h

Document that psql \encoding doesn't see SET CLIENT_ENCODING changes.

Fix missing tab in file.

Small syntax cleanup.

Check for sockaddr_in6 for IPv6 code, rather than ip6.h.

Document libpq service capability, and add sample file.

Tweak mdnblocks() to avoid doing lseek() on segments that it has
previously determined not to be the last segment of a relation.
This reduces the expected cost to one seek, rather than one seek per
segment.  We can get away with this because truncation of a relation
will cause a relcache flush and so the md.c file descriptor will be
closed; when it is re-opened we will re-determine the last segment.

Enable IPv6 libpq 'hostaddr' addresses.  Update docs.

Use our own version of getopt_long() if the OS doesn't have one.

Fix for systems that don't have INET_ADDRSTRLEN.

Update comments on IPv6  #define.

Add test for netinet/ip6.h and getaddrinfo, to enable IPv6.

Update ipv6 comment, move configure test to the right place

Add prototype for getaddrinfo().

Enable IPv6 connections to the server, and add pg_hba.conf IPv6 entries
if the OS supports it.  Code will still compile on non-IPv6-aware
machines (feature added by Bruce).

Nigel Kukard

Add note explaining that a mergejoinable equality operator is now
required if a datatype is to be accepted by GROUP BY, DISTINCT, or
ORDER BY.  This is documentation for code changes made pursuant to
pgsql-hackers discussion around 29-Nov-02.

ALTER DOMAIN OWNER, from Rod Taylor.

Revert rpath-mangling patch.  See discussion on -patches around Nov 29 for
the rationale.

Fix inherited UPDATE for cases where child column numbering doesn't
match parent table.  This used to work, but was broken in 7.3 by
rearrangement of code that handles targetlist sorting.  Add a regression
test to catch future breakage.

Partial code review for ALTER DOMAIN patch.  Incorporates Rod Taylor's
patches of 9-Dec (permissions fix) and 13-Dec (performance) as well as
a partial fix for locking issues: concurrent DROP COLUMN should not
create trouble anymore.  But concurrent DROP TABLE is still a risk, and
there is no protection at all against creating a column of a domain while
we are altering the domain.

Adjust lo type in contrib during pg_restore so that pg_restore could
reload the type.

flockfile/funlockfile is POSIX, so NetBSD has them too --- use them.

Enforces NOT NULL constraints to be applied against new PRIMARY KEY
columns in DefineIndex.  So, ALTER TABLE ... PRIMARY KEY will now
automatically add the NOT NULL constraint.  It appeared the alter_table
regression test wanted this to occur, as after the change the regression
test better matched in inline 'fails'/'succeeds' comments.

Rod Taylor

Fix BSD/OS thread file locking bug in new fseeko code.

Backpatch to 7.3.X.

fastpath code neglected to check whether user has privileges to call the
target function.  Also, move SetQuerySnapshot() call to avoid assert
failure when a fastpath call is attempted in an aborted transaction.

Awhile back I wrote that freebsd.h was probably broken in the places where
it diverged from netbsd.h and openbsd.h.  This has now been confirmed.
Accordingly, make all three exactly alike.

Update:

< * Have DEFAULT dependency track use of sequence, for DROP DEFAULT check
> * Disallow changing default expression of a SERIAL column

Add:

> * Have DEFAULT dependency track use of sequence, for DROP DEFAULT check

Add casts between lo and oid.

Adjust Tcl-related code to compile cleanly with Tcl 8.4 (add const modifiers as
needed).  Some desultory const-ification of SPI interface to support this.

Attached is a patch to provide makefiles, etc. to allow the compilation
of the libpq interface static and dynamic libraries with the freely
downloadable Borland C++ compiler version 5.5 and/or C++ Builder.

Lester Godwin

Code review for transaction-safe-TRUNCATE patch: minor cleanups.

Code review for CLUSTER ALL patch.  Fix bogus locking, incorrect transaction
stop/start nesting, other infelicities.

Make use of TCL_INCLUDE_SPEC if available (it's new in Tcl 8.4, too bad
it took 'em this long to realize it's needed...)