Back out commit.

Adjust expected output for new functions.

Add:

> * Allow creation of a libpq-only tarball
>

Seems the runtime.sgml and perform.sgml patches combined are
incompatible.

I believe the attached xref meets the intent of the perform.sgml change.

--
Rod Taylor <rbt@rbt.ca>

Update catversion for ipv6 addition.

Documentation about using CIDR addresses in pg_hba.conf.

Andrew Dunstan

> This change (I'm sure this will wrap poorly -- sorry):
> http://developer.postgresql.org/cvsweb.cgi/pgsql-server/src/include/libpq/pqcomm.h.diff?r1=1.85&r2=1.86
>
> modified SockAddr, but no corresponding change was made here
> (fe-auth.c:612):
>
>   case AUTH_REQ_KRB5:
> #ifdef KRB5
>   if (pg_krb5_sendauth(PQerrormsg, conn->sock, &conn->laddr.in,
>                        &conn->raddr.in,
>                        hostname) != STATUS_OK)
>
> It's not obvious to me what the change ought to be though.

This patch should hopefully fix both kerberos 4 and 5.

Kurt Roeckx

> I've worked with the Pl/Python code in the past and will see about removing
> rexec and making it an untrusted language.  Last time I looked, it didn't
> look particularly difficult.  I've set aside some time next week, so stay
> tuned.

Attached is a patch that removes all of the RExec code from plpython from
the current PostgreSQL CVS.  In addition, plpython needs to be changed to an
untrusted language in createlang.  Please let me know if there are any
problems.

Kevin Jacobs

Attached is a patch that provides *VERY* limited support for multiple
slave
servers.  I haven't tested it very well, so use at your own risk (and I
recommend against using it in production).

Basically, I have a central database server that has 4 summary tables
inside
it replicated to a remote slave (these database tables are for my mail
server
authentication, so these are replicated to another server tuned for many
connections, and so I don't have postgres connections opened straight to
my
back-end database server).

Unfortunately, I also wanted to implement a replication database server
for
hot-backups.  I realized, too late, that the replication process is
pretty
greedy and will try to replicate all tables marked as a
"MasterAddTable".

To make a long story, I made a patch to RServ.pm and Replicate that
allows you
to specify, on the command line, a list of tables that you want to
replicate...it'll ignore all others.

I haven't finished, since this has to be integrated with CleanLog for
instance, but this should (and does) suffice for the moment.

I have yet to test it with two slaves, but at least my mail server
replication
database now works (it was failing every time it tried to replicate, for
a
variety of reasons).

Anyone have any suggestions on how to improve on this?  (or, if someone
more
familiar with this code wants to take the ball and run with it, you're
welcome to).

--
Michael A Nachbaur <mike@nachbaur.com>

Here's a small patch to pg_hba.conf.sample that explains the use of CIDR
addresses.

Andrew Dunstan

Create <link> entries for the foreign keys in the system catalog
chapter.

I got tired of hitting the 'up' link, then finding the table in the
table of contents, and following it back down.

Rod Taylor

client-auth.sgml references a renamed ID in runtime.sgml

Josh Berkus

Attached is the fully corrected version of the re-ording patch for
Runtime.sgml and dependant files.

Josh Berkus

- Corrections for tables, columns with uppercase characters
- Don't attempt to convert partial or expressional unique indexes
- Don't attempt to convert unique indexes based on a non-default
opclasses

- Untested prevention of conversion of non-btree indexes unique
indexes.  Untested as postgresql doesn't allow hash, gist, or rtree
based indexes to be unique.

rbt=# create unique index t on a using hash (col);
ERROR:  DefineIndex: access method "hash" does not support UNIQUE
indexes
rbt=# create unique index t on a using gist (col);
ERROR:  DefineIndex: access method "gist" does not support UNIQUE
indexes
rbt=# select version();
                                version
------------------------------------------------------------------------
 PostgreSQL 7.4devel on i386-unknown-freebsd4.8, compiled by GCC 2.95.4

Rod Taylor

Please apply attached patch to contrib/dblink. It adds named persistent
connections to dblink.

Shridhar Daithanka

This is a bug in python interface module,
postgresql-7.3.3/src/interfaces/python/pg.py.

_quote() function fails due to integer overflow if input d is larger
than max integer.

In the case where the column type is "BIGINT", the input d may very well
be larger than max integer while its type, t, is labeled 'int'.
The conversion on line 19, return "%d" % int(d), will fail due to
"OverflowError: long int too large to convert to int".

Please describe a way to repeat the problem.   Please try to provide a
concise reproducible example, if at all possible:
----------------------------------------------------------------------

[1] create a table with a column type 'BIGINT'.
[2] use pg.DB.insert() to insert a value that is larger than max integer

If you know how this problem might be fixed, list the solution below:
---------------------------------------------------------------------

Just changing the conversion at line 19 of pg.py to long(d) instead of
int(d) should fix it. The following is a patch:

Chih-Hao Huang

>> If a transaction marks a tuple for update and later commits without
>> actually having updated the tuple, [...] can we simply
>> set the HEAP_XMAX_INVALID hint bit of the tuple?
>
>AFAICS this is a reasonable thing to do.

Thanks for the confirmation.  Here's a patch which also contains some
more noncritical changes to tqual.c:
 .  make code more readable by introducing local variables for xvac
 .  no longer two separate branches for aborted and crashed.
    The actions were the same in all cases.

Manfred Koizar

Most of the synopsis areas for DROP commands use "name" as the
identifier, while some areas do not.

The attached converts be below to "name":
conversion_name
index_name

The below have an existing, initdb supplied, entity named "name".  As
such, it could be confusing for the reader to see that identifier used
in the example.

domainname
typename

Rod Taylor

Revert ORDBMS change, per Peter.

Alter query.sgml to mention PostgreSQL as an ORDBMS.  This falls in line
with advocacy and 'portal' websites.

Link to createdb / dropdb from the tutorial page about create / dropdb.
A pair of notes were asking about more info...

Rod Taylor

In an attempt to simplify my life I'm submitting this patch that
restructures the deferred trigger queue.  The fundamental change is to
put all the static variables to hold the deferred triggers in a single
structure.

Alvaro Herrera

Add x86_64 support for spinlocks.

Jeffrey W. Baker

Small patch to link to the proper place in the "runtime" file,
and to add the "schemaname" column to the description of the
pg_stats view.

Greg Sabino Mullane

Array mega-patch.

Joe Conway

Done:

> * -Add IPv6 capability to INET/CIDR types

Jim C. Nasby wrote:
> Second argument to metaphone is suposed to set the limit on the
> number of characters to return, but it breaks on some phrases:
>
> usps=# select metaphone(a,3),metaphone(a,4),metaphone(a,20) from
> (select 'Hello world'::varchar AS a) a;
> HLW       | HLWR      | HLWRLT
>
> usps=# select metaphone(a,3),metaphone(a,4),metaphone(a,20) from
> (select 'A A COMEAUX MEMORIAL'::varchar AS a) a;
  > AKM       | AKMKS     | AKMKSMMRL
>
> In every case I've found that does this, the 4th and 5th letters are
> always 'KS'.

Nice catch.

There was a bug in the original metaphone algorithm from CPAN. Patch
attached (while I was at it I updated my email address, changed the
copyright to PGDG, and removed an unnecessary palloc). Here's how it
looks now:

regression=# select metaphone(a,4) from (select 'A A COMEAUX
MEMORIAL'::varchar AS a) a;
   metaphone
-----------
   AKMK
(1 row)

regression=# select metaphone(a,5) from (select 'A A COMEAUX
MEMORIAL'::varchar AS a) a;
   metaphone
-----------
   AKMKS
(1 row)

Joe Conway

Prevent compiler warning from sprintf in recent ipv6 patch.

    Add ipv6 address parsing support to 'inet' and 'cidr' data types.

        Regression tests for IPv6 operations added.

        Documentation updated to document IPv6 bits.

        Stop treating IPv4 as an "unsigned int" and IPv6 as an array of
        characters.  Instead, always use the array of characters so we
        can have one function fits all.  This makes bitncmp(), addressOK(),
        and several other functions "just work" on both address families.

        add family() function which returns integer 4 or 6 for IPv4 or
        IPv6.  (See examples below)  Note that to add this new function
        you will need to dump/initdb/reload or find the correct magic
        to add the function to the postgresql function catalogs.

        IPv4 addresses always sort before IPv6.

        On disk we use AF_INET for IPv4, and AF_INET+1 for IPv6 addresses.
        This prevents the need for a dump and reload, but lets IPv6 parsing
        work on machines without AF_INET6.

        To select all IPv4 addresses from a table:

                select * from foo where family(addr) = 4 ...

        Order by and other bits should all work.

Michael Graff

Update:

< * Have SELECT '13 minutes'::interval display zero seconds
> * Have SELECT '13 minutes'::interval display zero seconds in ISO datestyle

Add  -L../ecpglib -lecpg for Cygwin compile.

Add:

> * Have SELECT '13 minutes'::interval display zero seconds

Add:

> * Allow the regression tests to start postmaster with -i so the tests
>   can be run on systems that don't support unix-domain sockets

Add:

> * Rename /scripts directory because they are all C programs now

Add:

<
>  o Allow ALTER TABLE to modify column lengths and change to binary
>    compatible types

Add:

> * Modify pg_get_triggerdef() to take a boolean to pretty-print,
>  and use that as part of pg_dump along with psql
>

Update:

* Allow UPDATE, DELETE to handle table aliases for self-joins [delete]

typedef sa_family_t for cygwin.

Add:

>  o Allow columns to be reordered using ALTER ... POSITION i col1 [,col2];
>    have SELECT * and INSERT honor such ordering

Do not include:

#include <pwd.h>
#include <netdb.h>

in MS CC/Borland builds.

Remove references to sa_family_t, except when SOCKADDR_STORAGE requires
it.

Also handle __ss_family as a synonym for ss_family.

Kurt Roeckx

Add with '?':

> * Have initdb set DateStyle based on locale?

Add mention that the grep's have to be adjusted in pgtest.

Remove '?':

<   from making invalid dates valid?
>   from making invalid dates valid

Change pqInternalNotice to accept a format string and args instead of
just a preformatted message; per suggestion by Sean Chittenden.

Remove crypt.h from fe-connect.c --- not needed, and caused problems on
Solaris with Open SSL version 0.9.7b

Update Russian version of FAQ.

Add:

> * Exit postmaster if postgresql.conf can not be opened

Add:

* Allow SSL-enabled clients to turn off SSL transfers

Add:

> * Change CVS $Id$ to $PostgreSQL$

Revise hash join and hash aggregation code to use the same datatype-
specific hash functions used by hash indexes, rather than the old
not-datatype-aware ComputeHashFunc routine.  This makes it safe to do
hash joining on several datatypes that previously couldn't use hashing.
The sets of datatypes that are hash indexable and hash joinable are now
exactly the same, whereas before each had some that weren't in the other.

Fix some index entries.

Adjust chapter ordering in Internals part to something that seems more
reasonable.

Added missing terminating '\0' char for data put into char *.

Remove a lot of desperately obsolete material (which was all out of sight,
out of mind, because it'd been commented out years ago).  Try to bring the
remains up to a reasonable level of currency, and give it all approximately
the same high level of abstraction.

Add question mark to item:

<   from making invalid dates valid
>   from making invalid dates valid?

Add:

> * Allow current datestyle to restrict dates;  prevent month/day swapping
>   from making invalid dates valid
> * Prevent month/day swapping of ISO dates to make invalid dates valid

Add pgtest script.

Use libpq's new logic to get the server version, instead of doing it ourselves.

Bring the libpq example programs into the 21st century.

Get rid of extraneous newline in PQendcopy error output (was causing
regression test diffs...).

Update libpq to make new features of FE/BE protocol available to
client applications.  Some editorial work on libpq.sgml, too.

Fix some markup problems.

Fix for extended-query protocol: in event of error, backend was issuing
a ReadyForQuery (Z message) immediately and then another one after the
Sync message arrives.  Suppress the first one to make it work per spec.

Synced with backend.

Just another Informix compatibility change. They uses "free" for cursors as wellafter closing them.

Just another ecpg todo done.

Sorry, missed a file.

Allow constants in using clauses.

We neglected to set conn->raddr.salen, leading to breakage of CANCEL
and probably other stuff.

Disallow dollar sign in operator names, instead allow it as a non-first
character in identifiers.  The first change eliminates the current need
to put spaces around parameter references, as in "x<=$2".  The second
change improves compatibility with Oracle and some other RDBMSes.  This
was discussed and agreed to back in January, but did not get done.

Fixed fetch into char * and added missing prototype for an Informix function.

Move thread checking code farther down in conflgure.

Change clusterdb and vacuumdb into C programs.

Make FLOAT(p) measure the precision p in bits, not decimal digits, to
match the SQL standard.  Document FLOAT and FLOAT(p) notations in
datatype.sgml.  Per recent pghackers discussion.

More information schema views.

Add missing file to clean target.

Fixed several more parsing bugs.

Fix error line numbers reported for errors in plpgsql_parse_word and
siblings.

Fix bugs in interval-to-time conversion: HAVE_INT64_TIMESTAMP case did not
work at all, and neither case behaved sanely for negative intervals.

Fixed two small bugs.

Allow GROUP BY, ORDER BY, DISTINCT targets to be unknown literals,
silently resolving them to type TEXT.  This is comparable to what we
do when faced with UNKNOWN in CASE, UNION, and other contexts.  It gets
rid of this and related annoyances:
select distinct f1, '' from int4_tbl;
ERROR:  Unable to identify an ordering operator '<' for type unknown
This was discussed many moons ago, but no one got round to fixing it.

Adjust nestloop-with-inner-indexscan plan generation so that we catch
some cases of redundant clauses that were formerly not caught.  We have
to special-case this because the clauses involved never get attached to
the same join restrictlist and so the existing logic does not notice
that they are redundant.

Replace cryptic 'Unknown kind of return type' messages with something
hopefully a little more useful.

Cause GROUP BY clause to adopt ordering operators from ORDER BY when
both clauses specify the same targets, rather than always using the
default ordering operator.  This allows 'GROUP BY foo ORDER BY foo DESC'
to be done with only one sort step.

index() -> strchr().

Typo in version number.

Some minor changes for new version numbering.

Updating ECPG todo items.

Ecpg cleanups for prototypes.

Run autoconf/autoheader for ecpg change.

Make ecpg thread safe.

Lee Kindness

Move thread os defines into template files.

Handle threading in two more gethostbyname calls.

Make libpq thread-safe with configure --with-threads option.

Lee Kindness

Add thread.c for libpq threading, and hook it into libpq/configure.

Add --with-threads configure option to control threaded libpq.

- Enable FETCH without INTO.
- Compatibility functions for INFORMIX handling of DECLARE statement.

Okay, recognize freebsd 2.* and 3.* too.

Expect FreeBSD 5.* to have standard float arithmetic.

Fix broken markup.