Small update for the removal of some memory leaks in plpython SGML example.

Nigel J. Andrews

When I removed the cube based stuff from eathdistance I accidentally
left a reference to cube in a comment in the regression test (that also
shows up in the expected output). This doesn't cause any real problem,
but people who read the comment might be confused. Attached is a diff to
remove the reference.

Bruno Wolff III

Cleanup for CLUSTERDB failure:

On Sat, Oct 19, 2002 at 12:11:32AM +0200, Peter Eisentraut wrote:
> $ ./clusterdb
> psql: could not connect to server: No such file or directory
>         Is the server running locally and accepting
>         connections on Unix domain socket "/tmp/.s.PGSQL.5432"?
> psql: could not connect to server: No such file or directory
>         Is the server running locally and accepting
>         connections on Unix domain socket "/tmp/.s.PGSQL.5432"?
> clusterdb: While clustering peter, the following failed:
> $
>
> This could probably handled a little more gracefully.

Yes, sorry.  A patch for this is attached.  Please apply.

Alvaro Herrera

Fix ALTER TABLE ... ADD COLUMN for inheritance cases.

Alvaro Herrera

Remove unnecessary (and inadequate) check of IsTransactionBlock() in
pgstat_vacuum_tabstat().  Assume that caller (namely, VACUUM) has done
the appropriate state checking beforehand.

Avoid using IsTransactionBlock() in DeferredTriggerSetState(); no real
need for this optimization, and it's too easily fooled anyway.

Make CREATE/ALTER/DROP USER/GROUP transaction-safe, or at least pretty
nearly so, by postponing write of flat password file until transaction
commit.

Add DLLIMPORT declarations needed by contrib modules.

Need sys/param.h for endianness macros.

Might need to link with -lcrypt.

Add guards against double inclusion.

Translation updates

Remove obsoleted stuff merged in from branch.

Improve ECPG documentation.

Merged ecpg_big_bison back into HEAD

Update multibyte Japanese document

Since ANY is a reserved word, better suggest that ANY be quoted when
used for the input type of an aggregate.

Fix documented sizes of geometric types.

Fix example of doing a restore.

SET autocommit no longer needed in /contrib because pg_regress.sh does
it automatically now on regression session startup.

Updates from Ian Barwick.

Done:

> * -Move /contrib/retep to gborg.postgresql.org

Completely remove /contrib/retep, with Peter's approval;  now on Source
Forge.

First step to removing /contrib/retep, with Peter Mount's approval.

Remove trailing blanks in cvs.sgml.

Fix potential problem with btbulkdelete deleting an indexscan's current
item, if the page containing the current item is split while the indexscan
is stopped and holds no read-lock on the page.  The current item might
move right onto a page that the indexscan holds no pin on.  In the prior
code this would allow btbulkdelete to reach and possibly delete the item,
causing 'my bits moved right off the end of the world!' when the indexscan
finally resumes.  Fix by chaining read-locks to the right during
_bt_restscan and requiring btbulkdelete to LockBufferForCleanup on every
page it scans, not only those with deletable items.  Per my pghackers
message of 25-May-02.  (Too bad no one could think of a better way.)

Fix core dump recently introduced into pgbench.

Returns exit status 1 if invalid encoding name or
encoding number is given.

Update description of numeric constants to match 7.3 reality.
Miscellaneous other copy-editing.

Update char/varchar()/text/bytea to mention they have similar performance
characteristics.

Applied patch from Teofilis Martisius to improve performance.
Also removed some unused files and fixed the which needed a small change
after the previous patch to build.xml.

 Modified Files:
  jdbc/Makefile jdbc/org/postgresql/core/Encoding.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
 Removed Files:
  jdbc/utils/CheckVersion.java jdbc/utils/buildDriver
  jdbc/utils/changelog.pl

Add:

> * Make a transaction-safe TRUNCATE

Update Japanese FAQ, from Jun Kuwamura.

Add:

>  o Add GUC parameter to control the maximum number of rewrite cycles

Add warning that autocommit=off is not well-supported yet.

Move BeOS and QNX4 to unsupported-platform list, until someone cares
to expend the effort to update them for new semaphore code.

Rule rewriter was doing the wrong thing with conditional INSTEAD rules
whose conditions might yield NULL.  The negated qual to attach to the
original query is properly 'x IS NOT TRUE', not 'NOT x'.  This fix
produces correct behavior, but we may be taking a performance hit because
the planner is much stupider about IS NOT TRUE than it is about NOT
clauses.  Future TODO: teach prepqual, other parts of planner how to
cope with BooleanTest clauses more effectively.

Disallow aggregate functions in rule WHERE clauses.  Per gripe from
Fritz Lehmann-Grube back in January.

Applied patch submitted by Mike Beachy to give a better error message if
configure hasn't been run before trying to build.
Also cleaned up the README file and removed some obsolete files.

 Modified Files:
  jdbc/README jdbc/build.xml
 Removed Files:
  jdbc/CHANGELOG jdbc/Implementation jdbc/jdbc.jpx

Fix inconsistent formatting.

Fix incomplete definition of ALTER TABLE ADD/DROP CONSTRAINT syntax.
Add some verbiage about recent tweaks to behavior of ADD and DROP
COLUMN when there are descendant tables.

Fix within-function memory leaks in the various PLs' interfaces to
SPI_prepare: they all save the prepared plan into topCxt, and so the
procCxt copy that's actually returned by SPI_prepare ought to be freed.
Diagnosis and plpython fix by Nigel Andrews, followup for other PLs
by Tom Lane.

Applied patch submitted by Kris Jurka to result in a better error message
under some circumstances and handle negative money values better.

 Modified Files:
  jdbc/org/postgresql/jdbc1/AbstractJdbc1ResultSet.java

Fix pltcl self-test for recent changes to elog.

Backed out part of the change from 1.6.  The attempt to support int8 binds
in such a way that indexes on int8 columns would be used (by quoting the value)
caused other problems.  Will need to wait for the backend to properly fix
the root problem.

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

Fix case where a function in FROM returns a scalar type, but is
referred to with whole-tuple syntax.

Make psql's \df display functions that return sets as having return type
'setof something'; formerly you could not tell at all that the function
returns set.

Invert logic in pg_exec_query_string() so that we set a snapshot for
all utility statement types *except* a short list, per discussion a few
days ago.  Add missing SetQuerySnapshot calls in VACUUM and REINDEX,
and guard against calling REINDEX DATABASE from a function (has same
problem as VACUUM).

Fix rewrite code so that rules are in fact executed in order by name,
rather than being reordered according to INSTEAD attribute for
implementation convenience.
Also, increase compiled-in recursion depth limit from 10 to 100 rewrite
cycles.  10 seems pretty marginal for situations where multiple rules
exist for the same query.  There was a complaint about this recently,
so I'm going to bump it up.  (Perhaps we should make the limit a GUC
parameter, but that's too close to being a new feature to do in beta.)

Back out Alvaro's patch until regression tests pass.

Fix range-query estimation to not double-exclude NULLs, per gripe from
Ray Ontko 28-June-02.  Also, fix prefix_selectivity for NAME lefthand
variables (it was bogusly assuming binary compatibility), and adjust
make_greater_string() to not call pg_mbcliplen() with invalid multibyte
data (this last per bug report that I can't find at the moment, but it
was in July '02).

Add:

> * Move /contrib/retep to gborg.postgresql.org

Fix compile failure caused by new patch.

Remove mention of utils/getopt.c.  Not used anymore.

It includes
-Support for mirroring tables in different Schema's
-Improved documentation for compiling with 7.1.x and 7.2.x
-Fixes a buffer overrun bug.

Steven Singer

> Huh, I don't know where I got the idea you were (or someone else was?)
> in the position that attislocal should be reset.  I'll clean everything
> up and submit the patch I had originally made.

All right, this is it.  This patch merely checks if child tables have
the column.  If atttypid and atttypmod are the same, the attributes'
attinhcount is incremented; else the operation is aborted.  If child
tables don't have the column, recursively add it.

attislocal is not touched in any case.

Alvaro Herrera

This patch adds some missing functions for float8 math operations,
specifically ceil(), floor(), and sign(). There may be other functions
that need to be added, but this is a start. I've included some simple
regression tests.

Neil Conway

Make regression tests safe for autocommit = 'off'.

Add missing #include <errno.h>.

Fix psql's \copy to accept table names containing schemas, as well as
a column list.  Bring its parsing of quoted names and quoted strings
somewhat up to speed --- I believe it now handles all non-error cases
the same way the backend would, but weird boundary conditions are not
necessarily done the same way.

Improve formatting of --help output.

Fix breakage that had crept into setlocale() usage: once again we've
been bit by the fact that the locale functions return pointers to
modifiable variables.  I added some comments that might help us avoid
the mistake in future.

Make 'dummy' declarations in header files be 'extern int no_such_variable'
instead of 'extern int errno'; the latter is unsafe according to the
ANSI C standard, as well as in practice on some platforms.

Un-break createlang build.

Clean up for dblink autocommit OFF handling.

Update /contrib for "autocommit TO 'on'".

Create objects in public schema.

Make spacing/capitalization consistent.

Remove transaction block use for object creation.

Remove unneeded function GRANTs.

Improve discussion of FOR UPDATE.

Update for tables returning function, item 5.3, from Joe Conway.

Fix in updateable result sets to handle binding null values correctly
Patch submitted by Kris Jurka (applied with some modifications)

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

Fix free-slot search in PgSetResultId so it actually works.

Improve formatting of error message.

Fixed support in jdbc for 7.3 server autocommit.  With these changes the
jdbc regression tests pass for both autocommit on and autocommit off

 Modified Files:
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/test/jdbc2/ConnectionTest.java

Add:

>  o Allow copy to understand \x as hex

Put the disk information all in one chapter.

Make pg_dump save for autocommit = off.

Fix from Joe on timeout code.

Improve appearance of SET command.

Fix script to handle autocommit = 'off' by prepending autocommit 'on' to
the start of the psql commandline.  This is better than adding BEGIN/END
because it handles multiple queries well, and allows the return code for
psql to return the proper value.

Fix connection_timeout to use time() and handle timeout == 1.

Code cleanup.

Remove:

< * Rename oid2name to relfilenode2name and install by default
< * Move /contrib/oid2name to /bin, rename to relfilenode2name

Add:

> * Move /contrib/oid2name to /bin, rename to relfilenode2name

Fix unintentional breakage of COPY TO/FROM stdin.  Mea culpa.

Fix Linux dynloader code for pre-HAVE_DLOPEN systems, which evidently
are still in use out there.  Per report from Brendan LeFebvre.

Fix psql to cope with autocommit off, at least during startup.
Behavior of backslash commands (especially for large objects)
may still require some thought.

Fix libpq startup code to work correctly in autocommit off mode.
In passing, fix breakage for case where PGCLIENTENCODING is set in
environment.

Make SPI's execution of querystrings follow the rules agreed to for
command status at the interactive level.  SPI_processed, etc are set
in the same way as the returned command status would have been set if
the same querystring were issued interactively.  Per gripe from
Michael Paesold 25-Sep-02.

None.

Adjust handling of command status strings in the presence of rules,
as per recent pghackers discussions.  initdb forced due to change in
fields of stored Query nodes.

Add missing entries for bit<->int4 and bit<->int8 conversions.

Translation updates

Mention that resetting the timeout may be wrong on select retry.

Restore ptmp_timeout for cases where no timeout is passed.

libpq connection_timeout doesn't do subsecond timing, so make the code
clear on that point.

Arrange to copy relcache's trigdesc structure at the start of any
query that uses it.  This ensures that triggers will be applied consistently
throughout a query even if someone commits changes to the relation's
pg_class.reltriggers field meanwhile.  Per crash report from Laurette Cisneros.
While at it, simplify memory management in relcache.c, which no longer
needs the old hack to try to keep trigger info in the same place over
a relcache entry rebuild.  (Should try to fix rd_att and rewrite-rule
access similarly, someday.)  And make RelationBuildTriggers simpler and
more robust by making it build the trigdesc in working memory and then
CopyTriggerDesc() into cache memory.

- Link the entries in the table to the catalog heading
- Wrap them in the <database class="table"> tags, since thats what they
are (no markup rules for this, so it inherits from parent -- no style
change)
- Mention that pg_database, pg_shadow, and pg_group are global, and the
rest are local to the specific DB. (I believe this is correct).

> Works for me, though I suppose we could explain what the exceptions are
> like in general terms.  Perhaps something like
>
> 'Most system catalogs are copied from the template database during
> database creation, and are thereafter database-specific.  A few
> catalogs are physically shared across all databases in an installation;
> these are marked in the descriptions of the individual catalogs.'

Ok, new patch.

Rod Taylor

This trivial patch fixes a bunch of spelling mistakes in the
contrib/dbmirror/README.dbmirror doc file.

Neil Conway

As Niel so nicely pointed out this morning, the output of EXPLAIN
ANALYZE is not quite clear when branches of the query are never
executed. So this tiny patch fixes that.

The patch is attached and can also be found at:
http://svana.org/kleptog/pgsql/pgsql-explain.patch

Martijn van Oosterhout

I have attached two patches as per:

1) pltcl:
Add SPI_freetuptable() calls to avoid memory leaks (Me + Neil Conway)
Change sprintf()s to snprintf()s (Neil Conway)
Remove header files included elsewhere (Neil Conway)

2)plpython:
Add SPI_freetuptable() calls to avoid memory leaks
Cosemtic change to remove a compiler warning

Notes:

I have tested pltcl.c for
 a) the original leak problem reported for the repeated call of spi_exec
in a TCL fragment
and
 b) the subsequent report resulting from the use of spi_exec -array
in a TCL
fragment.

The plpython.c patch is exactly the same as that applied to make
revision 1.23,
the plpython_schema.sql and feature.expected sections of the patch are
also the
same as last submited, applied and subsequently reversed out. It remains
untested by me (other than via make check). However, this should be safe
provided PyString_FromString() _copies_ the given string to make a
PyObject.

Nigel J. Andrews

Improvements to Ian Barwick patch.

FAQ updates from Ian Barwick.