> OK, well as we wait on the fix for the stats system, let me submit my
> patch for pg_autovacuum.  This patch assumes that the stats system will
> be fixed so that all inserts, updates and deletes performed on shared
> tables reguardless of what database those commands were executed from,
> will show up in the stats shown in each database.

I had to make a further change to this to take quotes off the 'last
ANALYZE' in order for it to not overquote the relation name, so
there's a _little_ work left to get it to play well.

I have deployed it onto several boxes that should be doing some
vacuuming over the weekend, and it is now certainly hitting pg_
tables.

I would like to present a CVS-oriented patch; unfortunately, I had to
change the indentation patterns when editing some of it :-(.  The
following _may_ be good; not sure...

Matthew T. O'Connor
Christopher Browne

Attempt threading in this order:

 *      use non-*_r function names if they are all thread-safe
 *          (NEED_REENTRANT_FUNCS=no)
 *      use *_r functions if they exist (configure test)
 *      do our own locking and copying of non-threadsafe functions

New to this patch is the last option.

More SQLState updates from Kim Ho at Redhat.
Also a patch from Kris Jurka to correctly report SQLState support.

 Modified Files:
  jdbc/org/postgresql/Driver.java.in
  jdbc/org/postgresql/core/QueryExecutor.java
  jdbc/org/postgresql/fastpath/Fastpath.java
  jdbc/org/postgresql/geometric/PGbox.java
  jdbc/org/postgresql/geometric/PGcircle.java
  jdbc/org/postgresql/geometric/PGline.java
  jdbc/org/postgresql/geometric/PGlseg.java
  jdbc/org/postgresql/geometric/PGpath.java
  jdbc/org/postgresql/geometric/PGpoint.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1ResultSet.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1ResultSetMetaData.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSet.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSetMetaData.java
  jdbc/org/postgresql/jdbc2/Array.java
  jdbc/org/postgresql/jdbc3/AbstractJdbc3DatabaseMetaData.java
  jdbc/org/postgresql/util/PGmoney.java
  jdbc/org/postgresql/util/PSQLState.java

Marginal hacks to make tables format more nicely.

Mop-up for error-message updates in documentation.

Update obsolete examples of error messages; various other minor editing.

Revert to our pre-7.4 behavior of identifying Unix-socket connections in
ps status as '[local]', not as 'localhost' as the code has been doing
recently.  That's too easily confused with TCP loopback connections,
and there is no good reason to change the behavior anyway.

Someone (possibly me) foolishly reduced the response for failing
to create a TCP/IP socket from FATAL to LOG.  This was unwise;
historically we have expected socket conflicts to abort postmaster
startup.  Conflicts on port numbers with another postmaster can only
be detected reliably at the TCP socket level.

Mark FreeBSD as non-thread-safe.

Fix incorrect pfree in to_tsquery_name(), per Nigel Andrews.

Implement compiler #error if spinlock code not found, add configure flag
to bypass the error, --without-spinlocks.

Enable Opteron/Itanium spinlocks.

Remove WIN32_CONSOLE support, at the request of the author.

Fix old mention of exec() in AttachSharedMemoryAndSemaphores comment.

More cleanup of Diagnostics sections.

Small typo.

Message in the other exit from acquire_sample_rows(), as per update
from Mark Kirkwood.  Also show the sample size.

Try to make recently-added analyze log message look something like
the others in style.

This patch fixes a few missed GUC variables that were still upper case,
makes a few more small improvements to runtime.sgml, and makes some SGML
conventions more consistent.

Neil Conway

> It quotes table names for vacuum and analyze, and uppercases the
> keywords for clarity.

Yeah, this is basically what I meant, sorry I didn't get to it quicker.

However, I tested it out a little and the patch you made doesn't work
because it produces commands like:

VACUUM ANALYZE "public.FooBar"

Which doesn't work, so I made my own patch that creates commands like:

VACUUM ANALYZE "public"."FooBar"

This allows for mixed case schema names as well as tables.

Adam, can you please give this a test as you are the person who caught
the bug in the first place.

Thanks,

Matthew T. O'Connor

Consistenly lowercase GUC variable names, in docs and error messages.

This patch makes a few minor improvements to the docs: make the
<varname> conventions more consistent, and improve the ANALYZE ref page.

Neil Conway

This patch makes a number of improvements to the runtime config
documentation.

Neil Conway

Mention that pg_type_is_visible is used for domains as well.

Christopher Kings-Lynne

on my timetravel.c I find a bug: after the
ALTER TABLE mytable drop column last_column_of_table;

the timetravel trigger say on UPDATE/DELETE:

ERROR:  parser: parse error at end of input

Here is the patch for this bug

B?jthe Zolt?n

Here is a patch that removes contrib/array, leaving only the README with
some examples of the new syntax and a reference to the documentation.

Joe Conway.

seemed like a typo in one of the appendix tables

Robert Treat

Fix missed message update, per Alvaro Herrera.

Update row count for \? output, used by pager.

Improve error message for cp or rm failur during create/drop database,
per recent discussions.

Add:

> * Issue NOTICE if foreign key data type doesn't match primary key

Add:

> * Have EXPLAIN ANALYZE highlight poor optimizer estimates

This patch fixes a trivial typo in the CREATE FUNCTION ref page.

Neil Conway

copydir() is supposed to return on failure, not elog(ERROR).  Reduce
ERROR to WARNING so we keep control.

Add HINT if CREATE FUNCTION specifies a valid language, but the language
isn't loaded into the database.

Add quotes to vacuum/analyze for upper/lowercase table names, per
Matthew T. O'Connor.

Some uppercase type definitions were left.

Here is a patch to make PostgreSQL use a full path in DT_SONAME for
UnixWare.

Larry Rosenman

Clean up printability test in dbase code.

Add:

> * Have pg_dump -c clear the database using dependency information

Some "feature not supported" errors are better syntax errors, because the
feature they complain about isn't a feature or cannot be implemented without
definitional changes.

Improve wording of increasing page size.

Remove a bunch of content-free Diagnostics sections, as per previous
discussion.  (Still have some work to do editing the remainder.)

Update max tasble size to 32TB.

Update comments about how locale settings are handled.

Clean up some unused import warnings in the example jdbc code

Fixed a few bugs that came up on pgsql-interfaces.

Additional SQLState work for JDBC - thanks to Kim Ho at Redhat for input on this

 Modified Files:
  jdbc/build.xml jdbc/org/postgresql/core/QueryExecutor.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1DatabaseMetaData.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1ResultSet.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/util/PSQLState.java

- Added Dave patch for Informix handling of numeric/int conversion.
- Changed all new datatypes to lowercase.
- Fixed rounding bug in numerical types.

Fix a couple minor typos.

Update preface.

Use question marks rather than brackets to delimit optional elements in
Tcl synopses.

Fix stylesheet misfeature leading to excessively long cross-reference text
when linking to a different "part".

Remove <body> attributes -- CSS stylesheets should handle that.

Improve bibliography formatting.

Add fast-forward links for more convenient navigation.

Add:

       o Allow ALTER TABLE to change constraint deferrability and actions

"statenebt" ?

Document a bunch of formerly-undocumented date/time operators,
including the SQL-spec OVERLAPS construct.

Opps, can't forget the new file.  This is part of the last commit to apply
the redhat patch for SQLState support.

 Added Files:
  PSQLState.java

This set of changes applies a patch from KHO at redhat to add some SQLState
support to the jdbc driver.
That patch needed some work: it assumed the sqlcode in a server message was
fixed in its position, the patch lost the ability to pass exceptions, and the
patch missed a couple of places where server errors where being received.
In addition to fixing the above, I also added full support for the V3 protocol
error message syntax, I reversed the order of arguments in the PSQLException
constructor to more closely follow the constructors for SQLException, I changed
the new constructors that take PSQLState to take Object for additional
parameters as the old ones did.
Still todo are to add SQLState values to all existing exceptions thrown in the
driver and add support for parsing the V3 protocol format for notices.

 Modified Files:
  jdbc/build.xml jdbc/org/postgresql/Driver.java.in
  jdbc/org/postgresql/errors.properties
  jdbc/org/postgresql/core/Encoding.java
  jdbc/org/postgresql/core/PGStream.java
  jdbc/org/postgresql/core/QueryExecutor.java
  jdbc/org/postgresql/fastpath/Fastpath.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1ResultSet.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSet.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/util/MessageTranslator.java
  jdbc/org/postgresql/util/PSQLException.java

Avoid using s_addr as a local variable name, to work around broken
system headers on Unixware.

Update URL.

Update URL to be momjian.postgresql.org.

autoconf run.

Add $srcdir for Win32 builds, from Peter.

Remove another useless, counterproductive srandom() call.

Remove geqo_random_seed parameter.  Having geqo reset the global random()
sequence every time it's called is bogus --- it interferes with user
control over the seed, and actually decreases randomness overall
(because a seed based on time(NULL) is pretty predictable).  If you really
want a reproducible result from geqo, do 'set seed = 0' before planning
a query.

Remove check on source address of a statistics packet.  Check was broken
by recent IPv6 changes, and since it's redundant with a kernel-level check
anyway, it seems not worth trying to fix it.  Per recent discussions.

Changes for MinGW/WIN32:

        o allow configure to see include/port/win32 include files
        o add matching Win32 accept() prototype
        o allow pg_id to compile with native Win32 API
        o fix invalide mbvalidate() function calls (existing bug)
        o allow /scripts to compile with native Win32 API
        o add win32.c to Win32 compiles (already in *.mak files)

Make the default pg_hba.conf include an entry for ::1 only if configure
set HAVE_IPV6.  Per recent discussions.

load_file() has to remove pre-existing shlibs that match the new file
on either name or inode; otherwise load_external_function() won't do
anything.  At least on Linux, it appears that recompiling a shlib leads
to a new file with a different inode, so the old code failed to detect
a match.

Make all valid uses of the COLLATE clause yield the same result: a syntax
error.  There is no point in providing some kind of forward compatibility
now, because no one can tell what a future implementation will look like.

Rename variables for clarity.

Second try at IPv4-to-v6 mapping code; avoid assuming that the struct
returned by getaddrinfo_all will have enough room for an IPv6 address.

Allow IPv4-format entries in pg_hba.conf to match IPv6 connections
that have IPv4-embedded-in-IPv6 addresses.  Per idea of Andreas Pflug.

Done:

< * Allow psql to do table completion for SELECT * FROM schema_part and
> * -Allow psql to do table completion for SELECT * FROM schema_part and

ifdef out pqGetpwuid() because Win32 doesn't have struct passwd.

Needed for libpq builds, not just for native Win32 port, and probably
for cygwin too.

Add WIN32 URL.

Add missing space.

This fixes three minor typos in hba.c.

--
Alvaro Herrera (<alvherre[a]dcc.uchile.cl>)

Update information about Linux's overcommit memory behavior.

Andrew Dunstan

Add:

> * Maintain a map of recently-expired of pages so vacuum can reclaim
>   free space without a sequential scan

Clean up WIN32_CONSOLE code.

Update as done:

> * -Improve concurrency of hash indexes (Tom)

Enable Win32 to compile libpq again, and enable SSL compiles on that
platform.

Andreas Pflug

Improve prototype casting for thread calls.

Add:

>  o Allow ALTER TABLE ... ALTER CONSTRAINT ... RENAME

Add:

<
> * Print table names with constraint names in error messages, or make constraint
>   names unique within a schema

Reimplement hash index locking algorithms, per my recent proposal to
pghackers.  This fixes the problem recently reported by Markus KrÌutner
(hash bucket split corrupts the state of scans being done concurrently),
and I believe it also fixes all the known problems with deadlocks in
hash index operations.  Hash indexes are still not really ready for prime
time (since they aren't WAL-logged), but this is a step forward.

Avoid consuming unreasonable amounts of memory when GRANT has many
grantees.

Looks like plperl has same bug as pltcl.

Fix some problems with dropped columns in pltcl functions.

Allow non-super users to set log_duration to true, then false, when the
administrator has not turned it on, and fix other PGC_USERLIMIT
variables.

Re-upper case CHECKPOINT_SEGMENTS.

Lowercase CHECKPOINT_SEGMENTS.

Add:

> * Allow to_char to print localized month names (Karel)

Tighten casting of function call pointers for threads.

Add note that a for loop where the lower bound is greater than the upper
bound is valid but does nothing.

suggested by Richard Huxton <dev@archonet.com>

Pass session_authorization to the client and make psql update its prompt
accordingly.

Done:

* -Have standalone backend read postgresql.conf (Tom)

Mark unixware as verified.

Add Unixware as verified.

Update German FAQ.

Ian Barwick

Mark BSD/OS as verified not needing reentrant function names.