last-minute CreateTemplateTupleDesc fixes.

Change ownership:

<  o -ALTER TABLE ADD PRIMARY KEY (Christopher)
<  o -ALTER TABLE ADD UNIQUE (Christopher)
>  o -ALTER TABLE ADD PRIMARY KEY (Tom)
>  o -ALTER TABLE ADD UNIQUE (Tom)

Documentation improvements.

Code review and documentation updates for indisclustered patch.

Partial solution for 'unexpected EOF' problem with pg_disconnect: it
seems we have a choice between annoying messages and leaking memory
(or dumping core, but that's right out).  Patch also fixes several
other problems in pg_disconnect, such as being willing to close a
channel that isn't a PG channel.

Translation updates

Avoid shell syntax error if list of languages is empty.

Fix typos.

Convert control status codes to human-readable strings.

pg_on_connection_loss command for libpgtcl.  Patch from
Gerhard Hintermayer, revised and documented by Tom Lane.

This patch also fixes a 'must fix' bug: libpgtcl's LISTEN/NOTIFY
support was broken by the recent changes to the PGnotify structure.
Guess that change wasn't quite so safe as we thought.

Fix obsolete description of PGnotify struct.

Department of second thoughts: make checks for replacing a view slightly
more flexible, and improve the error reporting.  Also, add documentation
for REPLACE RULE/VIEW.

No more need for 'privileges' file.

Improve CVS log grouping.

Updated from Tom:

<  o -ALTER TABLE ALTER COLUMN SET/DROP NOT NULL (Christopher Kings-Lynne)
<  o ALTER TABLE ADD COLUMN column SERIAL doesn't create sequence
>  o -ALTER TABLE ALTER COLUMN SET/DROP NOT NULL (Christopher)
200a200,201
>  o ALTER TABLE ADD COLUMN column SERIAL doesn't create sequence because
>           of the item above
232c233
<  o -Disallow missing columns in INSERT ... VALUES, per ANSI
>  o -Disallow missing columns in INSERT ... (col) VALUES, per ANSI
335,336d335
< * Have SERIAL generate non-colliding sequence names when we have
<   auto-destruction

Add check for Ant version >= 1.5.

Updates from Tom:

< * Allow logging of query durations
> * -Allow logging of query durations
51,52d50
< * Make single-user local access permissions the default by limiting
<   permissions on the socket file (Peter E)
72,73c70,71
< * Reserve last few process slots for super-user if max_connections reached
< * Add GUC parameter to print queries that generate errors
> * -Reserve last few process slots for super-user if max_connections reached
> * -Add GUC parameter to print queries that generate errors
82,83c80,81
< * Declare typein/out functions in pg_proc with a special "C string" data type
< * Functions returning sets do not totally work
> * -Declare typein/out functions in pg_proc with a special "C string" data type
> * -Functions returning sets do not totally work
90c88
< * Allow bytea to handle LIKE with non-TEXT patterns
> * -Allow bytea to handle LIKE with non-TEXT patterns
94c92
<  o Store binary-compatible type information in the system
>  o -Store binary-compatible type information in the system
97d94
<  o -SELECT col FROM tab WHERE numeric_col = 10.1 fails, requires quotes
102c99
<  o Ensure we have array-eq operators for every built-in array type
>  o -Ensure we have array-eq operators for every built-in array type
119d115
< * Allow setting database character set without multibyte enabled
152d147
< * Have UPDATE/DELETE clean out indexes
198,199d192
<  o ALTER TABLE ADD COLUMN to inherited table put column in wrong place
<    [inheritance]
201d193
<  o Add ALTER FUNCTION
203,204c195,196
<  o -ALTER TABLE ADD PRIMARY KEY (Tom)
<  o -ALTER TABLE ADD UNIQUE (Tom)
>  o -ALTER TABLE ADD PRIMARY KEY (Christopher)
>  o -ALTER TABLE ADD UNIQUE (Christopher)
248c240
<  o -Remove SET KSQO option now that OR processing is improved (Tom)
>  o -Remove SET KSQO option now that OR processing is improved (Bruce)
280c272
< * Have pg_dump use LEFT OUTER JOIN in multi-table SELECTs
> * -Have pg_dump use LEFT OUTER JOIN in multi-table SELECTs
287d278
< * Add config file check for $ODBCINI, $HOME/.odbc.ini, installpath/etc/odbc.ini
318,322d308
< * ODBC
<  o ODBC 3.0 support
<  o Unicode(UCS-2) support
<  o Updatable cursors support
<
337c323
< * Fix foreign key constraints to not error on intermediate db states (Stephan)
> * -Fix foreign key constraints to not error on intermediate db states (Stephan)
352c338
< * Propagate column or table renaming to foreign key constraints
> * -Propagate column or table renaming to foreign key constraints
447c433
< * Remove wal_files postgresql.conf option because WAL files are now recycled
> * -Remove wal_files postgresql.conf option because WAL files are now recycled
460c446
< * Improve dynamic memory allocation by introducing tuple-context memory
> * -Improve dynamic memory allocation by introducing tuple-context memory
463c449
< * Nested FULL OUTER JOINs don't work (Tom)
> * -Nested FULL OUTER JOINs don't work (Tom)

Fix breakage introduced by evidently-completely-untested snprintf patch.

Fix broken markup.

Fix conversion regressin test failure.

Done:

> * -Add OR REPLACE clauses to non-FUNCTION object creation
> * -Allow autocommit so always in a transaction block
> * -Cache most recent query plan(s) (Neil) [prepare]

Add missing dblink files.

Fixes for intagg regression tests to match new array internal handling.

mlw

In case Florian and I don't finish his changes to this contrib before
beta, at least get this stuff in.

ftipatch.txt - Updates to docs and scripts.  Run in the fulltextindexdir
WARNING - Add to fulltextindex dir
uninstall.sql - Add to fulltextindex dir

Update catversion so we are sure everyone gets the bytea LIKE fix.

> Okay.  When you get back to the original issue, the gold is hidden in
> src/backend/optimizer/path/indxpath.c; see the "special indexable
> operators" stuff near the bottom of that file.  (It's a bit of a crock
> that this code is hardwired there, and not somehow accessed through a
> system catalog, but it's what we've got at the moment.)

The attached patch re-enables a bytea right hand argument (as compared
to a text right hand argument), and enables index usage, for bytea LIKE

Joe Conway

This patch fixes a minor inaccuracy in the documentation: NOT NULL is
not synonymous with CHECK (xxx IS NOT NULL) -- for example, consider
ALTER TABLE ADD PRIMARY KEY, which checks for 'NOT NULL', not a check
constraint.

Neil Conway

Tab completion in psql is missing the REINDEX command.

Alvaro Herrera

Attached is a fairly sizeable update to contrib/dblink. I'd love to get
review/feedback if anyone is interested and can spend the time. But I'd
also love to get this committed and address changes as incremental
patches ;-), so if there are no objections, please apply.

Below I'll give a synopsis of the changes. More detailed descriptions
are now in a new doc directory under contrib/dblink. There is also a new

dblink.test.sql file which will give a pretty good overview of the
functions and their use.

Joe Conway

I checked all the previous string handling errors and most of them were
already fixed by You. However there were a few left and attached patch
should fix the rest of them.

I used StringInfo only in 2 places and both of them are inside debug
ifdefs. Only performance penalty will come from using strlen() like all
the other code does.

I also modified some of the already patched parts by changing
snprintf(buf, 2 * BUFSIZE, ... style lines to
snprintf(buf, sizeof(buf), ... where buf is an array.

Jukka Holappa

I guess the intention was to test incomplete SELECT statements, not
missing semicolons.

I also added a SELECT statement without a target list.

Manfred Koizar

Okay, attached is a patch that moves the alter table regression test
into a separate parallel group. I can't confirm that this fixes the
problem for me (since it only occurs about one in ten or one in twenty
runs), but I think it's a good bet that it does...

Neil Conway

Here's a doc patch for the new string functions recently committed --
replace, split, and to_hex. The patch also moves encode and decode into
alphabetical order (since everything else in the table was).

Joe Conway

Move pg_settings doc into the right section.

Here is a documentation patch for the pg_settings virtual table. If
there are no objections, please apply.

Joe Conway

Attached is an update to contrib/tablefunc. It introduces a new
function, connectby(), which can serve as a reference implementation for

the changes made in the last few days -- namely the ability of a
function to return an entire tuplestore, and the ability of a function
to make use of the query provided "expected" tuple description.

Description:

   connectby(text relname, text keyid_fld, text parent_keyid_fld,
     text start_with, int max_depth [, text branch_delim])
   - returns keyid, parent_keyid, level, and an optional branch string
   - requires anonymous composite type syntax in the FROM clause. See
     the instructions in the documentation below.

Joe Conway

Add GUC variable to print original query to the server logs when there
is an error, warning, etc.

Gavin Sherry

Remove documentation that says debug_query_string is only used by
pgmonitor.

Now log_min_error_statement uses it.

Fix ordering regression problem on freebsd/alpha by adding ORDER BY.

Removed because we don't use cs() anymore on AIX:

< * Evaluate AIX cs() spinlock macro for performance optimizations (Tatsuo)

I tried to build PostgreSQL with the following step to see backends hung
during the regression test. The problem has been reproduced on two machine
but both of these are the same type of hardware and software. I also tried
to recreate the problem on other machines, on older version of AIX but I
couldn't.

After looked through pgsql-hackers mailing list, I focused on spin lock
issue to solve the problem. The easiest and may not be the best solution
for the problem is to give up HAS_TEST_AND_SET. This actually works.

One another and better solution for the problem is to use _check_lock() and
_clear_lock() as spin lock.  Important thing here is to define S_UNLOCK()
with _clear_lock().  This will solve the so called "Compiler bug" issue
someone wrote on the mailing list.

We have some other API such as cs(), compare_and_swap() and fetch_and_or()
to do test and set on AIX, but any of these didn't solve my problem.  I
wrote tiny testing program to see if we have any bug of these API of AIX,
but I couldn't see any problem except for compare_and_swap(). It seems that
you can not use compare_and_swap() for the purpose, as it would not work as
spin lock on any SMP machines I tested.  I don't know the reason why cs()
nor fetch_and_or()/fetch_and_and() will not work with PostgreSQL on p690.
These worked with my testing program on all machines I tested.

Tomoyuki Niijima

JDBC checkin fixing the following bugs:
  Fixed support in the driver for notifications (added PGConnection.getNotifications()) - problem reported by Benjamin.Feinstein@guardent.com
  Worked around server problems with int8/int2 and constants; quote values when they are intended to bind to an int8/int2 column - reported by many
  Fixed bug in the Array interface with string parsing not handling escaped characters correctly - reported by devajx@yahoo.com
  Added workaround to support 'infinity' and '-infinity' for dates - reported bydmitry@openratings.com
  Fixed some performance issues with setBlob - reported by d.wall@computer.org
  Added support for using new prepared statements functionality in 7.3 (added PGStatement.setUseServerPrepare() and isUseServerPrepare() methods)

 Modified Files:
  jdbc/org/postgresql/PGConnection.java
  jdbc/org/postgresql/PGStatement.java
  jdbc/org/postgresql/core/QueryExecutor.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1ResultSet.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/jdbc2/Array.java
 Added Files:
  jdbc/org/postgresql/PGNotification.java
  jdbc/org/postgresql/core/Notification.java

Remove sys/types.h in files that include postgres.h, and hence c.h,
because c.h has sys/types.h.

CREATE OR REPLACE VIEW, CREATE OR REPLACE RULE.
Gavin Sherry, Neil Conway, and Tom Lane all got their hands dirty
on this one ...

Code review for HeapTupleHeader changes.  Add version number to page headers
(overlaying low byte of page size) and add HEAP_HASOID bit to t_infomask,
per earlier discussion.  Simplify scheme for overlaying fields in tuple
header (no need for cmax to live in more than one place).  Don't try to
clear infomask status bits in tqual.c --- not safe to do it there.  Don't
try to force output table of a SELECT INTO to have OIDs, either.  Get rid
of unnecessarily complex three-state scheme for TupleDesc.tdhasoids, which
has already caused one recent failure.  Improve documentation.

patch from Thomas O'Dowd to ensure that connections are closed

testing for insertRow

Update MEMSET_LOOP_LIMIT to 1024 because it is the best value for most
platforms.

patch to allow insertRow on an empty resultSet

Changed to reflect the current reality, ie we do support updateable resultsets

Improve footer length computation for psql.

Greg Sabino Mullan

Add log_duration to GUC/postgresql.conf.

Rename debug_print_query to log_statement and rename show_query_stats to
show_statement_stats.

Add:

>  o Add table function support to pltcl, plperl, plpython

plpgsql functions can return RECORD, per Neil Conway.

Minor copy-editing.

Code cleanups: make non-implicit WITHOUT FUNCTION casts work, avoid
redundant pg_cast searches, fix obsolete comments.

Remove mention of wal_files in a SHOW example.

Remove code that suggested increasing wal_files.

Done:

>  o -SELECT col FROM tab WHERE numeric_col = 10.1 fails, requires quotes

Mark the float8 -> int8 cast as implicit.  This resolves the problem
pointed out by Barry Lind: UPDATE bigintcol = 10000000000 fails because
the constant is initially taken as float8.  We really need a better way,
but it's not gonna happen for 7.3.

Also, remove int4reltime() function, which is redundant with the
existing binary-compatibility coercion path from int4 to reltime,
and probably has been unreachable code for a long while.

Code review for domain-constraints patch.  Use a new ConstraintTest node
type for runtime constraint checks, instead of misusing the parse-time
Constraint node for the purpose.  Fix some damage introduced into type
coercion logic; in particular ensure that a coerced expression tree will
read out the correct result type when inspected (patch had broken some
RelabelType cases).  Enforce domain NOT NULL constraints against columns
that are omitted from an INSERT.

Wups, didn't mean to commit that just yet.

The UNDEFOID later causes an assertion failure in heap_formtuple when
you try to use the tupdesc to build a tuple.

Joe Conway

Code review for pg_locks feature.  Make shmemoffset of PROCLOCK structs
available (else there's no way to interpret the list links).  Change
pg_locks view to show transaction ID locks separately from ordinary
relation locks.  Avoid showing N duplicate rows when the same lock is
held multiple times (seems unlikely that users care about exact hold
count).  Improve documentation.

*** empty log message ***

Add expected tuple descriptor to ReturnSetInfo information for table
functions, per suggestion from John Gray and Joe Conway.  Also, fix
plpgsql RETURN NEXT to verify that returned values match the expected
tupdesc.

Updates to reflect availability of autocommit option.

AUTOCOMMIT mode is now an available backend GUC variable; setting it
to false provides more SQL-spec-compliant behavior than we had before.
I am not sure that setting it false is actually a good idea yet; there
is a lot of client-side code that will probably be broken by turning
autocommit off.  But it's a start.

Loosely based on a patch by David Van Wie.

Fix for breakage of C-coded SRFs, from Joe Conway.

Add attisinherited column to pg_attribute; use it to guard against
column additions, deletions, and renames that would let a child table
get out of sync with its parent.  Patch by Alvaro Herrera, with some
kibitzing by Tom Lane.

Back out unintented tab change.

Remove pgaccess from Makefile.

Run autoconf.

Update documentation for --with-perl.

Improve wording after wal_files removal.

Complete TODO item:

* Remove wal_files postgresql.conf option because WAL files are
  now recycled

Remove pgaccess from CVS.  It has its own home now.

Re-enable perl configure tests for psql help file generation.

Back out password packet length check.

Improve wording of pre-7.3 syntax mention.

perl5 interface moved to gborg

libpgeasy moved to gborg ...

Fix off-by-one allocation error in PasswordFromFile(), per Gordon Runkle.

Add comment on old sytax for SELECT FOR UPDATE/LIMIT and COPY.

Revert _LARGEFILE64_SOURCE addition; good try but doesn't work,
at least not on HPUX 10.20, and there's no reason to think it
is needed on later versions.

intagg subdirectory was missing.

Group-manipulating code neglected to fill in array element type,
which is now required.

PL/pgSQL functions can return sets.  Neil Conway's patch, modified so
that the functionality is available to anyone via ReturnSetInfo, rather
than hard-wiring it to PL/pgSQL.

Fix a bug introduced in 7.2.

> > > > If you want to put in security restrictions that are actually useful,
> > > > where is the code to verify that PGPASSWORDFILE points at a
> > > > non-world-readable file?  That needs to be there now, not later, or
> > > > we'll have people moaning about backward compatibility when we finally
> > > > do plug that hole.

Alvaro Herrera

Sir Mordred The Traitor <mordred@s-mail.com> writes:
> Upon invoking a polygon(integer, circle) function a
> src/backend/utils/adt/geo_ops.c:circle_poly() function will gets
> called, which suffers from a buffer overflow.
>
> 2) A src/backend/adt/utils/geo_ops.c:path_encode() fails to detect a
> buffer overrun condition. It is called in multiple places, the most
> interesting are path_out() and poly_out() functions.

> 5) A src/backend/utils/adt/geo_ops.c:path_add() also fails to detect
> a simple buffer overrun.

I've attached a patch which should fix these problems.

Neil Conway

Add cube changes file.

The changes I have made are described in CHANGES. This was based on
diffs to 7.3-devel and may not be applicable to 7.2. I have included a
change covered by a previous bugfix patch I submitted (the problem with
-.1 not being accepted by cube_in). It does not include a fix for the
potential buffer overrun issue I reported for cube_yyerror in
cubeparse.y.

Bruno Wolff III

Make pg_resetxlog options parsing more standard and prepare messages for
translation.

Create "po" subdirectory before writing there.

Workaround for broken large file support on HP-UX

Prevent problem with extra-long password packets from allocating lots of
memory.

Neil Conway

This patch reserves the last superuser_reserved_connections slots for
connections by the superuser only.

This patch replaces the last patch I sent a couple of days ago.

It closes a connection that has not been authorised by a superuser if it would
leave less than the GUC variable ReservedBackends
(superuser_reserved_connections in postgres.conf) backend process slots free
in the SISeg. This differs to the first patch which only reserved the last
ReservedBackends slots in the procState array. This has made the free slot
test more expensive due to the use of a lock.

After thinking about a comment on the first patch I've also made it a fatal
error if the number of reserved slots is not less than the maximum number of
connections.

Nigel J. Andrews

please apply small patch for README.tsearch.

I've documented space usage and using CLUSTER command

Oleg Bartunov

The unit for statement_timeout is s/1000, not s/1000000

Manfred Koizar

Back out patch, that was alpha, which I think still needs just -O.