No change.

> Now I'm testing connectby()  in the /contrib/tablefunc in 7.3b1, which would
> be a useful function for many users.   However, I found the fact that
> if connectby_tree has the following data, connectby() tries to search the end
> of roots without knowing that the relations are infinite(-5-9-10-11-9-10-11-)
.
> I hope connectby() supports a check routine to find infinite relations.
>
>
> CREATE TABLE connectby_tree(keyid int, parent_keyid int);
> INSERT INTO connectby_tree VALUES(1,NULL);
> INSERT INTO connectby_tree VALUES(2,1);
> INSERT INTO connectby_tree VALUES(3,1);
> INSERT INTO connectby_tree VALUES(4,2);
> INSERT INTO connectby_tree VALUES(5,2);
> INSERT INTO connectby_tree VALUES(6,4);
> INSERT INTO connectby_tree VALUES(7,3);
> INSERT INTO connectby_tree VALUES(8,6);
> INSERT INTO connectby_tree VALUES(9,5);
>
> INSERT INTO connectby_tree VALUES(10,9);
> INSERT INTO connectby_tree VALUES(11,10);
> INSERT INTO connectby_tree VALUES(9,11);    <-- infinite
>

The attached patch fixes the infinite recursion bug in
contrib/tablefunc/tablefunc.c:connectby found by Masaru Sugawara.

test=# SELECT * FROM connectby('connectby_tree', 'keyid',
'parent_keyid', '2', 4, '~') AS t(keyid int, parent_keyid int, level
int, branch text);
  keyid | parent_keyid | level |   branch
-------+--------------+-------+-------------
      2 |              |     0 | 2
      4 |            2 |     1 | 2~4
      6 |            4 |     2 | 2~4~6
      8 |            6 |     3 | 2~4~6~8
      5 |            2 |     1 | 2~5
      9 |            5 |     2 | 2~5~9
     10 |            9 |     3 | 2~5~9~10
     11 |           10 |     4 | 2~5~9~10~11
(8 rows)

test=# SELECT * FROM connectby('connectby_tree', 'keyid',
'parent_keyid', '2', 5, '~') AS t(keyid int, parent_keyid int, level
int, branch text);
ERROR:  infinite recursion detected

I implemented it by checking the branch string for repeated keys
(whether or not the branch is returned). The performance hit was pretty
minimal -- about 1% for a moderately complex test case (220000 record
table, 9 level tree with 3800 members).

Joe Conway

> BTW, clusterdb is not schema-aware and will surely fail in any database
> where more than one schema is in use, because it doesn't trouble to
> schema-qualify table names.

Ok, the following patch should solve this concern.  It also tries to
connect as little times as possible (the previous one would connect one
time per table plus one per database; this one connects two times per
database).

Alvaro Herrera

The attached small patch fixes the cause of the regression test failure
for contrib/intarray.

The cause was that the library uses its own function to construct a new
array, new_intArrayType, and that function did not set the new array
struct attribute elemtype.

Joe Conway

The attached removes the current non-standard file
"contrib/tablefunc/tablefunc-test.sql", and adds a standard regression
test suite to contrib/tablefunc.

Joe Conway

Add sprompt.obj to Win32 makefiles.

Add comment about sharing of sprompt.c file.

Tweak querytree-dependency-extraction code so that columns of tables
that are explicitly JOINed are not considered dependencies unless they
are actually used in the query: mere presence in the joinaliasvars
list of a JOIN RTE doesn't count as being used.  The patch touches
a number of files because I needed to generalize the API of
query_tree_walker to support an additional flag bit, but the changes
are otherwise quite small.

Patches submitted by Kris Jurka (jurka@ejurka.com) for the following bugs:
  - Properly drop tables in jdbc regression tests with cascade for 7.3
  - problem with Statement.execute() and executeUpdate() not clearing binds
  - problem with ResultSet not correctly handling default encoding
  - changes to correctly support show transaction isolation level in 7.3
  - changed DatabaseMetaDataTest to handle differences in FK names in 7.3
  - better fix for dynamically checking server NAME data length
  (With the fixes above the jdbc regression tests pass on jdbc2 and jdbc3
   against both a 7.2 and 7.3 server)
Patchs submitted by David Wall (d.wall@computer.org):
  - problem with getBlob when largeobject oid is null
  - improvements to BlobOutputStream
Patch submitted by Haris Peco (snpe@snpe.co.yu):
  - problem with callable statement not supporting prepared statement methods

 Modified Files:
  jdbc/org/postgresql/Driver.java.in
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1DatabaseMetaData.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSet.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/jdbc2/Jdbc2ResultSet.java
  jdbc/org/postgresql/jdbc3/Jdbc3ResultSet.java
  jdbc/org/postgresql/largeobject/BlobOutputStream.java
  jdbc/org/postgresql/largeobject/LargeObject.java
  jdbc/org/postgresql/test/TestUtil.java
  jdbc/org/postgresql/test/jdbc2/DatabaseMetaDataTest.java
  jdbc/org/postgresql/test/jdbc2/UpdateableResultTest.java
  jdbc/org/postgresql/test/jdbc2/optional/BaseDataSourceTest.java
  jdbc/org/postgresql/test/jdbc2/optional/ConnectionPoolTest.java
  jdbc/org/postgresql/test/jdbc2/optional/SimpleDataSourceTest.java

Add mention of config.log for configure failure debugging.

Fix atan() description.

Bruno Wolff III

Fix portability problem (size_t != int).

Powerup defaults for LC_foo GUC variables should match what main.c does.

Use different sed separator for configure arguments.

Remove more references to pgaccess as a build target in docs.

Fixed DatabaseMetaData to correctly handle NAME size of 64
Fixed Statement to correctly DEALLOCATE any prepared statements

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

Changes to documentation and the regression tests for the default
NAMEDATALEN of 64.

Kris Jurka

Done as far as possible:

< * Add documentation to lock shared memory into RAM for each OS, if possible

Remove pgaccess from docs; not shipping it anymore.

Allow pg_dumpall to work with previous releases again.  Don't pass the -c
option down to pg_dump, where it's useless, and clarify the meaning of -c
in the documentation.

Fix help output.

Make sure the pg_dump tar archiver can handle members larger than 2 GB, but
does not create members larger than allowed by the tar format.  Also, fix
the generation of the tar header to conform to POSIX.

Adjust the tarball splitting scheme to the new reality.

Run pgjindent for Java folks.

Fix another typo.

Fix typo.

Update for new IP.

Add JAVA_HOME test for Ant.

Fix printf() quote handling and improper exit(), per Tom.

Remove heap_mark4update from AlterTableCreateToastTable.  This has
never been the correct procedure for locking a relation, and the
recently-found ALTER TABLE bug with adding a constraint and a toast
table in the same command shows why it's a bad idea.

Fix some operator-precedence problems.  New constructs IS DISTINCT FRM
and IS [NOT] OF were not being parsed consistently with other IS forms.
Also, make the world a little safer for functions named LEFT, RIGHT, etc.

Fix compile error.

Missed mention of PGPASSWORDFILE.

Update HISTORY for PGPASSWORDFILE change.

Remove PGPASSWORDFILE and default to always trying $HOME/.pgpass.

Cleanup up memory allocation for $HOME in related psql places.

Update mention of $HOME/.pgpass in the docs;  add mention in pg_dumpall.

Update:

> * -Add ~/.pgpass to store passwords with user/host/password combinations

Fill in section on table modification.

Fix compile warning.

Fix bit-rotted reference to GetUserName() ...
it's GetUserNameFromId() now.

Fix breakage introduced by careless snprintf patching.

Fix unsafe macro definitions (which were producing incorrect code,
leading to compile warnings).

Remove compile warnings, ensure consistent build environment for
largefile usage.

Fix compile warning.

findoidjoins and tsearch are not broken anymore.

Fix compile errors.

Improve opr_sanity regression test to check oprltcmpop and opgtcmpop
mergejoin links.

Update oidjoins regression test for 7.3 catalogs.

findoidjoins is updated for schemas, does not use libpgeasy.
From Joe Conway.

Seems like a good idea for template1 to contain ANALYZE stats for the
system tables.

Commenting out doesn't work, so move the broken modules out of the list.

autoconf

Assorted fixes for Cygwin:

Eliminate the mysterious games that the Cygwin build plays with the linker
flag variables.  DLLLIBS is gone, use SHLIB_LINK like everyone else.
Detect cygipc in configure, after the linker flags are set up, otherwise
configure might not work at all.

Make sure everything is covered by make clean.

Fix the build of the new conversion procedure modules.

Add new DLLIMPORT markers where required.

Finally, the compiler complains if we use an explicit
-I/usr/local/include, so don't do that.  Curiously, -L/usr/local/lib is
still necessary.

Don't use gethostbyname2().  It's not portable and we don't claim to
support IPv6 anyway.

Fix compilation warning.  (Cygwin has char* as second argument of
recvfrom(), not void*.)

Remove:

< * Make pg_trigger.tgargs refer to columns by number, not name

Update:

> * Make pg_trigger.tgargs refer to columns by number, not name

Done:

> * -Make triggers refer to columns by number, not name

remove pgaccess from GNUmakefile.in if we want the beta to build :)

Remove include of libpq-int.h --- dblink.c should not be (and was not)
depending on libpq internals.

Be careful to include postgres.h *before* any system headers, to ensure
that the right flavors of largefile-related definitions are seen.
Most of these changes are probably unnecessary, but better safe than
sorry.

Guard against send-lots-and-lots-of-data DoS attack from unauthenticated
users, by limiting the length of string we will accept for a password.
Patch by Serguei Mokhov, some editorializing by Tom Lane.

Add change by Neil.

Remove leftovers from subproject removals.  Fixes for Python and Kerberos
configuration.

Fix compile warning.

Disable findoidjoins while it doesn't compile.

Fix includes for plperl: ensure postgres.h is included first,
remove unnecessary inclusions.

Add:

> * Remove Cyrillic recode support

Done:

>       o -Add SHOW command to see locale

Add:

>  o -Add SHOW command to see locale

pgindent run.

Update:

Improvements to /contrib/intarray (Oleg, Teodor Sigaev, Andrey
Oktyabrski)

Add Joe's table functions description.

Update symbols for 7.3.

Update wording for Tom.

Update for 7.3 typedefs.

Update to reflect Tom's suggestions.

Remove sh -x option that snuck in somehow.

Reorder items.

More HISTORY improvements from Joe.

Update HISTORY wording.

Avoid multiple scans of utils/mb/conversion_procs/ subdirectories during
'make install'; there are enough of 'em that this slowed down the make
noticeably.  Ensure that 'all' is the default make target in all these
directories (defaulting to 'make install' is surprising and dangerous
IMHO).  Fix a couple small typos.

Stamp configure.in with 7.3b1, not just 7.3.   Seems Marc does that
usually anyway.

Update based on TODO file contents.

File list cleanup.

Spellcheck of HISTORY.

Update files to be changed.

Brand 7.3.  Ready for beta!

Update new HISTORY file for 7.3!

Refrect changes made by Tom Lane

Refrect the changes to src/test/regress/sql/conversion.sql By Tom.

Add:

> * Gavin Sherry <swm@linuxworld.com.au>

EXTRACT(EPOCH FROM timestamp) gave wrong answers in the int64-timestamp
case for timestamptz input, and differently wrong answers in the float-
timestamp case for timestamp input.

Arrange for the default permissions on a database to allow temp table
creation to world, but disallow temp table creation in template1.  Per
latest round of pghackers discussion.
I did not force initdb, but the permissions lockdown on template1 will
not take effect unless you do one (or manually REVOKE TEMP ON DATABASE template1 FROM public).

conversion test fails if there is an existing user named foo.  Choose a name
somewhat less likely to provoke a conflict.

Remove all traces of multibyte and locale options.  Clean up comments
referring to "multibyte" where it really means character encoding.

Work around mktime() brain damage in recent versions of glibc by using
a series of localtime() calls to determine the local timezone offset
when mktime() fails.  This eliminates regression failures on RHL 7.3,
and should continue to work until it occurs to the glibc boys to break
localtime() as well.  By then I hope we'll have our own timezone code...

Eliminate unnecessary dependency on mktime(), and consequent 'Unable to
convert date to tm' failures, by using DetermineLocalTimeZone() instead.

Workaround for format strings that are concatenated from macros
(INT64_FORMAT), which gettext cannot handle.

AIX shlib fix for building libpq.so from Tomoyuki Niijima

Add:

> * to_char(0,'FM999.99') returns a period, to_char(1,'FM999.99') does not