Done:

> * -SELECT cash_out(2) crashes because of opaque

Enhancements to how queries with bind values are stored internally and sent to
the server.  Previously we allocated a new String object for the entire final
query we were sending to the database.  If you had a big query, or especially
if you had large bind values you ended up with essentially two copies in memory.
This change will reuse the existing objects and therefore should take 1/2 the
memory it does today for a given query.  This restructuring will also allow
in the future the ability to stream bytea data to the server instead of the current approach of pulling it all into memory.
I also fixed a test that was failing on a 7.2 database.
Also renamed some internal variables and some minor cleanup.

 Modified Files:
  jdbc/org/postgresql/core/QueryExecutor.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSet.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/test/jdbc2/DatabaseMetaDataTest.java

Further cleanup around the edges of OPAQUE/pseudotype changes.  Correct
the declarations of some index access method support functions.  Support
SQL functions returning VOID.

Adapt for SRF(Set Returning Function).

Not sure how I fixed it the first time, but here's a fix for another
instance in which the docs mention that 'NAMEDATALEN == 32".

Neil Conway

Add space.k>

Add:

> * Allow xlog directory location to be specified during initdb, perhaps
>   using symlinks

Patch resolve ERROR problem for non-goog query_txt.

Teodor Sigaev

This patch updates the CREATE OPERATOR sgml docs for the new default
settings of NAMEDATALEN. I looked through the docs for other
references to NAMEDATALEN, but this is the only one I could find.

Neil Conway

Update Russian FAQ.

Add:

> * Add GUC parameter to print queries that generate errors

Very minor copy-editing.

Add note that explains that \df omits functions it thinks are I/O functions.

Update reference to value of NAMEDATALEN.  Someone needs to troll the
docs and find the other obsolete statements that no doubt lurk.

Update contrib regression tests for OPAQUE datatype changes.

Since the plpgsql test has been part of the standard regression tests
for a good long while, I see no reason to maintain this original copy.

Transpose info from src/pl/plpgsql/test/README into standard regression test file.

Mark 7.2.2 as newest release.

Update FAQ.

Remove all traces of the ODBC driver, which is now on GBorg as the psqlodbc
project ...

Fix bit rot in pg_dump's ability to dump from 7.2 and 7.1 servers.

Fixed-size buffer in dumpClasses is not big enough anymore given the
addition of a column list clause to the COPY command.  Spotted by
Martin Renters.

Improve wording of upgrade section.

Code review for recent TRUNCATE changes.  Tighten relation-kind check,
tighten foreign-key check (a self-reference should not prevent TRUNCATE),
improve error message, cause a relation's TOAST table to be truncated
along with the relation.

Commit updated repeat() patch, from  Neil Conway

Fix for documention:

>>" It's also possible to select no escape character by writing ESCAPE ''.
>>In this case there is no way to turn off the special meaning of
>>underscore and percent signs in the pattern."

Joe Conway

This patch should fix the problem. Doesn't include my previous patch
for repeat(). Again, somewhat off-the-cuff, so I might have missed
something...

test=# select lpad('xxxxx',1431655765,'yyyyyyyyyyyyyyyy');
ERROR:  Requested length too large
test=# select rpad('xxxxx',1431655765,'yyyyyyyyyyyyyyyy');
ERROR:  Requested length too large

(That's on a Unicode DB, haven't tested other encodings but AFAICT
this fix should still work.)

Neil Conway

repeat() fix:

> Neil Conway <neilc@samurai.com> writes:
> > +   /* Check for integer overflow */
> > +   if (tlen / slen != count)
> > +           elog(ERROR, "Requested buffer is too large.");
>
> What about slen == 0?

Good point -- that wouldn't cause incorrect results or a security
problem, but it would reject input that we should really accept.

Revised patch is attached.

Neil Conway

> > I had great difficulty in finding how to change the search path, so here
> > is a patch to add some cross-referencing.

Oliver Elphick

# Disallow TRUNCATE on tables that are involved in referential
constraints

The issue with finding and removing foreign key constraints is no longer
an issue, so please apply the attached.

It does NOT check for rules or on delete triggers (old style foreign
keys) as those are difficult to deal with (remove, truncate, re-add).

Rod Taylor

Add:

> * Allow bytea to handle LIKE with non-TEXT patterns

As suggested by Tom, this patch restricts the right-hand argument of
bytealike to TEXT.

This leaves like_escape_bytea() without anything to do, but I left it in
place in anticipation of the eventual bytea pattern selectivity
functions. If there is agreement that this would be the best long term
solution, I'll take it as a TODO for 7.4.

Joe Conway

Oops, we got duplicate oids from patches again;  pick a unique one.

Add:

replace(string, from, to)
   -- replaces all occurrences of "from" in "string" to "to"
split(string, fldsep, column)
   -- splits "string" on "fldsep" and returns "column" number piece
to_hex(int32_num) & to_hex(int64_num)
   -- takes integer number and returns as hex string

Joe Conway

Remove libpq++ SGML documentation.

Remove erroneous character from Makefile due to editor error.

Okay, libpq++ is moved to GBorg, and all traces of it have been removed
from the core repository ... I haven't *moved* the libpq++ files out of the
tree, mainly as we want to keep them in place for past branches ...

Peter, I think I've covered all the files I need, and re-ran autoconf to make
sure the configure file is in place properly ...

Add a bunch of pseudo-types to replace the behavior formerly associated
with OPAQUE, as per recent pghackers discussion.  I still want to do some
more work on the 'cstring' pseudo-type, but I'm going to commit the bulk
of the changes now before the tree starts shifting under me ...

Add --help and --version options.  Message cleanup.  NLS.

Changes made so new conversion Makefiles will build out of the source tree.

Move PO files into subdirectories separate from the source code.

Add sys/types.h to common includes.

Fix bad markup detected by Alvaro Herrera

Fix markup,  Alvaro Herrera.

Use a non-duplicate OID for pg_lock_status.

Enable large file support.

Use off_t and size_t in pg_dump to handle file offset arithmetic correctly.

Increase buffer size in cash_words().  Pure paranoia; I don't think the
code is broken, but any small change in the output format might overrun
the buffer with the old size.

Update expected files for recent changes of domain tests.
From Rod Taylor.

This patch improves the behavior of FOUND in PL/PgSQL. In Oracle,
FOUND is set whenever a SELECT INTO returns > 0 rows, *or* when an
INSERT, UPDATE, or DELETE affects > 0 rows. We implemented the first
part of this behavior, but not the second.

I also improved the documentation on the various situations in which
FOUND can be set (excluding inside FOR loops, which I still need to
think about), and added some regression tests for this behavior.

Neil Conway

Cleanup.

- test for indexes on a domain
- don't fail type-sanity checks if a domain exists

Rod Taylor

Allow pg_statistics to be reset by calling pg_stat_reset().

Christopher Kings-Lynne

Add current_database().

> Quick system function to pull out the current database.
>
> I've used this a number of times to allow stored procedures to find out
> where they are.  Especially useful for those that do logging or hit a
> remote server.
>
> It's called current_database() to match with current_user().

It's also a necessity for an informational schema.  The catalog
(database) name is required in a number of places.

Rod Taylor

Removed code that is no longer used and has been commented out
for the last two releases.

 Modified Files:
  jdbc/org/postgresql/Driver.java.in
  jdbc/org/postgresql/PG_Stream.java
 Removed Files:
  jdbc/org/postgresql/core/BytePoolDim1.java
  jdbc/org/postgresql/core/BytePoolDim2.java
  jdbc/org/postgresql/core/MemoryPool.java
  jdbc/org/postgresql/core/ObjectPool.java
  jdbc/org/postgresql/core/SimpleObjectPool.java

Add mention of -F not supported by pg_dumpall.  Code already disallows it.

Done by Rod:

> * -Make other constraints clearer in dump file

Fix pg_dump to dump serial columns as serials.  Per pghackers discussion,
cause SERIAL column declaration not to imply UNIQUE, so that this can be
done without creating an extra index.

Remove Ident nodetype in favor of using String nodes; this fixes some
latent wrong-struct-type bugs and makes the coding style more uniform,
since the majority of places working with lists of column names were
already using Strings not Idents.  While at it, remove vestigial
support for Stream node type, and otherwise-unreferenced nodes.h entries
for T_TupleCount and T_BaseNode.
NB: full recompile is recommended due to changes of Node type numbers.
This shouldn't force an initdb though.

Fix bug in pg_convert() per report from MaC.Yui.
It pfree() wrong pointer.

Fix broken tag.

Clean up leftover bugs from recent COPY feature patch --- missed
required changes to copyfuncs/equalfuncs.

Add missing copyfuncs/equalfuncs entries, including T_Null which has
been missing forever; surprising it wasn't noticed before.  The other
additions are, um, sloppiness in certain recent feature additions.

Improve obsolete comment.

Fix small copy-and-pasteo.

Fix potential buffer overrun in cube_out(), per report from
Bruno Wolff.

Remove optimization whereby parser would make only one sort-list entry
when two equal() targetlist items were to be added to an ORDER BY or
DISTINCT list.  Although indeed this would make sorting fractionally
faster by sometimes saving a comparison, it confuses the heck out of
later stages of processing, because it makes it look like the user
wrote DISTINCT ON rather than DISTINCT.  Bug reported by joe@piscitella.com.

Make CREATE CONSTRAINT TRIGGER check for REFERENCES privilege on both
master and slave tables.

Make pg_dump output more portable and more pleasing to look at.

The -n and -N options were removed.  Quoting is now smart enough to
supply quotes if and only if necessary.

Numerical types are now printed without quotes, except in cases of
special values such as NaN.

Boolean values printed as true and false.

Most string literals now do not escape whitespace characters (newlines,
etc.) for portability.

SET SESSION AUTHORIZATION argument is a string literal, to follow SQL.

Made commands output by pg_dump use consistent spacing and indentation.

Add  #include <sys/time.h>.

Code review for 'at character n' patch --- point at proper end of
a token scanned by multiple lex rules.

Add db-local user names, per discussion on hackers.

More changes to match new error format.

Use variable for format to suppress overly-anal-retentive gcc warning
about %c

Modify regression tests to match new error reporting format from Gavin.

Back out tcl patch, per Tom Lane:

Everytime if I do PQconsumeInput (when the backend channel gets
readable) I check for the return value. (0 == error) and generate a
notification manually, e.g. fixed string connection_closed) and pass it to the

Add includes for prototype using timeval.

Clean up compile warnings.

Fix typo.

Reverse out XLogDir/-X write-ahead log handling, per discussion.
Original patch from Thomas.

Add:

> * Use dependency information to dump data in proper order

Done:

>  o -Add command to display locks

Add lock file.

In that case, attached is a patch which locates the beginning of the
offending token more efficiently (per your suggestion of using
scanbuf). The new patch does the same as before:

template1=# select * frum pg_class;
ERROR:  parser: parse error at or near "frum" at character 10

It also implement's Tom's suggestion:

template1=# select * from pg_class where\g
ERROR:  parse: parse error at end of input

Gavin Sherry

[ Newest version of patch applied.]

This patch is an updated version of the lock listing patch. I've made
the following changes:

    - write documentation
    - wrap the SRF in a view called 'pg_locks': all user-level
      access should be done through this view
    - re-diff against latest CVS

One thing I chose not to do is adapt the SRF to use the anonymous
composite type code from Joe Conway. I'll probably do that eventually,
but I'm not really convinced it's a significantly cleaner way to
bootstrap SRF builtins than the method this patch uses (of course, it
has other uses...)

Neil Conway

Add libpq connection timeout parameter.

Denis A Ustimenko

What I have done for libpgtcl:
Everytime if I do PQconsumeInput (when the backend channel gets
readable) I check for the return value. (0 == error) and generate a
notification manually, e.g. fixed string connection_closed) and pass it to the
TCL event queue. The only other thing I had to do is to comment out removing
all pending events in PgStopNotifyEventSource whenever the connection was
unexpectedly closed (so the manually generated event will not be deleted).

A broken backend connection triggers a notify event to the client (fixed
notification string "connection_closed") so proper action can be taken to switch
to another database server etc. Remember that this is event driven. If you have
applications, that have idle database connections most of the time, you'll get
immediate feedback of a dying server. Upon connection to the server issue a
pg_notify for notify event "connection_closed" and whenever the backend crashes
(which it does do in very very rare cases) you get an event driven recovery. (of
course the Tcl-Event loop has to be processed). Issuing a notification
"connection_closed" on a still working database could be used for switching to
another db-server (which I've actually impelemented right now).

Gerhard Hintermayer

Improve documentation of trigger firing queue handling, cleanup.

Neil Conway

Editorial corrections.

Broken markup.

Cleanup.

Fix broken Makefiles.

Add missing links.

Add doc file for pg_resetxlog.

Add files for pg_resetxlog.

Move pg_controldata to /bin.

Modify pg_dump to dump foreign-key constraints as constraints, not as
sets of triggers.  Also modify psql \d command to show foreign key
constraints as such and hide the triggers.  pg_get_constraintdef()
function added to backend to support these.  From Rod Taylor, code
review and some editorialization by Tom Lane.

Code review for patch to dump primary and unique constraints as
constraints, rather than as CREATE INDEX commands.

Make domain types indexable.  Rod Taylor & Tom Lane.

Improve wording.

Move pg_controldata from /contrib to src/bin.