If a shutdown request comes in while we're still starting up, don't
service it until after we execute SetThisStartUpID().  Else shutdown
process will write the wrong SUI into the shutdown checkpoint, which
seems likely to be trouble --- although I've not quite figured out
how significant it really is.

Repair nasty oversight in bt_split: if the index is temporary, we weren't
updating the left-link from the split page's right sibling.  This could
result in backwards index scans missing some entries.
Same bug exists in HEAD, will be patched later.

Fix timestamptz_in so that parsing of 'now'::timestamptz gives right
answer when SET TIMEZONE has been done since the start of the current
transaction.  Per bug report from Robert Haas.
I plan some futher cleanup in HEAD, but this is a low-risk patch for
the immediate issue in 7.3.

Back patch for GUC client_encoding variable not being handled
correctly. However the patch for PostgresPollingStatusType() is not
included to avoid 7.3 libpq vs. pre-7.3 backend
compatibility problem. See following thread for more details.

Subject: [HACKERS] client_encoding directive is ignored in postgresql.conf
From: Tatsuo Ishii <t-ishii@sra.co.jp>
Date: Wed, 29 Jan 2003 22:24:04 +0900 (JST)

Please apply patches for contrib/ltree.

ltree_73.patch.gz - for 7.3 :
        Fix ~ operation bug: eg '1.1.1' ~ '*.1'

ltree_74.patch.gz - for current CVS
    Fix ~ operation bug: eg '1.1.1' ~ '*.1'
    Add ? operation
    Optimize index storage

Last change needs drop/create all ltree indexes, so only for 7.4

Teodor Sigaev

Async_NotifyHandler must save and restore ImmediateInterruptOK.  Fixes
known problem with failure to respond to 'pg_ctl stop -m fast', and
probable problems if SIGINT or SIGTERM arrives while processing a
SIGUSR2 interrupt that arrived while waiting for a new client query.

Fix SPI result logic for case where there are multiple statements of the
same type in a rule.  Per bug report from Pavel Hanak.

Update FAQ's in head and 7.3.X.

Repair incorrect indexing for atttypmod, per Brad McLean.

Result of lo_read() is int, not size_t.  Per Oleg Drokin.

Parser was dropping foreign-key constraints on the floor if present in
an ALTER TABLE ADD COLUMN command.  Per bug #896.

Repair rule permissions-checking bug reported by Tim Burgess 10-Feb-02:
the table(s) modified by the original query would get checked for the
type of write permission needed by a rule query.

Prevent timetz2tm() from scribbling on its input in HAVE_INT64_TIMESTAMP case.

> > They work the same as table constraints with in-line declaration (no
> > comma).
>
> OK. But the documentation implies there is a comma, so it should probably
> get chenged then.

Yes, it should. (attached)

[ Backpatched to 7.3.X too.]

Rod Taylor

Applied patch to update translation file, submitted by Zhenbang Wei
Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/errors_zh_TW.properties

Backport patch from Fernando Nasser and Harald Krake to fix an invalid datatype problem.

 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java

Backport a patch from Kris Jurka to 7.3.  Allows getTables() method to work
against 7.1 servers.

 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/jdbc1/AbstractJdbc1DatabaseMetaData.java

Use a varno not chosen at random for dummy variables in the top-level
targetlist of a set-operation tree.  I'm not sure that this solution
will really stand the test of time --- perhaps we need to make a special
RTE for such vars to refer to.  But this quick hack fixes Brandon Craig
Rhodes' complaint of 10-Feb-02 about EXCEPT in CREATE RULE, while not
changing any behavior in the better-tested cases where leftmostRTI is
one anyway.

up build # to 108

Merge changes from head to 7.3 branch: better error message on character set conversion problems and patch from Kris Jurka for numeric scale
 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/errors.properties
  jdbc/org/postgresql/core/Encoding.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1DatabaseMetaData.java

Revise mechanism for getting rid of temp tables at backend shutdown.
Instead of grovelling through pg_class to find them, make use of the
handy dandy dependency mechanism: just delete everything that depends
on our temp schema.  Unlike the pg_class scan, the dependency mechanism
is smart enough to delete things in an order that doesn't fall foul of
any dependency restrictions.  Fixes problem reported by David Heggie:
a temp table with a serial column may cause a backend FATAL exit at
shutdown time, if it chances to try to delete the temp sequence first.

Avoid generating relid lists with duplicate entries.  Fixes bugs
reported by Stefanos Harhalakis 2/2/2003.

Fix core dump when pltcl_elog is called with wrong number of parameters,
per report from Ian Harding.

Fix regexp slowness reported by Wade Klaver.

Allow qualified type names in CREATE CAST, DROP CAST.  Also allow the
construction 'SETOF type[]' which for some reason was previously
overlooked (you'd have to name the array type directly to make it work).

Fix for rollback SQLException from Kris Jurka

Add mention of CURRENT_SCHEMA for object creation.

Brand 7.3.2.

Update release history for 7.3.2.

Fix nodeUnique to behave correctly when reversing direction after reaching
either end of subplan results.  This prevents misbehavior of cursors
on SELECT DISTINCT ... queries.  Per bug report 1-Feb-02.

Remove restriction that cast functions cannot be volatile.  This
restriction was debatable to begin with, but it has now become obvious
that it breaks forward-porting of user-defined types; contrib/lo being
the most salient example.

Cleaner solution to the problem of loading pre-7.3 dumps containing
columns of type lo (see contrib/lo).  Rather than hacking the function
definitions on-the-fly, just modify the queries issued by FixupBlobRefs
so that they work even if CREATE CAST hasn't been issued.

Changes of 6-Sep-02 broke pg_restore's ability to recognize tar-format
files.  Fix it.

Fix assign_session_authorization() to not be confused by all-numeric
user names.  Per recent reports.

Prevent core dump from calling Tcl_DontCallWhenDeleted() with a null
interp pointer.  Per report from Gerhard Hintermayer.

Apply ljb's patch to prevent both memory leak and core dump during
connection shutdown.  This is a grotty workaround for a Tcl bug, but
said bug has been there long enough that I'm not holding my breath
for a real fix.  Per discussions and testing from ljb and g.hintermayer.

Make plpython's spi_execute interface handle NULLs properly.
From Andrew Bosma.

Clean up plpython error reporting so that its regression test passes
with some amount of cleanliness.  I see no need to report the internal
Python name rather than the SQL procedure name in error tracebacks.

Tweak bison build rules so that we get the same error messages from
bison 1.875 and later as we did from earlier bison releases.  Eventually
we will probably want to adopt the newer message spelling ... but not yet.
Per recent discussion on pgpatches.
Note: I didn't change the build rules for bootstrap, ecpg, or plpgsql
grammars, since these do not affect regression test results.

Handle mixed-case names properly in plpgsql_parse_tripwordtype.
From Neil Conway.

Fix regression in .pgpass support.  From Neil Conway.

Update release history for 7.2.4.

SPI_exec shouldn't return SPI_OK_SELECT if it hasn't actually returned
a tuple table.  Fixes core dump in pltcl (and probably other PLs) when
executing a query rewritten by a rule.  Per bug report from Wolfgang Walter.

Repair array subscript overrun identified by Yichen Xie.  Reduce the
value of MAX_TIME_PRECISION in floating-point-timestamp-storage case
from 13 to 10, which is as much as time_out is actually willing to print.
(The alternative of increasing the number of digits we are willing to
print looks risky; we might find ourselves printing roundoff garbage.)

Fix array subscript overruns identified by Yichen Xie.

Convert variable name to canonical spelling before checking for matches
in GUCArrayAdd/GUCArrayDelete.  This prevents the multiple-entry bug
exhibited by Frank Lupo 28-Jan-2003.

Revert off setting mention for client_min_messages.

Set log_min_error_messages to the proper and agreed-upon default, PANIC
(off).  (BACKPATCH)

Prevent core dump when die_horribly() is called with null AH pointer.
Problem reported and fixed by Oliver Elphick.

Back-patch fix for not shell-quoting editor name, as it also fixes
silly buffer overrun (someone needs to learn to count characters...)

Fix obsolete error message (isImmutable doesn't exist).

Change CREATE TABLE AS / SELECT INTO to create the new table with OIDs,
for backwards compatibility with pre-7.3 behavior.  Per discussion on
pgsql-general and pgsql-hackers.

Update 'Mathematical Functions' table to reflect 7.3 addition of float8
forms of ceil(), floor(), sign().  Back-patch this and other recent
doc fixes in this file to the 7.3 branch.

Make estimation of mergejoin scan selectivities more robust, per recent
example from RaÇl GutiÅrrez.

Fix coredump problem in plpgsql's RETURN NEXT.  When a SELECT INTO
that's selecting into a RECORD variable returns zero rows, make it
assign an all-nulls row to the RECORD; this is consistent with what
happens when the SELECT INTO target is not a RECORD.  In support of
this, tweak the SPI code so that a valid tuple descriptor is returned
even when a SPI select returns no rows.

Back-patch typprtlen fix, two other minor post-7.3 fixes.

Fix bogus formatting of DROP DATABASE command, per Christopher Kings-Lynne.

Repair an embarrassingly large number of alphabetization mistakes in the
datetime token tables.  Even more embarrassing, the regression tests
revealed some of the problems --- but evidently the bogus output wasn't
questioned.  Add code to postmaster startup to directly check the tables
for correct ordering, in hopes of not being embarrassed like this again.

More comment cleanups for shared_buffers in postgresql.conf.

Update shared_buffer comment in postgresql.conf.

This patch fixes an incorrect statement and makes a few cleanups to
contrib/fulltextindex/README.fti

Backpatched to 7.3.X too.

Neil Conway

Bumped up build number to 107

Backport to 7.3 of patch to correctly handle fractional seconds

 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/jdbc1/AbstractJdbc1ResultSet.java
  jdbc/org/postgresql/test/jdbc2/TimestampTest.java

Patch from Kris Jurka to improve the performance of getImportedKeys().  The
fix uses explicit joins to avoid using the genetic query optimizer for
consistent results.

 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/jdbc1/AbstractJdbc1DatabaseMetaData.java

Fix capitalization.

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 typo in an error messages

> > 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

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

Manfred Koizar

Add missing <limits.h> for INT64_IS_BUSTED case.

Fix indenting of log_min_error_statement.

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.

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

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

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.

Backpatch extra slash removal into 7.3.X. HEAD already has it.

Fix doc chmod .pgpass line in 7.3.X.  Head already fixed.

Upgrade to my2pg 1.9

Upgrade to even newer ora2pg 2.0.

Upgrade to ora2pg 1.10.  Backpatch to 7.3.X.

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.

Add missing include --- seems include file references are different in
CVS tip than in this branch.

Translation updates

Add casts between oid and lo in contrib.

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.

Point does not return a center of lseg.

Kenji Sugita

Adjust hash table sizing algorithm to avoid integer overflow in
ExecHashJoinGetBatch().  Fixes core dump on large hash joins, as in
example from Rae Stiening.

Deliver better error message when a relation name is used in an expression.
Per report from Ian Barwick.

Remove overenthusiastic free'ing of comment dependencies; could lead to
core dump in pg_dump when dumping views having comments.  See bug #855.

Small fix in documentation and some examples of usage. Please, apply to
7.3 and current CVS

Oleg Bartunov

Trivial patch to increase max_fsm_pages as per earlier discussion.

Philip Warner

Update SSL error message for improper permissions.  Backpatch to 7.3.X.

Fix possible buffer overrun in \d command: substr(..., 128) produces
a result of at most 128 chars, but that could be more than 128 bytes.
Also ensure we don't try to pfree uninitialized pointers during error
cleanup.

Update 7.3.1 to give proper attribution to pg_dump item.

Fix release notes and installation instructions for 7.3.1 release.