Editorial review

If we fail to fork a new backend process, (try to) report the failure
to the client before closing the connection.  Before 7.2 this was done
correctly, but new code would simply close the connection with no report
to the client.

Fix ecpg wording to be consistent.

Remove ecpg TODO section:

> Looking at this I also found an ecpg TODO list in the docs:
>
>
http://candle.pha.pa.us/main/writings/pgsql/sgml/ecpg-develop.html
>
> Seems that TODO section should be removed.  Some items are done,
others
> are on the main TODO list.

That's correct. I did not fix the docs for quite some time.

Michael
--
Michael Meskes

Fix copy-and-paste brain damage.  (These commands do not create a
database.)

Add CHECK_FOR_INTERRUPTS() in various strategic spots, per comments
from Hiroshi.

Bugfix for bug reported by Marcus Better (marcus@dactylis.com).  When preforming
a get on a bytea value the code was running the raw value from the server
through character set conversion, which if the character set was SQL_ASCII
would cause all 8bit characters to become ?'s.

Spelling fix:

< * Consider use of open/fctl(O_DIRECT) to minimize OS caching
---
> * Consider use of open/fcntl(O_DIRECT) to minimize OS caching

Re-order items:

> * Consider use of open/fctl(O_DIRECT) to minimize OS caching
> * Make blind writes go through the file descriptor cache
391d392
< * Make blind writes go through the file descriptor cache
409d409
< * Consider use of open/fctl(O_DIRECT) to minimize OS caching

Added:

> * Consider use of open/fctl(O_DIRECT) to minimize OS caching

ecpg.sgml manual page updated to ON.

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

When you run 'ecpg --help' you get the following:

          -t   turn on autocommit of transactions

amongst the other options... Shouldn't this be OFF as per the
documentation?

Best regards, Lee.

--
 Lee Kindness,   Senior Software Engineer,   lkindness@csl.co.uk

Change:

<   perhaps CREATE DATABASE dbname WITH USER = "user" (Gavin)
---
>   perhaps CREATE DATABASE dbname WITH OWNER = "user" (Gavin)

Add from Vadim:

> * Allow command blocks that can ignore certain types of errors

Fix some incorrect and obsolete commentary.

Add more complete details on date/time keywords and parsing rules.
Move some tabular information on these from the chapter on data types to
 the appendix on dates and times.

Have to_date() call timestamptz_date() per Karel's email instructions.
Fixes time zone problems introduced by Thomas' implementation of
 TIMESTAMP WITHOUT TIME ZONE which caused the behavior of the previously
 appropriate routine, timestamp_date(), to change for the worse in this
 context.

Replace #ifdef, #endif with # requires comment.

Update FAQ.

Give attribution:

<   perhaps CREATE DATABASE dbname WITH USER = "user"
---
>   perhaps CREATE DATABASE dbname WITH USER = "user" (Gavin)

Remove:

< * Allow only owner to create indexes

Require ownership permission for CREATE INDEX, per bug report.
Disallow CREATE INDEX on system catalogs, non-tables (views, sequences, etc).
Disallow CREATE/DROP TRIGGER on system catalogs, non-tables.
Disallow ALTER TABLE ADD/DROP CONSTRAINT on system catalogs.
Disallow FOREIGN KEY reference to non-table.
None of these things can actually work in the present system structure,
but the code was letting them pass without complaint.

Instead of waiting a fixed amount of time for the test postmaster to
start up, wait until a psql connection attempt succeeds.  Per gripe
from Jason Tishler.

Prefer savepoints term:

<    allow error codes to be specified; requires nested transactions
---
>    allow error codes to be specified; requires savepoints
321c321
< * Allow nested transactions / savepoints [transactions]
---
> * Allow savepoints / nested transactions [transactions]

Update:

<    allow error codes to be specified
---
>    allow error codes to be specified; requires nested transactions

SPI_cursor_open must copy by-reference parameter values into the
portal's memory context, so that they will live as long as the portal does.

Add:

>  o Allow COPY to report error lines and continue;  optionally
>    allow error codes to be specified

Add:

> * Make equals sign optional in CREATE DATABASE WITH param = 'val'

Added:

> * Prevent create/drop scripts from allowing extra args (Bruce)

Shouldn't try to copy null datums with datumCopy.

Added:

> * Allow only owner to create indexes

Updated:

< * Add config file check for $ODBCINI, $HOME/.odbc.ini, and /etc/odbc.ini
---
> * Add config file check for $ODBCINI, $HOME/.odbc.ini, installpath/etc/odbc.ini

Add to java mention of pl/j.

Droplang didn't have test for -l, list.  This was really broken too.
Took test from createlang.

Add mention that createlang.sh indirection not needed but kept for later.

Update FAQ_DEV.

Update FAQ_DEV.

Added

> * Add config file check for $ODBCINI, $HOME/.odbc.ini, and /etc/odbc.ini

Added suggested syntax:

> * Make it easier to create a database owned by someone who can't createdb,
>   perhaps CREATE DATABASE dbname WITH USER = "user"

> > On Fri, Dec 21, 2001 at 11:43:21AM +0800, Christopher Kings-Lynne
wrote:
> > > Just testing pgcrypto on freebsd/alpha.  I get some warnings:
> > They should be harmless, although I should fix them.
>
> The actual code is:
>
>     if ((dlen & 15) || (((unsigned) res) & 3))
>         return -1;

> Hard to imagine how (uint *) & 3 makes any sense, unless res isn't
> always a (uint8 *).  Is that true?

At some point it was casted to (uint32*) so I wanted to be sure its ok.
ATM its pointless.  Please apply the following patch.

--
marko

Add mention of index on foreign key for frequently updated primary key
tables.

droplang had same problem as createlang;  default are backwards and did
not work.  Fixed to match createlang.

Add to pool discussion.

Added

<  o Move to using 'make' rather than 'ant'(?)
---
>  o Support both 'make' and 'ant'

Add mention of PRIMARY in ALTER TABLE ... [ UNIQUE | PRIMARY ]

In createlang, make dbname optional, not langname, or rather, make
existing code of dbname optional actually work.

Remove:

< * Change 'createlang [langname] dbname' to 'createlang langname [dbname]'

Added:

> * Change 'createlang [langname] dbname' to 'createlang langname [dbname]'

Guard against roundoff errors in new selectivity-estimation code,
per bug report from Laurette Cisneros.

Fix to_timestamp/to_date so that zero year input for Y, YY, or YYY
formats will be taken as 2000, not year zero.  Per bug report from
Aasmund Midttun Godal.  Fix from Karel Zak.

Fix incorrect description of rowtype variable declarations.  Eventually
(IMHO) the code should be fixed to do what the documentation formerly
claimed, and then revert this change.

Do not accept interrupts in RESUME_INTERRUPTS() and END_CRIT_SECTION()
macros, but only at explicit CHECK_FOR_INTERRUPTS() calls.  Not clear
whether overenthusiastic acceptance of interrupts accounts for any real
bugs, but it definitely seems risky and unnecessary.

Fix race condition that could allow two concurrent transactions
to insert the same key into a supposedly unique index.  The bug is of
low probability, and may not explain any of the recent reports of
duplicated rows; but a bug is a bug.

Rename TIME token to ISOTIME to eliminate conflict with gram.y parser
 token. Seems to be isolated to datetime.c and datetime.h.

Update dbase for Maarten.Boekhold.

Make sure that all <ctype.h> routines are called with unsigned char
values; it's not portable to call them with signed chars.  I recall doing
this for the last release, but a few more uncasted calls have snuck in.

Repair example, per note in interactive docs.

Update FAQ_DEV.

*** empty log message ***

Add

> * Make locking of shared data structures more fine-grained

Oops, only wanted datetime.c changes in there.  lock stuff reversed out.

Fix newly introduced datetime.c compile failure;  not enough parens.

Added

> * Change factorial to return a numeric

Add missing <varlistentry>;  shown in build logs.

Check a bit more carefully for preceeding ISO field tags
 when decoding date fields.

Document the precision arguments for date/time types per SQL9x.
Update the list of recognized time zones.
Document the range of arguments allowed for SET TIME ZONE.
Still need to add info on other date/time symbols (e.g. "AM", "T")
 and to freshen the docs on the date/time parsing rules.

Rework the date/time parsing to tighten up some cases and to enable other
 cases which should have worked but did not.
Now supports julian day (J2452271), ISO time labels (T040506) and various
 combinations of spaces and run-togethers of dates, times, and time zones.
All regression tests pass, and I have more tests to add after the 7.2
 release (don't want to require changes to the ancillary horology result
 files until after then).

Add crossdb TODO.detail item.

Add crossdb TODO.detail item:

---
> * Allow queries across multiple databases [crossdb]

Update server DEBUG startup tag fixes:

> * Change DEBUG startup tag to NOTICE;  change NOTICE to output to client
>   only if client exists (Bruce)

This item is not done:

< * -Make elog(LOG) in WAL its own output type, distinct from DEBUG (Peter E)
---
> * Make elog(LOG) in WAL its own output type, distinct from DEBUG

Add to java.

Added

> * Add --port flag to regression tests

Update FAQ_DEV.

Update FAQ_DEV.

Added

> * Change FIXED_CHAR_SEL to 0.20 from 0.04 to give better selectivity (Bruce)

Update FAQ_DEV.

Improve LOCK_DEBUG logging code for LWLocks.

Added

> * Report failure to find readline or zlib at end of configure run

Add
>  o Change syntax to WITH DELIMITER, (keep old syntax around?)

Add

> * Add WAL index reliability improvement to non-btree indexes

Update from Serguei Mokhov

Add pljava messages.

Typo.

< * Improve access-permissions check on data directory in CYGWIN (Tom)
---
> * Improve access-permissions check on data directory in Cygwin (Tom)

Added

<
---
> * Improve access-permissions check on data directory in CYGWIN (Tom)

Ensure that all direct uses of spinlock-protected data structures use
'volatile' pointers to access those structures, so that optimizing
compilers will not decide to move the structure accesses outside of the
spinlock-acquire-to-spinlock-release sequence.  There are no known bugs
in these uses at present, but based on bad experience with lwlock.c,
it seems prudent to ensure that we protect these other uses too.
Per pghackers discussion around 12-Dec.  (Note: it should not be
necessary to worry about structures protected by LWLocks, since the
LWLock acquire and release operations are not inline macros.)

Add

> * Allow binding query args over FE/BE protocol

Added

>  o Have ALTER TABLE OWNER change all dependant objects like indexes

Add thread item.

Add more Christof Petig ecpg items.

Christof Petig changes where for HISTORY, not TODO.

Oops, back out paren fix.  That is for 7.3.

Add mention of Christof Petig for ecpg items.

Added for Christof Petig:

<  o Allow SELECT of array of strings into a auto-sized variable
---
>  o -Allow SELECT of array of strings into a auto-sized variable

>  o auto allocation for indicator variable arrays (int *ind_p=NULL)
>  o auto allocation for string arrays (char **foo_pp=NULL)
>  o ECPGfree_auto_mem fixed
>  o all function names with external linkage are now prefixed by ECPG

Added:

> * Add OpenBSD's getpeereid() call for local socket authentication (Bruce)

Update to show current capability and future direction, from Peter:

> * -Allow secure single-user access without passwords using Unix socket permissions
> * Make single-user local socket access permissions the default (Peter E)

Added:

! * Add PGPASSWORDFILE environment variable or ~/.pgpass to store
!   user/host/password combinations

I know we talked about this in the past but I never got it on the TODO
list:

* Add optional CRC checksum to heap and index pages

Expand on description of when to use or not use TRUST auth method.

Document obj_description and col_description functions; expand
description of COMMENT command.

Small fixes

Update from Serguei Mokhov