Renamed file.

Stylistic cleanups...

Add comment about possible zero value.

Change shell-style comments to C-style.

Fix assert to allow zero.  OK'ed by Bryan.

Fix args to match prototype.

Replace double // with /.  THis confuses mkid.

Added single-letter options and case statement.

Improved creation of mkid files.

Here the fix for the first assertion failure I had which killed
my postmaster 1.07.
It's really simple, the loop dealing with all sockets
can't handle more than one ready socket :-)
A simple logic error dealing with lists.
OR IS THERE ANY REASON FOR SETTING curr TO 0?

Submitted by:  Carsten Heyl <Heyl@nads.de>

And bring in change from 1.0.7 tree to here...

Moved the define of SIGNAL_ARGS that D'Arcy just created to config.h so that
hopefully this sort of thing can eventually be determined using configure?

The remainder of D'Arcy's changes, most notibly the usage of SIGNAL_ARGS

This patch corrects some errors in sample commands in the INSTALL file.

Submitted by:  bryanh@giraffe.netgate.net (Bryan Henderson)

This patch fixes problems in initdb, including that it doesn't run
with some versions of sh, and a bug in the master make file that
causes it to issue the message "postgres has been built" at the wrong
time.

Submitted by: bryanh@giraffe.netgate.net (Bryan Henderson)

Moved files.

new TODO.

Added purge(l) mention in manual page.

Removed -S from example and added nohup to log file.

Added dlfcn.h to dl.c in BSD44_derived port

Added restriction to tables vacuum'able.
Added NOTICE messages for bad table names.

Added chown to Makefile to set files to Postgres user.
Does not display and generates no error messages on failure.

MOved them.

Moved them.

New FAQ.

Renamed createdb.l  to create_database.l.
Changed portal to cursor.

Brought psql help up to date with syntax and manual.

New vacuum optoin.

New vacuum option.

Fixes extra space in printing.

Allow vacuum to run for only one table.  Fixes allocation bug.

Fixed file to properly do SEMUN for BSDI.
Grouped some PORTNAME's for SEMUN.

Manual changes for initdb.sh change.

Added new single-letter options to initdb.sh.
Removed long-option names that used ${#} shell feature which
was a compatability problem.

change stdout to fout

Submitted by: Carsten Heyl <heyl@nads.de>

Added NEED_UNION_SEMUN to config.h

Moved whether to define union semun from include/storage/ipc.h to
a define in include/config.h

This file was superceeded by README.fsync.

Changed default port name.

Change README to 2.0.

Add fsync option to Postgres manual page.

palloc() finfo memory.  From Kurt.

Fix cluster finfo problem Kurt reported.

Fixed YACC depenencies and 'make clean' now cleans totally.

Updated version.

This should cause 'make clean' to clean up bki files.  Looks like a typo.

Prevents possible packet loss from SIGCHLD.

upgrade.

Updated version.

Fix location of include files to match postgres.h includes.

Changed gawk to awk.

Patch for:

#     _fUnKy_POSTPORT_sTuFf_    -  port to run the postmaster on

Submitted by: D'Arcy

Change the comments pertaining to Makefile.custom

Submitted by: darcy@druid.druid.com (D'Arcy J.M. Cain)

Prototype setMaxOid()

Submitted by: Bruce

CFLAGS +- -I... got messed by one of Bryan's patches

Fix Submitted by: Bryan himself :)

Fix for using Makefile.custom

Suggested/Submitted by: darcy@druid.druid.com
Cleaner Method Submitted by: Bryan Henderson <bryanh@giraffe.netgate.net>

From: Bruce Momjian <maillist@candle.pha.pa.us>
To: Bryan Henderson <bryanh@giraffe.netgate.net>
Cc: Postgres95-development <pg95-dev@ki.net>
Subject: Re: [PG95-DEV] DELETE statement

>
> Does the man page for DELETE need to be corrected?  It gives as syntax
>
>   DELETE instance_variable [FROM fromlist] [WHERE qual]   .
>
> But the actual syntax appears to be
>
>   DELETE FROM classname WHERE qual   .
>
> --
> Bryan Henderson                                    Phone 408-227-6803
> San Jose, California
>

This patch fixes the delete manual page in 2.0:

From: Bruce Momjian <maillist@candle.pha.pa.us>
To: Postgres95-development <pg95-dev@ki.net>
Subject: [PG95-DEV] postgres.h patch

This removes the parameters from the func_ptr prototype in postgres.h in
2.0.

From: Bruce Momjian <maillist@candle.pha.pa.us>
To: Postgres95-development <pg95-dev@ki.net>
Subject: [PG95-DEV] Makefile.global patch

Can we apply the following patch to make EUROPEAN_DATES consistent with
other parameters?

Date: Sun, 22 Sep 1996 13:47:39 -0400 (EDT)
From: Bruce Momjian <maillist@candle.pha.pa.us>
To: Postgres95-development <pg95-dev@ki.net>
Subject: [PG95-DEV] 2.0 patch for fastgetattr

diff -ru ../../../orig/postgres95/src/backend/utils/adt/varlena.c
./utils/adt/varlena.c
- --- ../../../orig/postgres95/src/backend/utils/adt/varlena.c    Mon Jul 22
23:56:04 1996
+++ ./utils/adt/varlena.c       Tue Sep 17 15:12:55 1996
@@ -166,7 +166,7 @@

     if (inputText == NULL)
        return(NULL);
- -    len = strlen(inputText) + VARHDRSZ;
+    len = strlen(inputText) + VARHDRSZ + 1 /* terminating 0 */;
     result = (struct varlena *) palloc(len);
     VARSIZE(result) = len;
     memmove(VARDATA(result), inputText, len - VARHDRSZ);

Submitted by: skimo@breughel.ufsia.ac.be (Sven Verdoolaege)

diff -ru ../../../orig/postgres95/src/backend/libpq/be-fsstubs.c
./libpq/be-fsstubs.c
- --- ../../../orig/postgres95/src/backend/libpq/be-fsstubs.c     Tue Jul  9
08:21:30 1996
+++ ./libpq/be-fsstubs.c        Tue Sep 17 15:30:09 1996
@@ -241,7 +241,7 @@
      */
     fd = open(VARDATA(filename), O_RDONLY, 0666);
     if (fd < 0)  {   /* error */
- -       elog(WARN, "lo_import: can't open unix file\"%s\"\n", filename);
+       elog(WARN, "lo_import: can't open unix file \"%s\"\n",
VARDATA(filename));
     }

     /*

Submited by: skimo@breughel.ufsia.ac.be (Sven Verdoolaege)

Finish commiting Bryan's patches...

Here's a patch that I discussed recently on pg95-dev that changes the
way one creates a database system.  Parts that were in "make install"
are not either in "make all" or initdb.  Nothing goes in the PGDATA
directory besides user data.  Creating multiple database systems is
easier.

In addition to applying the patch, it is necessary to move the file
libpq/pg_hba to backend/libpq/pg_hba.sample.

Submitted by: Bryan Henderson <bryanh@giraffe.netgate.net>

*** src/backend/storage/file/fd.c.orig  Thu Sep 12 17:17:21 1996
--- src/backend/storage/file/fd.c       Thu Sep 12 17:23:38 1996
***************
*** 262,268 ****
      Delete(file);

      /* save the seek position */
!     fileP->seekPos = lseek(fileP->fd, 0L, SEEK_CUR);
      Assert( fileP->seekPos != -1);

      /* if we have written to the file, sync it */
--- 262,268 ----
      Delete(file);

      /* save the seek position */
!     fileP->seekPos = (long) lseek(fileP->fd, 0L, SEEK_CUR);
      Assert( fileP->seekPos != -1);

      /* if we have written to the file, sync it */

Submitted by: Randy Terbush <randy@zyzzyva.com>

Turned off EUROPEAN_DATES as default...

Pointed out by Bruce

|For starters, here's a small patch that removes a compiler warning in that
|regex stuff on Linux.
|
|--
|Bryan Henderson                                    Phone 408-227-6803
|San Jose, California
|

|This fixes the bug introduced yesterday that causes "not found" errors
|with gram.c and parser.h.
|
|--
|Bryan Henderson                                    Phone 408-227-6803
|San Jose, California
|

Nothing patched...just testing something with CVS...

Patches to make POSTPORT changes in scripts

Originally submitted by: ernst.molitor@uni-bonn.de
resubmitted by: D'Arcy Cain

Various patches from Bryan that *should* clean up the compile problems
ppl are seeing with v2.0

Add the regex include files to the repository...

In my cvs source tree, tihs directory existed, which is why it compiled
on my system, but nobody elses...

Documentation on the fsync() patch from OpenLink

Submitted by:  Cees de Groot <C.deGroot@inter.nl.net>

Newer version of Bruce's pginterface library...

Clean out the old...

Here is a /doc patch for the oid preservation code.

Submitted by: Bruce Momjian <maillist@candle.pha.pa.us>

nonulls is not supported in gram.y, so why advertise it.

Submitted by: Bruce Momjian <maillist@candle.pha.pa.us>

This shows the new use of defaults.

Submitted by: Bruce Momjian <maillist@candle.pha.pa.us>

I have two small patches which correct some very obscure bug in the parser
of the array constants and in one of the loadable modules I posted some time
ago.

Submitted by: Massimo Dal Zotto <dz@cs.unitn.it>

Another small change.  Added the void keyword to an empty arg list.

Submitted by: darcy@druid.druid.com (D'Arcy J.M. Cain)

Just a small change to flesh out the func_ptr typedef declaration.

Submitted by: darcy@druid.druid.com (D'Arcy J.M. Cain)

A few changes to cleanup the code.

  - Added the header access/heapam.h.
  - Changed all instances of "length" to "data_length" to quiet
    the compiler.
  - initialized a few variables.  The compiler couldn't see that
    the code guaranteed that these would be initialized before
    being dereferenced.  If anyone wants to check my work follow
    the usage of these variables and make sure that this true
    and wasn't actually a bug in the original code.
  - added a missing break statement to a default case.  This
    was a benign error but bad style.
  - layed out heap_sysattrlen differently.  I think this way
    makes the structure of the code crystal clear.  There should
    be no actual difference in the actual behaviour of the code.

Submitted by: darcy@druid.druid.com (D'Arcy J.M. Cain)

Cleanup patch of pg_version

Submitted by:  darcy@druid.druid.com (D'Arcy J.M. Cain)

Added querylimit patch...

Submitted by:  Massimo Dal Zotto <dz@cs.unitn.it>

Fix for pg_log bug

Submitted by: "Vadim B. Mikheev" <vadim@sable.krasnoyarsk.su>

Add missing includes to silence compile

Submitted by: D'Arcy Cain

Various standardizations and fixes submitted by D'Arcy Cain

NOTE: Makefile.custom is commented out, since it isn't there by default.
      If you read the section telling you about it to know to create it,
      you can uncomment it while you are there ...

At Andrew's suggestion, upgrade the Version numbers to reflect the
current state of development...namely, we are on 2.0

NOTE:

BTW, the is also a check in postmaster which won't let you use an older
version of the database by checking the version number. The version number
of a database is in data/PG_VERSION (a plain ASCII file).

- Andrew

The following diffs remove the various definitions of dupstr() and replace
all the calls with strdup.

Submitted by: darcy@druid.druid.com (D'Arcy J.M. Cain)

I have made some corrections to my previous patches for retrieving array
attributes as tcl arrays.  The previous code had problems with some chars
used as delimiter by Tcl.  The new code should be more robust.

By: Massimo Dal Zotto <dz@cs.unitn.it>

enable PQ_NOTIFY_PATCH in config.h

Fixes:

Async notifies received while a backend is in the middle of a begin/end
transaction block are lost by libpq when the final end command is issued.

The bug is in the routine PQexec of libpq. The routine throws away any
message from the backend when a message of type 'C' is received. This
type of message is sent when the result of a portal query command with
no tuples is returned. Unfortunately this is the case of the end command.
As all async notification are sent only when the transaction is finished,
if they are received in the middle of a transaction they are lost in the
libpq library. I added some tracing code to PQexec and this is the output:

Submitted by: Massimo Dal Zotto <dz@cs.unitn.it>

More code cleanups from "Kurt J. Lidl" <lidl@va.pubnix.com>

Enable FUNC_UTIL_PATCH by default

There is a bug in the function executor. The backend crashes while trying to
execute an sql function containing an utility command (create, notify, ...).
The bug is part in the planner, which returns a number of plans different
than the number of commands if there are utility commands in the query, and
in part in the function executor which assumes that all commands are normal
query commands and causes a SIGSEGV trying to execute commands without plan.

Submitted by: Massimo Dal Zotto <dz@cs.unitn.it>

|Subject: Postgres patch: Assert attribute type match
|
|Here's a patch for Version 2 only.  It just adds an Assert to catch some
|inconsistencies in the catalog classes.
|
|--
|Bryan Henderson                                    Phone 408-227-6803
|San Jose, California
|

Add use of INDEXSCAN_PATCH to config.h instead of putting it in Makefile.global

Fixes:

The problem is that the function arguments are not considered as possible key
candidates for index scan and so only a sequential scan is possible inside
the body of a function.  I have therefore made some patches to the optimizer
so that indices are now used also by functions.  I have also moved the plan
debug message from pg_eval to pg_plan so that it is printed also for plans
genereated for function execution.  I had also to add an index rescan to the
executor because it ignored the parameters set in the execution state, they
were flagged as runtime variables in ExecInitIndexScan but then never used
by the executor so that the scan were always done with any key=1. Very odd.
This means that an index rescan is now done twice for each function execution
which uses an index, the first time when the index scan is initialized and
the second when the actual function arguments are finally available for the
execution.  I don't know what is the cost of an double index scan but I
suppose it is anyway less than the cost of a full sequential scan, at leat
for large tables. This is my patch, you must also add -DINDEXSCAN_PATCH in
Makefile.global to enable the changes.

Submitted by: Massimo Dal Zotto <dz@cs.unitn.it>

Fixes:

The comparison routines for text and char data type give incorrect results
if the input data contains characters greater than 127.  As these routines
perform the comparison using signed char variables all character codes
greater than 127 are interpreted as less than 0.  These codes are used to
encode the iso8859 char sets.
The other text-like data types seem to work as expected as they use unsigned
chars in comparisons.

Submitted by: Massimo Dal Zotto <dz@cs.unitn.it>

Cleaned up a spurious '-' ...

Pointed out by: ernst.molitor@uni-bonn.de