Fix misstatement in release-notes item.

Update for 7.3.3.

Brand 7.3.3.

Update release history for impending 7.3.3 release.

Repair sometimes-incorrect computation of StartUpID after a crash, per
example from Rao Kumar.  This is a very corner corner-case, requiring
a minimum of three closely-spaced database crashes and an unlucky
positioning of the second recovery's checkpoint record before you'd notice
any problem.  But the consequences are dire enough that it's a must-fix.

Back-patch Jan's fix to avoid primary key lookup (and lock) if foreign key
does not change on UPDATE.

Use -fPIC on Sparc, per Tom Callaway.

Back-patch change to avoid O(N^2) behavior with lots of deferred triggers,
by making deferredTriggerInvokeEvents only scan events added since it last ran.

Upped JDBC build number to 110 for the 7.3.3 release

 Modified Files:
  Tag: REL7_3_STABLE
  Driver.java.in

Repair quoting sloppiness, lack of schema awareness in reindexdb.

Small translation updates for 7.3.3 release.

Fix failure when uniq-ifying an array of zero elements.

Teodor Sigaev

Back-patch fix to allow createuser to exit on control-C (Oliver Elphick)

Back-patch fix from Oliver Elphick to force ISO datestyle in dumps.

Small changes to use the absolute path to system catalogs.

Greg Sabino Mullane

Check calling context for connectby_text(), per Joe Conway.

Apply fixes for problems with dropped columns whose types have also been
dropped.  Add regression test, too.

Adjust CreateCheckpoint so that buffer dumping activities and cleanup of
dead xlog segments are not considered part of a critical section.  It is
not necessary to force a database-wide panic if we get a failure in these
operations.  Per recent trouble reports.

Be more precise about check for flex 2.5.3, backpatched to 7.3.X.

Allow 60 in seconds fields of timestamp, time, interval input values.
Per recent discussion on pgsql-general, this is appropriate for spec
compliance, and has the nice side-effect of easing porting from old
pg_dump files that exhibit the 59.999=>60.000 roundoff problem.

When a TIMESTAMP, TIME, or INTERVAL precision is specified larger than our
implementation limits, do not issue an ERROR; instead issue a NOTICE and use
the max supported value.  Per pgsql-general discussion of 28-Apr, this is
needed to allow easy porting from pre-7.3 releases where the limits were
higher.

Unrelated change in same area: accept GLOBAL TEMP/TEMPORARY as a synonym
for TEMPORARY, as per pgsql-hackers discussion of 15-Apr.  We previously
rejected it, but that was based on a misreading of the spec --- SQL92's
GLOBAL temp tables are really closer to what we have than their LOCAL ones.

Fix erroneous space calculation leading to core dump in dumpProcLangs,
per report from Olivier Prenant.  Also fix off-by-one space calculation
in ReadToc; this woould not have hurt us until we had more than 100
dependencies for a single object, but wrong is wrong.

Repair permissions problem in RI triggers: query parsing has to be done
as the correct user, not only query execution.  Per report from Sean
Chittenden.

Correct oversight in createlang: test for pre-existing handler function
was broken by opaque->language_handler change.  I see this is already
fixed in CVS tip, but must back-patch for 7.3.3.

Back-patch fixes for zero-column tables in COPY, pg_dump.

Make [VACUUM] ANALYZE safe on zero-column tables.

Fix misbehavior of func_error() on type names containing '%'.
Will patch separately but equivalently in HEAD.

Fix misbehavior of replace() on strings containing '%'.
Will patch separately but equivalently in HEAD.

Prevent palloc(0) error when parent table has zero columns.

Fix char-vs-pg_wchar confusion in p_ere(), per failure report from
Tom O'Dowd.  This fix is not relevant to CVS tip anymore, but we should
fix it in 7.3.*.

Fix abstime-to-time cast function, which has had broken implementation
since 7.2, per bug #947.  Turns out it had wrong volatility label, too.
Can't force initdb in 7.3 branch, but fix anyway for future installs.

Applied patches from Kris Jurka fixing a string tokenizing problem and
fixing an order by problem for index metadata results.
Also includes removing some unused code as well as a fix to the toString
method on statement.

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

Fix stupid oversight ...

eqjoinsel's logic for case where MCV lists are not present should
account for NULLs; in hindsight this is obvious since the code for
the MCV-lists case would reduce to this when there are zero entries
in both lists.  Per example from Alec Mitchell.

Second try at avoiding conflicts with system isblank().

Guard against macro versions of isblank().

Fix encoding conversion function bug.
See following posting for more details.

Subject: Re: [HACKERS] [BUGS] Bug #943: Server-Encoding from EUC_TW to UTF-8 doesn't
From: Tatsuo Ishii <t-ishii@sra.co.jp>
To: michael.enke@wincor-nixdorf.com, pgsql-bugs@postgresql.org
Cc: pgsql-hackers@postgresql.org
Date: Sat, 12 Apr 2003 10:51:45 +0900 (JST)

Fix error recovery for SSL_read/SSL_write calls.

Remove premature attempt to constant-fold type coercion expressions.
While usually safe, this fails if the coercion function needs the
query snapshot to be set already.  Per example from Nigel Andrews.

added addDataType to the interface

Back-patch changes to validate page header fields immediately after
reading in any page.  Also back-port the zero_damaged_pages boolean
that determines what to do about it.

Repair incorrect checking of grouped/ungrouped variables in the presence
of unnamed joins; per pghackers discussion 31-Mar-03.

Fix buffer overrun in to_ascii(), per report from Guido Notari.

Please, apply patch for contrib/ltree to current CVS and 7.3.2

CHANGES

Mar 28, 2003
Added finctions index(ltree,ltree,offset), text2ltree(text),
ltree2text(text)

Teodor Sigaev

TestConfiguration returns int, not bool.  This mistake is relatively
harmless on signed-char machines but would lead to core dump in the
deadlock detection code if char is unsigned.  Amazingly, this bug has
been here since 7.1 and yet wasn't reported till now.  Thanks to Robert
Bruccoleri for providing the opportunity to track it down.

Properly document default value of log_min_error_statement in postgresql.conf.

Backpatch SSL return value changes.

[ Backpatch to 7.3.X.]

typing error in src/backend/libpq/be-secure.c ???

Long Description
In src/backend/libpq/be-secure.c: secure_write
on SSL_ERROR_WANT_WRITE call secure_read instead
secure_write again. May be is this a typing error?

Sergey N. Yatskevich (syatskevich@n21lab.gosniias.msk.ru)

Fix bogus coding of SET DEFAULT ri triggers ... or at least make it less
bogus than it was.  Per bug report from Adrian Pop.

GetTupleForTrigger must use outer transaction's command counter for time
qual checking, not GetCurrentCommandId.  Per test case from Steve Wolfe.

fixed problem where information from previous updates was leaking into subsequent updates patch from Shawn Green, slightly modified

added DISTINCT to the query to get cross reference. This is required when two columns in a table are both foreign keys to another table. From Peter Royal proyal@pace2020.com

fixed problem where information from previous updates was leaking into subsequent updates patch from Shawn Green

New build number

Backport fix for parsing acls (didn't correctly interpret null acl to mean
table owner has privs)
 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/jdbc1/AbstractJdbc1DatabaseMetaData.java

Band-aid patch for Shraibman's 'out of free buffers' bug: disable the
keys_are_unique optimization introduced in 7.3.  A better fix will
appear in 7.4, but I think back-patching it is too risky for the stable
branch.

Tweak selectivity and related routines to cope with domains.  Per report
from Andreas Pflug.

The attatched patch fixes a memory error with contrib/dbmirror/pending.c
when running it with older(Pre 7.3.x) versions of Postgresql.

Backpatched to 7.3.X.

Steven Singer

Backport patch to work around a server bug.  Server incorrectly handles the
following:  select 1; commit; set autocommit true;
If this is submitted in one call to the server (the select 1 doesn't start a
new transaction like it should), however if the select 1 is sent as a separate
call then it works correctly.

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

Applied translation fixes

 Modified Files:
  Tag: REL7_3_STABLE
  errors_zh_TW.properties

pg_dumpall failed on groups having no members.  Per report from
Nick Eskelinen.

Backport fix for parsing \n in updateable result sets to 7.3 branch

 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSet.java

Repair incorrect prorettype entry for timestamptz_izone.  Can't force
initdb in the 7.3 branch, but we can at least make it right for people
who install 7.3.3 from scratch.

Backport batch update error message improvements to 7.3 branch

 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/errors.properties
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/util/PSQLException.java

Prevent infinite loop in ln_var() due to roundoff error.
Per report from Dave Marin.

GROUP BY got confused if there were multiple equal() GROUP BY items.
This bug has been latent since 7.0 or maybe even further back, but it
was only exposed when parse_clause.c stopped suppressing duplicate
items (see its rev 1.96 of 18-Aug-02).

pg_restore failed to restore blobs if -X disable-triggers is specified.

Back ported patches from head to 7.3 branch (fixes for NPE in result set absolute() and NPE when logging enabled)
 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSet.java
  jdbc/org/postgresql/util/PSQLException.java

Tweak dependency code to suppress NOTICEs generated by new method for
cleaning out temp namespaces.  We don't really want the server log to be
cluttered with 'Drop cascades to table foo' every time someone uses a
temp table...

Add missing --globals-only (long form of -g).

Repair bug reported by Laurent Perez: bad plan generated when UPDATE or
DELETE of an inheritance tree references another inherited relation.
This bug has been latent since 7.1; I'm still not quite sure why 7.1 and
7.2 don't manifest it (at least, they don't crash on a simple test case).

Prevent clustering on incomplete indexes: partial indexes are verboten,
as are non-amindexnulls AMs unless first column is attnotnull.

Repair memory leak introduced by recent change to make SPI return a
tupdesc even with zero tuples returned: some plpgsql routines assumed
they didn't need to do SPI_freetuptable() after retrieving no tuples.

Change EXTRACT(EPOCH FROM timestamp) so that a timestamp without time zone
is assumed to be in local time, not GMT.  This improves consistency with
other operations, which all assume local timezone when it matters.  Per
bug #897.

Back patch bug fix to quote function.

Backport a fix from 7.4 to 7.3 to better handle case in updateable result sets.
This backports part of the fix made in version 1.11.

 Modified Files:
  Tag: REL7_3_STABLE
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSet.java

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.