This trivial patch fixes a small memory leak in pg_dump.

Neil Conway

please find attached patch to current CVS ( contrib/ltree )

Changes:

July 31, 2002
   Now works on 64-bit platforms.
   Added function lca - lowest common ancestor
   Version for 7.2 is distributed as separate package -
   http://www.sai.msu.su/~megera/postgres/gist/ltree/ltree-7.2.tar.gz

Oleg Bartunov

This patch fixes a "multi-line string literal" warning encountered
when compiling psql with GCC 3.1.

Neil Conway

The attached patch implements START TRANSACTION, per SQL99. The
functionality of the command is basically identical to that of
BEGIN; it just accepts a few extra options (only one of which
PostgreSQL currently implements), and is standards-compliant.
The patch includes a simple regression test and documentation.

[ Regression tests removed, per Peter.]

Neil Conway

This patch fixes in intermittent failure in the regression tests:
there was a race condition between the "alter_table" and "rules"
regression tests. Depending on scheduling, sometimes an ALTER
TABLE command would operate on a relation created by the "rules"
tests, leading to unexpected results.

Neil Conway

This patch adds support for inet[] arrays to the /contrib/array module.

Neil Conway

Back out pl/pgsql quotation fix.  Has problems.

*** empty log message ***

*** empty log message ***

    I send a simple patch for PL/pgSQL parser which allow now to use
whitespaces in identifers of any kind(table names,attribute
names,variables ...) in Pl/pgSQL procedural language.Explicit definition
of bug can be found in Re: [HACKERS] Bug of PL/pgSQL parser

TODO item completed:

        o -Fix PL/PgSQL to handle quoted mixed-case identifiers

eutm

Here is a doc patch for the SHOW X changes and new config-settings
functions. If there are no objections, please apply.

Joe Conway

When compiling with --enable-cassert, check for reference count leaks
in the relcache.  It's rather silly that we have reference count leak
checks in bufmgr and in catcache, but not in relcache which will normally
have many fewer entries.  Chris K-L would have caught at least one bug
in his recent DROP patch if he'd had this.

RemoveAttrDefaultById() neglected to obtain exclusive lock on the
relation being modified.  In most paths of control we'd already have
such a lock, but if we were dropping the default due to a cascaded
delete of some function it depended on, maybe not.

Remove unused drop TODO.detail.

No need for TODO.detail link anymore:

>  o -Add ALTER TABLE DROP COLUMN feature

Done:

>  o -Add ALTER TABLE DROP COLUMN feature [drop]

ALTER TABLE DROP COLUMN works.  Patch by Christopher Kings-Lynne,
code review by Tom Lane.  Remaining issues: functions that take or
return tuple types are likely to break if one drops (or adds!)
a column in the table defining the type.  Need to think about what
to do here.

Along the way: some code review for recent COPY changes; mark system
columns attnotnull = true where appropriate, per discussion a month ago.

* -Remove LockMethodTable.prio field, not used (Bruce)

1) Improve the handling of the queries like (select ..) union (select ..)
   whose first non-space character is '('.
2) Handle Insert .. () VALUES ().

Instead of having a configure-time DEFAULT_ATTSTATTARGET, store -1 in
attstattarget to indicate 'use the default'.  The default is now a GUC
variable default_statistics_target, and so may be changed on the fly.  Along
the way we gain the ability to have pg_dump dump the per-column statistics
target when it's not the default.  Patch by Neil Conway, with some kibitzing
from Tom Lane.

Done:

> * -Remove LockMethodTable.prio field, not used (Bruce)

Move backend_pid to functions/misc.

Rename pid function to backend_pid().

Add function to access backend pid, pg_stat_get_backend_mypid.

Done:

> * -Add getpid() function to backend

Teach pg_dump to dump user-defined operator classes.  For the moment,
this only works against 7.3 or later databases; the pushups required
to do it without regprocedure/regtype/etc seem more trouble than they're
worth, considering that existing users aren't expecting pg_dump support
for this.

Updated:

<  o ALTER TABLE ADD COLUMN column SET DEFAULT should fill existing
>  o ALTER TABLE ADD COLUMN column DEFAULT should fill existing

Clarification from Tom:

<  o ALTER TABLE ALTER COLUMN column SET DEFAULT should fill existing
>  o ALTER TABLE ADD COLUMN column SET DEFAULT should fill existing

Would it be wise to have the plsql.sgml file renamed to plpgsql.sgml?
All of the internal tags are of the latter.

The other thing I noticed is that most of the quick examples in the file
use a para and synopsis.  Is there a reason we're not using <example/> ?

Rod Taylor

Added for Neil Conway:

> * Use CHECK constraints to improve optimizer decisions

If we're cleaning out _deadcode, might as well zap this one too.

Replace ad-hoc insertions into pg_opclass and friends with CREATE
OPERATOR CLASS commands.

Remove last recipe file.

Ensure that src/tutorial gets cleaned by top-level make clean.

Replace ad-hoc insertions into pg_opclass and friends with CREATE
OPERATOR CLASS commands.  Further tweaking of documentation for same.

Use Max/Min macros, not MAX/MIN, to eliminate portability issues.

Done:
>  o -Generate failure on short COPY lines rather than pad NULLs

The attached patch removes the last remnants of support for
'tioga recipes', whatever those are -- Peter E. killed most
of it a couple days ago, but this patch removes the rest. Most
of it was #ifdef'ed out anyway.

Neil Conway

IMPROVED VERSION APPLIED:

The attached patch completes the following TODO item:

    * Generate failure on short COPY lines rather than pad NULLs

I also restructed a lot of the existing COPY code, did some code
review on the column list patch sent in by Brent Verner a little
while ago, and added some regression tests. I also added an
explicit check (and resultant error) for extra data before
the end-of-line.

Neil Conway

Add ltree data type to contrib, from Teodor Sigaev and Oleg Bartunov.

Added support for schemas and quotes in tab-complete.c, as well as
a few other things:

* Made all references to the pg_* tables absolute, by specifying
  the pg_catalog schema.

* Added SCHEMA as a create/delete completion option.

* Added SCHEMA completion as: SELECT nspname FROM
pg_catalog.pg_namespace
  WHERE substr(nspname,1,%d)='%s'

* Added completion of "INSERT INTO <table> (" with attribute names.

* Added completion of "INSERT INTO <table> (attribs)" with
  VALUES or SELECT

* Added limited locking completion: only for one table:
  "LOCK" and "LOCK TABLE" now both get a completion list of tables
  Complete with "IN" for LOCK [TABLE] <table>
  Complete LOCK [TABLE] <table> IN with a lock mode

* Added a very simple WHERE finisher that uses the previous word
  as a table lookup for attributes.

* Added quote support when parsing "previous words". In other words,
  hitting tab after INSERT INTO "foo bar baby"
  now does the right thing and recognizes "foo bar baby" as one word.

Letting tab-complete quote things that should be quoted seems to be
temporarily ifdef'ed out due to readline compatibility problems.
Can anyone elaborate on this?

Greg Sabino Mullane

    This should fix a bug where a row that was updated or
deleted that had another row inserted/updated to its old
value during the same statement or other statements before the
integrity check for noaction would incorrectly error.  This
could happen in deferred constraints or due to triggers or
functions. It's effectively a reworking of the previous patch that
did a not exists to instead do a separate check.

Stephan Szabo

Since we're depending on %option noyywrap in the main scanner now,
we may as well use it in all our flex files.  Make all the flex files
have a consistent set of options.

Add tablefunc to contrib makefile.

As mentioned above, here is my contrib/tablefunc patch. It includes
three functions which exercise the tablefunc API.

show_all_settings()
   - returns the same information as SHOW ALL, but as a query result

normal_rand(int numvals, float8 mean, float8 stddev, int seed)
   - returns a set of normally distributed float8 values
   - This routine implements Algorithm P (Polar method for normal
     deviates) from Knuth's _The_Art_of_Computer_Programming_, Volume 2,
     3rd ed., pages 122-126. Knuth cites his source as "The polar
     method", G. E. P. Box, M. E. Muller, and G. Marsaglia,
     _Annals_Math,_Stat._ 29 (1958), 610-611.

crosstabN(text sql)
   - returns a set of row_name plus N category value columns
   - crosstab2(), crosstab3(), and crosstab4() are defined for you,
     but you can create additional crosstab functions per directions
     in the README.

Joe Conway

Here are two patches. The guc_and_tablefunc patch addresses the two
changes mentioned above, and also adds a new function to the tablefunc
API. The tablefunc API change adds the following function:

* Oid foidGetTypeId(Oid foid) - Get a function's typeid given the
* function Oid. Use this together with TypeGetTupleDesc() to get a
* TupleDesc which is derived from the function's declared return type.

In the next post I'll send the contrib/tablefunc patch, which
illustrates the usage of this new function. Also attached is a doc patch
for this change. The doc patch also adds a function that I failed to
document previously.

Joe Conway

This patch fixes one serious bug (runaway INSERT) and a few rare (and
hard to reproduce) error conditions.

Manfred Koizar

Update contributions link.

added tests for crossReferences

changed test user back to test

Added DataSource code and tests submitted by Aaron Mulder

changes for new Datasource

Move alarm timers to proper location.

Rewrite xindex.sgml for CREATE OPERATOR CLASS.  catalogs.sgml finally
contains descriptions of every single system table.  Update 'complex'
tutorial example too.

Make statement_timeout apply to entire query string, not per statement.

--enable-syslog no longer required to use syslog.

Move item:

< * Allow logging of query durations

Update FAQ.

Fix a bug about automatic client_encoding setting.

Update FAQ.

Centralize code for interpreting schema references, which had gotten
copied more places than I first thought it would.  This fixes a bug:
a couple of these places were neglecting to enforce USAGE access on
explicitly-referenced schemas.

Use aclcheck_error() in place of ad-hoc permissions complaints.

Implement CREATE/DROP OPERATOR CLASS.  Work still remains: need more
documentation (xindex.sgml should be rewritten), need to teach pg_dump
about it, need to update contrib modules that currently build pg_opclass
entries by hand.  Original patch by Bill Studenmund, grammar adjustments
and general update for 7.3 by Tom Lane.

Fix from Neil Conway:

<  o ALTER TABLE ADD COLUMN column SET DEFAULT should fill existing
>  o ALTER TABLE ALTER COLUMN column SET DEFAULT should fill existing

Fix minor coding error: don't ReleaseSysCache until after last use of
cache entry.

Fix typo. Remove #ifdef MULTIBYTE

Structure reference pages consistently.  Document that structure.
Add information about environment variables.

Clean documentation subtree during make clean.

Assemble portability modules into libpgport library.
Some makefile simplifications.

Fix a bug about the handling of CX parameter of the connection string
reported by Sergey Smirnov.

Fouth (and final) phase of restructuring to add jdbc3 support.
 Modified Files:
  jdbc/org/postgresql/Driver.java.in
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc1/Jdbc1Connection.java
  jdbc/org/postgresql/jdbc1/Jdbc1ResultSet.java
  jdbc/org/postgresql/jdbc2/Jdbc2Connection.java
  jdbc/org/postgresql/jdbc2/Jdbc2ResultSet.java
 Added Files:
  jdbc/org/postgresql/jdbc1/AbstractJdbc1DatabaseMetaData.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1ResultSetMetaData.java
  jdbc/org/postgresql/jdbc1/Jdbc1DatabaseMetaData.java
  jdbc/org/postgresql/jdbc1/Jdbc1ResultSetMetaData.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2DatabaseMetaData.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSetMetaData.java
  jdbc/org/postgresql/jdbc2/Jdbc2DatabaseMetaData.java
  jdbc/org/postgresql/jdbc2/Jdbc2ResultSetMetaData.java
 Removed Files:
  jdbc/org/postgresql/jdbc1/DatabaseMetaData.java
  jdbc/org/postgresql/jdbc1/ResultSetMetaData.java
  jdbc/org/postgresql/jdbc2/DatabaseMetaData.java
  jdbc/org/postgresql/jdbc2/ResultSetMetaData.java

Third phase of restructuring to add jdbc3 support.
 Modified Files:
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1ResultSet.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc1/DatabaseMetaData.java
  jdbc/org/postgresql/jdbc1/Jdbc1Connection.java
  jdbc/org/postgresql/jdbc1/Jdbc1ResultSet.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSet.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/jdbc2/Array.java
  jdbc/org/postgresql/jdbc2/DatabaseMetaData.java
  jdbc/org/postgresql/jdbc2/Jdbc2Connection.java
  jdbc/org/postgresql/jdbc2/Jdbc2ResultSet.java
 Added Files:
  jdbc/org/postgresql/jdbc1/Jdbc1CallableStatement.java
  jdbc/org/postgresql/jdbc2/Jdbc2CallableStatement.java
 Removed Files:
  jdbc/org/postgresql/jdbc1/CallableStatement.java
  jdbc/org/postgresql/jdbc2/CallableStatement.java
  jdbc/org/postgresql/jdbc2/UpdateableResultSet.java

Remove extra comma.

Implement DROP CONVERSION
Add regression test

Second phase of restructuring to add jdbc3 support.

Remove _deadcode.

Remove unused system table columns:

pg_language.lancompiler
pg_operator.oprprec
pg_operator.oprisleft
pg_proc.proimplicit
pg_proc.probyte_pct
pg_proc.properbyte_cpu
pg_proc.propercall_cpu
pg_proc.prooutin_ratio
pg_shadow.usetrace
pg_type.typprtlen
pg_type.typreceive
pg_type.typsend

Attempts to use the obsoleted attributes of pg_operator or pg_proc
in the CREATE commands will be greeted by a warning.  For pg_type,
there is no warning (yet) because pg_dump scripts still contain these
attributes.

Also remove new but already obsolete spellings
isVolatile, isStable, isImmutable in WITH clause.  (Use new syntax
instead.)

Add mention of copyright year update.

Done:

> * -Add Intimate Shared Memory(ISM) for Solaris

Add Japanese README explaining how to add new conversion.
English README will come soon...

Add pg_conversion system catalog. Update description for multibyte support.

Done:

> * -Allow UPDATE/DELETE on inherited table

Initial restructuring to add jdbc3 support.  There was a significant amount
of duplicated code between the jdbc1 and jdbc2.  This checkin restructures
the code so that the duplication is removed so that the jdbc3 support
can be added without adding yet another copy of everything.  Also many
classes were renamed to avoid confusion with multiple different objects
having the same name.  The timestamp tests were also updated to add support
for testing timestamp without time zone in addition to timestamp with time zone

 Modified Files:
  jdbc/Makefile jdbc/build.xml jdbc/example/ImageViewer.java
  jdbc/example/basic.java jdbc/example/blobtest.java
  jdbc/example/threadsafe.java
  jdbc/org/postgresql/Driver.java.in
  jdbc/org/postgresql/Field.java
  jdbc/org/postgresql/core/QueryExecutor.java
  jdbc/org/postgresql/fastpath/Fastpath.java
  jdbc/org/postgresql/jdbc1/CallableStatement.java
  jdbc/org/postgresql/jdbc1/DatabaseMetaData.java
  jdbc/org/postgresql/jdbc1/PreparedStatement.java
  jdbc/org/postgresql/jdbc2/Array.java
  jdbc/org/postgresql/jdbc2/CallableStatement.java
  jdbc/org/postgresql/jdbc2/DatabaseMetaData.java
  jdbc/org/postgresql/jdbc2/PreparedStatement.java
  jdbc/org/postgresql/jdbc2/UpdateableResultSet.java
  jdbc/org/postgresql/largeobject/LargeObjectManager.java
  jdbc/org/postgresql/largeobject/PGblob.java
  jdbc/org/postgresql/largeobject/PGclob.java
  jdbc/org/postgresql/test/jdbc2/BlobTest.java
  jdbc/org/postgresql/test/jdbc2/ConnectionTest.java
  jdbc/org/postgresql/test/jdbc2/DatabaseMetaDataTest.java
  jdbc/org/postgresql/test/jdbc2/TimestampTest.java
  jdbc/org/postgresql/test/jdbc2/UpdateableResultTest.java
  jdbc/org/postgresql/util/Serialize.java
 Added Files:
  jdbc/org/postgresql/PGConnection.java
  jdbc/org/postgresql/PGStatement.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1ResultSet.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Statement.java
  jdbc/org/postgresql/jdbc1/Jdbc1Connection.java
  jdbc/org/postgresql/jdbc1/Jdbc1ResultSet.java
  jdbc/org/postgresql/jdbc1/Jdbc1Statement.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Connection.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2ResultSet.java
  jdbc/org/postgresql/jdbc2/AbstractJdbc2Statement.java
  jdbc/org/postgresql/jdbc2/Jdbc2Connection.java
  jdbc/org/postgresql/jdbc2/Jdbc2ResultSet.java
  jdbc/org/postgresql/jdbc2/Jdbc2Statement.java
 Removed Files:
  jdbc/org/postgresql/Connection.java
  jdbc/org/postgresql/ResultSet.java
  jdbc/org/postgresql/Statement.java
  jdbc/org/postgresql/jdbc1/Connection.java
  jdbc/org/postgresql/jdbc1/ResultSet.java
  jdbc/org/postgresql/jdbc1/Statement.java
  jdbc/org/postgresql/jdbc2/Connection.java
  jdbc/org/postgresql/jdbc2/ResultSet.java
  jdbc/org/postgresql/jdbc2/Statement.java

Add unique index on pg_cast.oid, and document pg_cast table.

Fix typo.

Add CREATE CONVERSION/DROP CONVERSOION reference manual

Fixed some minor typos.

Re-Committed old file.

Schema TODO.detail file not needed anymore.

Tweak CreateTrigger() so that the OID used in the name of an
RI_ConstraintTrigger is the same OID assigned to the pg_trigger row.
This reduces consumption of OIDs and may ease debugging.

Mark items as done:

> * -Add GUC parameter for DATESTYLE
>  o -Allow specification of column names
>  o -Change syntax to WITH DELIMITER, (keep old syntax around?)
>  o -Remove SET KSQO option now that OR processing is improved (Tom)
>  o -Allow SHOW to output as a query result, like EXPLAIN
> * -Add SQL92 schemas (Tom)

Code review for SHOW output changes; fix horology expected files for
new SHOW output format.

Synced parser yet again.

Michael

Fix regression tests for new SHOW output.

Fix problems caused by code drift of API for without-oids.

The attached patch fixes 2 trivial warnings generated by bison 1.35,
as a result of Peter's recent CREATE CAST changes.

Neil Conway

I can't remember who said they were working on schema related psql
changes, but I kept finding myself wishing I could see what schema a
table or view exists in when I use \dt, \dv, etc. So, here is a patch
which does just that.

It sorts on "Schema" first, and "Name" second.

It also changes the test for system objects to key off the namespace
name starting with 'pg_' instead of the object name.

Sample output:

test=# create schema testschema;
CREATE SCHEMA
test=# create view testschema.ts_view as select 1;
CREATE VIEW
test=# \dv
                  List of relations
         Name        |   Schema   | Type |  Owner
--------------------+------------+------+----------
  __testpassbyval    | public     | view | postgres
  fooview            | public     | view | postgres
  master_pg_proc     | public     | view | postgres
  rmt_pg_proc        | public     | view | postgres
  vw_dblink_get_pkey | public     | view | postgres
  vw_dblink_replace  | public     | view | postgres
  ts_view            | testschema | view | postgres
(7 rows)

Joe Conway

> 2. This patch includes the same Table Function API fixes that I
>    submitted on July 9:
>
>    http://archives.postgresql.org/pgsql-patches/2002-07/msg00056.php
>
>    Please disregard that one *if* this one is applied. If this one is
>    rejected please go ahead with the July 9th patch.

The July 9th Table Function API patch mentioned above is now in CVS, so
here is an updated version of the guc patch which should apply cleanly
against CVS tip.

Joe Conway

Hello, i noticed that win32 native stopped working/compiling after the SSL merge
.
So i took the opportunity to fix some stuff:

1. Made the thing compile (typos & needed definitions) with the new pqsecure_* s
tuff, and added fe-secure.c to the win32.mak makefile.
2. Fixed some MULTIBYTE compile errors (when building without MB support).
3. Made it do that you can build with debug info: "nmake -f win32.mak DEBUG=1".
4. Misc small compiler speedup changes.

The resulting .dll has been tested in production, and everything seems ok.
I CC:ed -hackers because i'm not sure about two things:

1. In libpq-int.h I typedef ssize_t as an int because Visual C (v6.0)
doesn't de fine ssize_t. Is that ok, or is there any standard about what
type should be use d for ssize_t?

2. To keep the .dll api consistent regarding MULTIBYTE I just return -1
in fe-connect.c:PQsetClientEncoding() instead of taking away the whole
function. I wonder if i should do any compares with the
conn->client_encoding and return 0 if not hing would have changed (if so
how do i check that?).

Regards

Magnus Naeslund

Add new configure files for missing-oid patch.