This problem had nothing to do with the upgrades I made.  There
was a 2000 character buffer allocated for results, and the files
you refer to produce a 2765 byte column called formsource.  This
should not have worked with any version of libpgtcl.

Nevertheless, the limit is an artificial one, since there is no
need to use this intermediate buffer where it is being used and
abused.

Randy Kunkee <kunkee@pluto.ops.NeoSoft.com>

The following uuencoded, gzip'd file will ...

1. Remove the char2, char4, char8 and char16 types from postgresql
2. Change references of char16 to name in the regression tests.
3. Rename the char16.sql regression test to name.sql.  4. Modify
the regression test scripts and outputs to match up.

Might require new regression.{SYSTEM} files...

Darren King

The buffer overrun while creating sequences turns out to be a read
access overrun. For the sake of doing things properly here is a
patch which fixes it.

This patch is for the file backend/commands/sequence.c.

Maurice Gittens

There's a patch attached to fix gcc 2.8.x warnings, except for the
yyerror ones from bison. It also includes a few 'enhancements' to
the C programming style (which are, of course, personal).

The other patch removes the compilation of backend/lib/qsort.c, as
qsort() is a standard function in stdlib.h and can be used any
where else (and it is). It was only used in
backend/optimizer/geqo/geqo_pool.c, backend/optimizer/path/predmig.c,
and backend/storage/page/bufpage.c

> > Some or all of these changes might not be appropriate for v6.3,
since we > > are in beta testing and since they do not affect the
current functionality.  > > For those cases, how about submitting
patches based on the final v6.3 > > release?

There's more to come. Please review these patches. I ran the
regression tests and they only failed where this was expected
(random, geo, etc).

Cheers,

Jeroen

I started adding the Having Clause and it works quite fine for
sequential scans! (I think it will also work with hash, index, etc
but I did not check it out! I made some High level changes which
should work for all access methods, but maybe I'm wrong. Please
let me know.)

Now it is possible to make queries like:

select s.sname, max(p.pid), min(p.pid) from part p, supplier s
where s.sid=p.sid group by s.sname having max(pid)=6 and min(pid)=1
or avg(pid)=4;

Having does not work yet for queries that contain a subselect
statement in the Having clause, I'll try to fix this in the next
days.

If there are some bugs, please let me know, I'll start to read the
mailinglists now!

Now here is the patch against the original 6.3 version (no snapshot!!):

Stefan

Upgrade to v0.86

Prevent pgindent from being run on odbc in the future.

I have been troubled by a segmentation fault when reloading from
a dumpall.  This has been happening when a second \connect is
encountered.

The faulty code was in fe-connect.c, where the memory for the user
password was freed, but the pointer itself was not set to NULL.
Later, the memory was reused and the password appeared not to be
empty, so that an attempt was made to reference it.

Oliver Elphick

BETWEEN and LIKE patch from Thomas

Update manual

Mention rollback instead of abort.

install cleanup

FAQ_Irix update

update for 6.3.1

Manual page fixes.

autoconf again.

more tcl/tk fixes

Another tcl/tk fix.

oops, forgot to run autoconf.

another tcl/tk patch

another tcl/tk patch

fix for tcl/tk in configure

Add NetBSD specific expected files

Slightly updated regression output for FreeBSD 3.0-CURRENT

Made some changes to clean up how initdb works...just more made a 'PGSQL_OPTS'
variable that can be set in one place to affect everything...

Change 2>$1 to -o /dev/null instead...

From: Andreas Klemm <andreas@klemm.gtn.com>

Moved sys/types.h to before netdb.h

From: Vince Vielhaber <vev@michvhf.com>

Cleanup for Thomas

Contribution for logging Apache Web stats to PostgreSQL

From: Terry Mackintosh <terry@terrym.com>

change in item person;

cleanup instructions for patch upgrade.

Update for 6.3p1 release.

Better identify tcl and tk.

This patch fixes a couple of minor bugs:

1) DatabaseMetaData.getPrimaryKeys() would fail saying that there
is no
   table t.

2) PreparedStatement.getObject() was missing some break statements,
which
   was causing updates not to work with JBuilder (supplied by Aaron
   Dunlop).

jdbc fixes from Peter.

Disable -lbsd on alpha.

Add missing file.

OK...here is a patch that will cause the magnetic disk storage
manager to not try to split files in 2 gig chunks.  It will just
try to get another block.

If applied, everything is just as before. But if LET_OS_MANAGE_FILESIZE
is defined, the chaining disappears and the file just keeps on
going, and going, and going, til the OS barfs.

Darren King

Sorry. I made above mistakes. "__svr4" should be "__svr4__" or
"__SVR4" as you pointed out.  There is another file that has the
same mistakes.  Included is a patche for include/c.h.

t-ishii@sra.co.jp

Fix spelling of "including

Edward J. Huff

HPUX build does not build shared lib versions of libpq and libpq++

--------------------------------------------------------------------------

Test Case:  ----------

--------------------------------------------------------------------------

Solution:  --------- Add this to the libpq and libpq++ Makefiles
to build shared libs:

Mike Ferrara

In libpq-fe.h:  #if defined(sun) && defined(sparc) && !defined(__svr4)
extern char *sys_errlist[]; #define strerror(A) (sys_errlist[(A)])
#endif                                                  /* sunos4
*/

is picked up by Solaris when the above is intended only for SunOS.

Fix Solaris.  Albert Chin-A-Young

The real trick is to add -Dalpha to the CFLAGS setting.  The changes
to main.c are only to add some extra includes to support some code
that's suddenly being used.

The #define ASSEMBLER is to prevent most of the code of sys/proc.h
from being included, as it ends up conflicting with some of the
postgresql definitions.  This may or may not work on other versions
of Digital Unix.

Get alpha working.  Yea.  Dwayne Bailey

> > I'm using text[] arrays.  Some of my array elements have '"'
> > characters in them.  Dumping and reloading using pg_dumpall >
> doesn't work with this and dumping the entire array and > > then
trying to parse it is hopeless.

Doug Gibson

Here's my next patch to bring ecpg to version 1.1. It now correctly
handles all transaction commands and the exec sql include command.

Michael Meskes

Allow parsing expressions with ") -" (scan.l, scan.c only).
Make "TABLE" optional in "LOCK TABLE" command
 and "... INTO TABLE..." clause.
Explicitly parse CREATE SEQUENCE options to allow a negative integer
 as an argument; this is an artifact of unary minus handling in scan.l.
Add "PASSWORD" as an allowed column identifier.
These fixes will require a "make clean install" but not a dump/reload.

Add checks for UNION target fields, and add optional TABLE to LOCK
and SELECT manual pages and psql help.

SunOS .similar cleanup.

Add SunOS entries.

Prevent \do from wrapping.

Make pg_user /xpg_user change complete.

Remove duplicates

Fix for Datasize = 0 error, from Vadim.

Fix for pg_dump of large objects, from alvin@camberlo.demon.co.uk.

subselect notes from Vadim.

From: Jeroen van Vianen <jeroenv@design.nl>

tools.patch patches tools/find_static (use indices to increase
performance) and tools/ccsym (no hardcoded paths).

From: hankin <hankin@consultco.com>

a while back I posted a patch for pg_ident, the patch worked but I didn't
diagnose the problem properly.
on my compiler(gcc2.7.2) this compiles with no errors...

char buf[1000]; if(buf != '\0') {

...but it doesn't compare '\0' with the first char of buf.

Reply-To: Jordi MacDonald <jordi@spartanmedia.com>
There is an error in the configure script when using
--with-pgport= that will cause the compiled version of
PostgreSQL to no longer allow connections to the
new port and to treat shared memory improperly.

What happens is that if the port is changed, the configure
script defines DEF_PGPORT as "", which atoi() will return
as 0, which makes the IPC_KEY value 0. This then causes
semaphores to be allocated, but never released. Postgres
eventually returns from semget() with
"no space left on device". The source of this error could
easily be overlooked in version 6.3 since it is possible
to connect via UNIX domain sockets, and having DEF_PGPORT
defined as "0" would not be noticed until TCP was used.

From: Randy Kunkee <kunkee@pluto.ops.NeoSoft.com>

The following patch is to src/interfaces/libpq of postgresql-6.3.
The purpose of the patch is to make the initialization of
const char *pgresStatus[] match the ExecStatusType enum.

From: t-ishii@sra.co.jp

6.3 postmaster is supposed to work with pre 6.3 protocol. This is true
for little endian architecture servers. But for big endian machines
such as Sparc the backward compatibility function do not work.
Attached are patches to fix the problem.

From: "Thomas G. Lockhart" <lockhart@alumni.caltech.edu>

For substr() and substring() on the text data type, the relevant code is in
varlena.c. You are right, there is a problem. I have a patch which I will
apply to the source tree soon. The copy enclosed below probably does not
preserve tabs correctly so cannot be applied directly; the relevant change
is simply changing the ">=" to ">"...

From: Randy Kunkee <kunkee@pluto.ops.NeoSoft.com>

It is my hope that the following "patches" to libpgtcl get included
in the next release.

See the update to the README file to get a full description of the changes.
This version of libpgtcl is completely interpreter-safe, implements the
database connection handle as a channel (no events yet, but will make it
a lot easier to do fileevents on it in the future), and supports the SQL
"copy table to stdout" and "copy table from stdin" commands, with the
I/O being from and to the connection handle.  The connection and result
handles are formatted in a way to make access to the tables more efficient.

From: t-ishii@sra.co.jp

Included are patches intended for allowing PostgreSQL to handle
multi-byte charachter sets such as EUC(Extende Unix Code), Unicode and
Mule internal code. With the MB patch you can use multi-byte character
sets in regexp and LIKE. The encoding system chosen is determined at
the compile time.

To enable the MB extension, you need to define a variable "MB" in
Makefile.global or in Makefile.custom. For further information please
take a look at README.mb under doc directory.

(Note that unlike "jp patch" I do not use modified GNU regexp any
more. I changed Henry Spencer's regexp coming with PostgreSQL.)

From: t-ishii@sra.co.jp

Included are patches intended for allowing PostgreSQL to handle
multi-byte charachter sets such as EUC(Extende Unix Code), Unicode and
Mule internal code. With the MB patch you can use multi-byte character
sets in regexp and LIKE. The encoding system chosen is determined at
the compile time.

To enable the MB extension, you need to define a variable "MB" in
Makefile.global or in Makefile.custom. For further information please
take a look at README.mb under doc directory.

(Note that unlike "jp patch" I do not use modified GNU regexp any
more. I changed Henry Spencer's regexp coming with PostgreSQL.)

From: Peter T Mount <patches@maidast.demon.co.uk>

Ok, this fixes three things:

1. It seems (from tests submitted by two people with JBuilder) that
   JBuilder expects a responce from ResultSetMetaData.getPrecision() &
   getScale() when used on non numeric types. This patch makes these
   methods return 0, instead of throwing an exception.

2. Fixes a small bug where getting the postgresql type name returns null.

3. Fixes a problem with ResultSet.getObject() where getting it's string
   value returns null if you case the object as (PGobject), but returns
   the value if you case it as it's self.

CLUSTER cleanup

Cluster  fixup.

Change Postgres95 to PostgreSQL.  Update CLUSTER manual page.

Remove postgres95.

Update Pgaccess to 0.83

Get the ultrix4 ports back in sync...

Repair "LIKE" behavior with two adjacent wildcard characters ("_").
 Was ignoring second wildcard.

Add one-line fix to allow redefining built-in functions.
 Bug introduced in mid-January.

Fix bad oid in table.

xinv fix for tclsh

pg_user cleanup.

Rename pg_user to pg_shadow.

Add ecpg back in again

Get rid of those pesky long constants, 'l'.  Has no affect.

One last change to configure for 'non-gcc' compiler

From: Darren King <darrenk@insightdist.com>

Patch1:

Postgres thinks dist_pl (dist of a point to a line) is expecting a box (603)
for the right arg, but it really should be a line (628).

Otherwise the left & right args match those of dist_pb (dist of a point to a
box) two lines further down.

Patch2:

Anyways, these two functions take a path (602) whereas in pg_proc.h they are
listed as taking a lseg (601).

Reminder for next time.

Last minute clean up for SunOS...

From: t-ishii@sra.co.jp

Change configure to reflect v6.3 vs v6.2 :)

Bring pgaccess up to 0.82 - last update before release

Fix money type USE_LOCALE support at least for default "C" locale.
Still has questionable code for some locale-specific strings.

Update for release.  Add mention of new manuals.

Install new 0.81  pgaccess release.

Update html tar files to include only necessary gif files.
Incorporate a few typo fixes.

Make this rely on a real Makefile in sgml/.
Implement installation and cleaning.

Fix a few typos. Try <Note><Title>Author</Title>... for author credit.

Try a fancier makefile. Allows real building of html and rtf output
 rather than just executing ../Makefile.

Replace incorrect gif files.

Allow installation of documentation packages.

Stopgap makefile for new sgml source documentation.
Should redo this before v6.3 release if possible.

SGML source for new documentation.

Graphics sources and GIF bitmaps for new documentation.

New documentation generated from SGML source.

Fix boolean operator declarations for path to return boolean, not float8.

Prepare for release.

Prepare for 6.3 release.

From: "Billy G. Allie" <Bill.Allie@mug.org>

The following patch will change the meaning of the isinf function in the
HAVE_FPCLASS section from "not finite" to "is infinity".