Fixed segfault in Informix mode.

Downgrade LOG messages to DEBUG1 for normal recycling of xlog, clog,
subtrans segments.  Per Greg Mullane and Chris K-L.

Simplify IsXactIsoLevelSerializable test.  A cycle saved is a cycle
earned ...

On further consideration, there's another problem here: the existing
elog() emulation code always calls errstart with ERROR error level.
This means that a recursive error call triggered by elog would do
MemoryContextReset(ErrorContext), whether or not this was actually
appropriate.  I'm surprised we haven't seen this in the field...

Tweak elog.c's logic for promoting errors into more severe errors.
Messages of less than ERROR severity should never be promoted (this
fixes Gaetano Mendola's problem with a COMMERROR becoming a PANIC,
and is obvious in hindsight anyway).  Do all promotion in errstart
not errfinish, to ensure that output decisions are made correctly;
the former coding could suppress logging of promoted errors, which
doesn't seem like a good idea.  Eliminate some redundant code too.

Fix shared library linking operations to work properly on all (or at
least more) combinations of HP-UX hardware, compiler, and linker.
Shinji Teragaito and Tom Lane.

Apparently HPUX runs the IA64 in big-endian mode.

Fix ancient declaration inconsistency in cube & seg ... came across a
compiler that warns about it.

Some versions of lex will drop a lex.yy.c file when we probe to see if
they are flex.  Clean up after them.

Yet another place where someone was being careless about the arguments
of <ctype.h> macros.

Spelling correction, per Greg Mullane.

Fix getrusage() emulation on Windows.  Magnus Hagander

FAQ_HPUX updates from Shinji Teragaito.

Cope with recent HPUX versions providing isfinite() instead of finite().

Add s_lock support for HPUX on IA64, per Shinji Teragaito.

Remove useless variable.

Use $PATH_SEPARATOR like the rest of the autoconf code, instead of
hardwiring IFS=: when searching paths.

Remove line specifying Andreas as author.

Remove pg_config_paths.h from libpq *.mak builds so later MinGW builds
will succeed.

Add:

> * Consider using hash buckets to do DISTINCT, rather than sorting
>
>   This would be beneficial when there are few distinct values.

Back out timezone detection patch.  Tom already applied it.

This patch attempts to fix the issue with localized timezones on
Windows.

Recap: When running on a localized windows version, the timezone name
returned is also localized, and therefor does not match our lookup
table.

Solution: The registry contains both the name of the timezone in english
and the localized name. The patch adds code to scan the registry for the
localized name and gets the english name from that, and then rescans the
table.

I have tested this on a Swedish WinXP, and it works without problems.
The registry layout is the same in Win2k, but I haven't specifically
tested it. It's also the same on different languages but again only
Swedish is tested.

Magnus Hagander

The current implementation of dbsize doesn't handle tables in
tablespaces correctly, and is quite restricted on objects covered (only
tables and databases, but not tablespaces and indexes).

The attached patch contributes:

- database_size(name)
- relation_size(text)
These are the well-known functions, tablespace-aware.

- pg_tablespace_size(oid)
- pg_database_size(oid)
- pg_relation_size(oid)
Tablespace-aware implementations, used by the upper functions.
pg_relation_size will report sizes of indexes as well.

- pg_size_pretty(bigint)
Formatting of sizes, to display '146MB' instead of '152885668'

Andreas Pflug

Remove obsolete comment.

Tweak prettyprinting rules for saner indenting of UNION, INTERSECT,
EXCEPT constructs.

Fix bcc to compile libpq by creating include file for path.c.

Realign libpq and psql for need for path.c under Win32 VC and BCC.

Back out Cygwin timezone change until we have a more global solution.

Fix mistaken comment.

Whack Wisconsin benchmark around until it actually works again.
It's still useless because it tests a standalone backend, but at least
the bit rot is repaired.

Add code to be able to match the timezone name on localized Windows
systems.  Magnus Hagander.

Remove a stray reference to sequences as having tablespaces.

Translation updates

Add a note about the difference between Postgres' treatment of the rights
of an object owner and the SQL spec's treatment of these rights.

Cast _timezone to int from time_t for Cygwin.

Add index mention:

    <P>In pre-8.0 releases, indexes often can not be used unless the data
    types exactly match the index's column types.  This is particularly
    true of int2, int8, and numeric column indexes.</P>

src/bin/psql/tab-complete.c =~ s/CONVERSATION/CONVERSION/;

Greg Sabino Mullane

needs_toast_table() should ignore dropped columns.

Code review for recent changes in guc-file.l.  Avoid multiple frees,
use of already-freed strings, other silliness.  Also fix reporting of
config file syntax errors so that it actually works reasonably well
(eg, points at the correct line).  Use palloc instead of malloc for
temporary storage to reduce code clutter.

Code review for various recent GUC hacking.  Don't elog(ERROR) when
not supposed to (fixes problem with postmaster aborting due to mistaken
postgresql.conf change); don't call superuser() when not inside a
transaction (fixes coredump when, eg, try to set log_statement from
PGOPTIONS); some message style guidelines enforcement.

Fix unintended assignment of sequences to the containing schema's
default tablespace --- they should always go in the database's default
tablespace.  Adjust heap_create() API so that it is passed the relkind
to make this easier; should simplify any further tweaking of the same
sort.

FlushRelationBuffers was also being a bit cavalier about whether the
relation is already opened by smgr.

copy_relation_data was mistakenly assuming that the source relation
would always be already open at the smgr level.  Per bug report from
Fabien Coelho.

Define lstat with parameters, rather than just redefining the symbol.

Fix typo in lstat() macro exposed by new tablespace code.

Replace log_filename_prefix with more general log_filename parameter,
to allow DBA to choose the form in which log filenames reflect the
current time.  Also allow for truncating instead of appending to
pre-existing files --- this is convenient when the log filename pattern
rewrites the same names cyclically.  Per Ed L.

tag configure beta2

Improve spinlock selftest to make it able to detect misdeclaration of
the slock_t datatype (ie, declared type smaller than what the hardware
TAS instruction needs).

slock_t must be int not char for MIPS.  7.4 got this right, but the
info was apparently mistranscribed in s_lock code rearrangement.

Avoid ambiguity, as per suggestion from Peter Stricker.

Rather than referring to src/tutorial/README, put the build instructions
for the tutorial files right here.

Fix obviously-utterly-untested noTocComments code.

Ensure that the remainder of the current pg_clog page is zeroed during
startup, just to be sure that there's no leftover junk there.

Fix failure to advance nextXID beyond subtransactions whose XIDs appear
only within COMMIT or ABORT records.

Remove item because it is on the open item list:

< * Fix oid2name and dbsize for tablespaces

Tweak md.c logic to cope with the situation where WAL replay tries to
write into a high-numbered segment of a relation that was later deleted.
We need to temporarily recreate missing segment files, instead of
failing.

Dept. of second thoughts: it'd be a good idea to flush buffers
during replay of CREATE DATABASE as well as the first time around.
Else it's possible that the copy operation will copy obsolete blocks.
We are still a long way from guaranteeing anything about using a
recently-written database as a CREATE template, but this seems needed
to ensure the existing behavior holds up during replay.

Add Win32 service capability to pg_autovacuum.

Dave Page

Another pgindent run with lib typedefs added.

Add note that contrib/rserv is gone.

Add explicit note that empty-string array element values now have to
be written with quotes.  Minor copy-editing too.

Add WAL logging for CREATE/DROP DATABASE and CREATE/DROP TABLESPACE.
Fix TablespaceCreateDbspace() to be able to create a dummy directory
in place of a dropped tablespace's symlink.  This eliminates the open
problem of a PANIC during WAL replay when a replayed action attempts
to touch a file in a since-deleted tablespace.  It also makes for a
significant improvement in the usability of PITR replay.

Update typedefs with /lib info.

Replace bcopy by memmove for more portability.

Widen xl_len field of XLogRecord header to 32 bits, so that we'll have
a more tolerable limit on the number of subtransactions or deleted files
in COMMIT and ABORT records.  Buy back the extra space by eliminating the
xl_xact_prev field, which isn't being used for anything and is rather
unlikely ever to be used for anything.
This does not force initdb, but you do need to do pg_resetxlog if you
want to upgrade an existing 8.0 installation without initdb.

Pgindent run for 8.0.

Update with new typedefs.  Remove java and c++ parts of readme.

Update copyright to 2004.

Fix for postmaster.c function win32_waitpid(int *exitstatus) call to
Win32 WaitForMultipleObjects:

ret = WaitForMultipleObjects(win32_numChildren, win32_childHNDArray,
FALSE, 0);

Problem is 'win32_numChildren' could be more then 64 ( function supports
), problem basically arise ( kills postgres ) when you create more then
64 connections and terminate some of them sill leaving more then 64.

Claudio Natoli

Adjust regression expected file for new grant language hint.

Add hint about using GRANT with non-trusted languages.

James William Pye

Several Cygwin fixes pointed out by Reini Urban.

Add comment on palloc use and DLLIMPORT>

>>> I understand your disliking of non-posix stuff. OTOH,
>>GetLastError will
>>> give much more details than errno.
>>
>>How much more, really?  That mapping table gave me the impression that
>>the win32 error codes aren't all that much more detailed than errno...
>
>The mapping table is not complete. My winerror.h from the SDK
>lists 2209
>error codes, whereas errno.h lists 42...
>
>I still don't think we'll get that much more stuff. Right now,
>the Win32
>code paths that actually use the more advanced functions already write
>out the error number in case something happens. We can keep doing that
>for the other paths (ereport the error *number* when the mapping does
>not have a match). The map to errno will catch almost all cases, I
>think. And in the corner cases we can do with just the number, and use
>"net helpmsg" to get the actual message when checking...

Here's an attempt on this. new file goes in backend/port/win32.

Magnus Hagander

The attached patch improves pg_ctl's win32 service code to eliminate
some possible causes of the stale postmaster.pid problem that some users
have reported.

- The service did not properly report that it accepts
SERVICE_CONTROL_SHUTDOWN events, thus it's possible the SCM simply
killed the postmaster on shutdown.

- 'WaitHints' are now given to the SCM to prevent it timing out if
pg_ctl doesn't respond to a control event quickly enough.

- During shutdown, the service checkpoint counter is incremented every
five seconds for up to a minute to prevent the SCM timing out and
assuming the service is not responding.

Dave Page

> Am Dienstag, 17. August 2004 14:26 schrieb Fabien COELHO:
> > The patch adds missing the "libpgport.a" file to the installation under
> > "install-all-headers". It is needed by some contribs. I install the
> > library in "pkglibdir", but I was wondering whether it should be "libdir"?

Please find attached a small patch against current CVS head that fixes
pgport library installation so that it goes to libdir instead of
pkglibdir. It works for me.

Fabien Coelho

Fix high-bit comparison compiler warning in pg_dump.

Philip Warner

Add regression tests for ALTER INDEX.

Gavin Sherry

Now that TransactionIdDidAbort doesn't think it should try to modify
pg_clog, there's no reason to do abort marking of subtransactions in a
nonintuitive order.

Propogate pg_ctl -D to the postmaster for command-line identification.

Add missing Assert to make TransactionIdDidAbort more consistent with
TransactionIdDidCommit.

Remove contrib/rserv, since there are better replication solutions
available elsewhere (including a descendant project on gborg).

Rearrange order of operations in heap_drop_with_catalog and index_drop
so that we close and flush the doomed relation's relcache entry before
we start to delete the underlying catalog rows, rather than afterwards.
For awhile yesterday I thought that an unexpected relcache entry rebuild
partway through this sequence might explain the infrequent parallel
regression failures we were chasing.  It doesn't, mainly because there's
no CommandCounterIncrement in the sequence and so the deletions aren't
"really" done yet.  But it sure seems like trouble waiting to happen.

> > Another issue is that when installing the Postgres service with
> > "pg_ctl register -w ...." the "-w" parameter was not put in
> the registry "ImagePath"
> > value for the Postgres service. (I added it manually to test.) So I
> > suspect that "pg_ctl register" will need to be enhanced to add the
> > "-w" parameter to the registry settings.

Dave Page

Use dynamic buffer for token buffer in win32 admin check

Magnus Hagander

Fix relcache to account properly for subtransaction status of 'new'
relcache entries.  Also, change TransactionIdIsCurrentTransactionId()
so that if consulted during transaction abort, it will not say that
the aborted xact is still current.  (It would be better to ensure that
it's never called at all during abort, but I'm not sure we can easily
guarantee that.)  In combination, these fix a crash we have seen
occasionally during parallel regression tests of 8.0.

Further tightening of the array literal parser. Prevent junk
from being accepted after the outer right brace. Per report from
Markus Bertheau.

Also add regression test cases for this change, and for previous
recent array literal parser changes.

Can't truncate pg_subtrans during a recovery checkpoint --- subtrans
module isn't fully initialized yet.

Add missing semicolon; some bison versions warn of this.

Fix Windows emulation of kill(pid, 0).  This will now succeed, but only
if the target PID is a PG postmaster or backend --- for our purposes that
is actually better than the Unix behavior.  Per Dave Page and Andrew Dunstan.

Introduce local hash table for lock state, as per recent proposal.
PROCLOCK structs in shared memory now have only a bitmask for held
locks, rather than counts (making them 40 bytes smaller, which is a
good thing).  Multiple locks within a transaction are counted in the
local hash table instead, and we have provision for tracking which
ResourceOwner each count belongs to.  Solves recently reported problem
with memory leakage within long transactions.

Update wording:

<  This would require some background daemon to maintain clustering
>  This might require some background daemon to maintain clustering
397,398c397,398
<  paritally filled for easier reorganization.  It also might require
<  creating a merged heap/index data file so an index lookup would
>  paritally filled for easier reorganization.  Another idea would
>         be to create a merged heap/index data file so an index lookup would

Update win32 comments.

Update:

<  This would require some background daemon to restore clustering
>  This would require some background daemon to maintain clustering
397c397,399
<  paritally filled for easier reorganization.
>  paritally filled for easier reorganization.  It also might require
>  creating a merged heap/index data file so an index lookup would
>  automatically access the heap data too.

Fix user locks.  Broken some time ago for all platforms by Windows-related
changes.

Improve some comments.

New Win32 wording that mentions "code".

Update documentation to prefer CIDR format for pg_hba.conf and use new
CIDR column in examples first.

Update Win32 wording.

Add warning about Win32 bugs in this release.