tag it 8.3beta1 ... the beta cycle begins

Fix distprep and maintainer-clean targets so that ecpg's .def files are
made and removed at the correct times.

Add .cvsignore files to suppress CVS gripes about ecpg .def files.

Some desultory markup improvements in the new release notes.

Results from buildfarm show that ecpglib was depending on pg_strcasecmp,
which evidently it got as an unofficial export from pgtypeslib.

Keep the planner from failing on "WHERE false AND something IN (SELECT ...)".
eval_const_expressions simplifies this to just "WHERE false", but we have
already done pull_up_IN_clauses so the IN join will be done, or at least
planned, anyway.  The trouble case comes when the sub-SELECT is itself a join
and we decide to implement the IN by unique-ifying the sub-SELECT outputs:
with no remaining reference to the output Vars in WHERE, we won't have
propagated the Vars up to the upper join point, leading to "variable not found
in subplan target lists" error.  Fix by adding an extra scan of in_info_list
and forcing all Vars mentioned therein to be propagated up to the IN join
point.  Per bug report from Miroslav Sulc.

Stamp 8.3beta1, except in configure.in/configure.

Update timezone data files to release 2007h of the zic database.
Might as well have the latest when we wrap 8.3beta1.

DLL_DEFFILE should be defined when PORTNAME is win32, not when it isn't,
per the example of libpq/Makefile.

Replaced tabs by white spaces

Removed newline at end of exports files.

Change Assert() to a plain test and elog, just to see if that works
around the icc bug exhibited by buildfarm member dugong.

Added def-files to all: target so they are build everytime.

Two fixes for the release notes from Guillaume Smet. In passing, make
a minor markup improvement.

First pass at SGML-ifying the 8.3 release notes. Plenty of improvements
left to made, but this is a start.

Second pass at 8.3 release notes: put items in a bit more logical order
where possible, add overview and incompatibilities lists, minor cleanups.
The SGML-ification still leaves much to be desired.

Various minor improvements to the 8.3 release notes.

Teach chklocale.c how to extract encoding info from Windows locale
names.  ITAGAKI Takahiro

Some editorial improvements for recently-added ALTER SEQUENCE/VIEW
documentation.  Heikki and Tom

Suppress compiler warning in non-threaded build.

Tweak recently-added tests to suppress scary-looking warnings on 64-bit
machines about casts between pointers and integers of different sizes.
While they're harmless, we shouldn't expect users to have to go through
and figure that out for themselves.

Argh, missing dll in filename

Minor improvements to hack for old OpenSSL libraries: avoid unused
variable warning on Windows, improve comment.

Fix command for fetching snprintf.c.

This could be what's missing on some systems.

Stripped two symbols that are needed.

Attempt to open certificate file "manually" using fopen before
trying BIO functions.
Helps problem with older versions of OpenSSL that lacks error
stack functions and would show an incorrect error message for
file-not-found-or-not-openable. The problem may still exist for
other errors, but file open error is by far the most common one.

Build two more .DEF files for ecpg instead of using default ones.

Also build snprintf from pgport if needed.

Remove exports.list in clean target.

More detailed error msg (with stack trace) if a file copy fails.

Also created export list for pgytpeslib and compatlib.
Set pgtypes library version to 3.0.
Set compat library version to 3.0.

Use snprintf from libpgport in ecpg compatlib. Required since
we restricted exports from ecpglib.

Fix commandlines for final two ECPG regression tests

Fix typo

Build DEF file for libecpg instead of generating a default one.

Make ECPG regression tests run with -c only for array_of_struct.pgc
on MSVC.

Fix strange nonstandard version of __stdcall specifyer in thread
tests on win32.

Fix typo

Cleaned up ecpglib and renamed functions that do not need to be exported.
Created export list for ecpglib.

Hopefully fixed some stuff that causes Windows builds to fail.

First cut at 8.3 release notes.  SGML markup is nonexistent, and
the 'overview' and 'incompatibilities' summary lists remain to be
written.  But I think all the raw info is there (indeed maybe too
verbose).

Fix a compiler warning on Win32. Hannes Eder.

Default to thread safety on, and support more CPU options. Also
make sure that a CPU option is actually chosen.

Hiroshi Saito

Made new test also work without threading enabled.

No idea where this file came from.

ITAGAKI Takahiro <itagaki.takahiro@oss.ntt.co.jp> added thread-safe
descriptor handling

Cope with ERR_set_mark() and ERR_pop_to_mark() not existing in older
OpenSSL libraries --- just don't call them if they're not there.  This
might possibly lead to misleading error messages, but we'll just have
to live with that.

Add catalogs.sgml documentation for text search catalogs.

Use BIO functions to avoid passing FILE * pointers to OpenSSL functions.
This fixes potential crashes on old versions of OpenSSL and the requirement on
"Applink" in new versions when building with MSVC and using different
runtimes.

Dave Page with fixes from me.

Add some examples.

Add note warning against use of pre-8.4 multithreaded Tcl.

Avoid assuming that struct varattrib_pointer doesn't get padded by the
compiler --- at least on ARM, it does.  I suspect that the varvarlena patch
has been creating larger-than-intended toast pointers all along on ARM,
but it wasn't exposed until the latest tweak added some Asserts that
calculated the expected size in a different way.  We could probably have
fixed this by adding __attribute__((packed)) as is done for ItemPointerData,
but struct varattrib_pointer isn't really all that useful anyway, so it
seems cleanest to just get rid of it and have only struct varattrib_1b_e.
Per results from buildfarm member quagga.

Build with /GS on 64-bit to work with modern Platform SDK.

Hiroshi Saito

Update pgcvslog text to simply "<branch>"

Enable __FUNCTION__ on MSVC builds.

Hannes Eder

Fix a minor typo.

Fixup for new pgcvslog feature.

Have pgcvslog mark back-branch commits with "<back-patch>".

Add an extra header byte to TOAST-pointer datums to represent their size
explicitly.  This means a TOAST pointer takes 18 bytes instead of 17 --- still
smaller than in 8.2 --- which seems a good tradeoff to ensure we won't have
painted ourselves into a corner if we want to support multiple types of TOAST
pointer later on.  Per discussion with Greg Stark.

In hopes of un-breaking the buildfarm, add missing file from
ITAGAKI Takahiro's patch.

Adjust recovery PS display as agreed with Simon: 'waiting for XXX'
while the restore_command does its thing, then 'recovering XXX' while
processing the segment file.  These operations are heavyweight enough
that an extra PS display set shouldn't bother anyone.

Properly mark mergeable/hashable equality operators (found by opr_sanity
testing).  Combine the formerly independent opclasses for the various
ISN types into opfamilies.  The latter causes some extra bleating from
opr_sanity, since the module doesn't provide complete sets of cross-type
operators, but it's still a good idea because it will give the planner
more information to work with.  The missing cross-type operators no longer
pose a risk of unexpected planner errors in 8.3, so there's no need to
insist on filling them in (and I gather it wouldn't be very sound
semantically to add them all).

Support functions for index opclasses should be immutable.
Found by running opr_sanity on contrib modules.

Applied another patch by ITAGAKI Takahiro <itagaki.takahiro@oss.ntt.co.jp>
to get memory allocation thread-safe. He also did some cleaning up.

Support functions for index opclasses should be immutable.
Found by running opr_sanity on contrib modules.

Support functions for index opclasses should be immutable.
Found by running opr_sanity on contrib modules.

Support functions for index opclasses should be immutable.
Found by running opr_sanity on contrib modules.

Support functions for index opclasses should be immutable.
Found by running opr_sanity on contrib modules.

Remove bogus commutator marking --- the module doesn't actually supply
any commutator operator for =(chkpass,text), so this was creating a
shell operator that would fail on use.  Found by opr_sanity testing.

Support functions for index opclasses should be immutable.
Found by running opr_sanity on contrib modules.

Make recovery show the current input WAL segment name in the startup
process' PS display.  After a suggestion by Simon (not exactly his
patch though).

Disallow CLUSTER using an invalid index (that is, one left over from a failed
CREATE INDEX CONCURRENTLY).  Such an index might not have entries for every
heap row and thus clustering with it would result in silent data loss.
The scenario requires a pretty foolish DBA, but still ...

Improve consistency of the error messages generated when you try to use
ALTER TABLE on a composite type or ALTER TYPE on a table's rowtype.
We already rejected these cases, but the error messages were a bit
random and didn't always provide a HINT to use the other command type.

In the release checklist, mention packagers will see the minor upgrade
numbering for additional functions.

Add chklocale.c to pgport files for msvc builds, per
buildfarm failures.

Defend against openssl libraries that fail on keys longer than 128 bits;
which is the case at least on some Solaris versions.  Marko Kreen

Make archive recovery always start a new timeline, rather than only when a
recovery stop time was used.  This avoids a corner-case risk of trying to
overwrite an existing archived copy of the last WAL segment, and seems
simpler and cleaner all around than the original definition.  Per example
from Jon Colverson and subsequent analysis by Simon.

Make use of additional chklocale.c entries to reject locales that we
cannot support.

Teach chklocale.c about a few names for frontend-only encodings,
since this will allow initdb to reject attempts to initdb in a locale
that uses such an encoding.  We'll probably find out more such names
during beta ...

On OS X, assume that an empty-string result for nl_langinfo(CODESET)
means UTF-8.  Per examination of /usr/share/locale in 10.4.10.

Update TCL comment:

 * We can only fix this with Tcl >= 8.4, when Tcl_SetNotifier()

Change initdb and CREATE DATABASE to actively reject attempts to create
databases with encodings that are incompatible with the server's LC_CTYPE
locale, when we can determine that (which we can on most modern platforms,
I believe).  C/POSIX locale is compatible with all encodings, of course,
so there is still some usefulness to CREATE DATABASE's ENCODING option,
but this will insulate us against all sorts of recurring complaints
caused by mismatched settings.

I moved initdb's existing LC_CTYPE-to-encoding mapping knowledge into
a new src/port/ file so it could be shared by CREATE DATABASE.

Remove:

< * SQL*Net listener that makes PostgreSQL appear as an Oracle database
<   to clients

Tweak initdb's text search configuration selection code so it can
cope with LANG settings like 'es_ES@euro'.

exclude contrib/tsearch2 from regression tests.

Turn build and vcregress .bat files into pure one line wrappers for
the perl scripts. Remove the now superfluous getregress.pl.

Fix typos in two comments.  Spotted by Brendan Jurd

Tweak pgbench.c to remove the hidden assumption that a WIN32 machine
couldn't possibly HAVE_GETOPT.  I believe this is the most appropriate
form of the patch submitted 2007-08-07 by Hiroshi Saito, though not
having a Windows build environment I won't know for sure till I see
the buildfarm results.

Define the FRONTEND symbol in postgres_fe.h, which allows us to eliminate
duplicative -DFRONTEND flags from many Makefiles.  We still need Makefile
control of the symbol in a few places that compile frontend-or-backend
src/port/ files, but it's a lot cleaner than before.

Hiroshi Saito

Add virtual transaction IDs to CSVLOG output, so that messages coming from
the same transaction can be identified even when no regular XID was assigned.
This seems essential after addition of the lazy-XID patch.  Also some
minor code cleanup in write_csvlog().

Fix Assert failure in ExpandColumnRefStar --- what I thought was a can't
happen condition can happen given incorrect input.  The real problem is that
gram.y should try harder to distinguish * from "*" --- the latter is a legal
column name per spec, and someday we ought to treat it that way.  However
fixing that is too invasive for a back-patch, and it's too late for the 8.3
cycle too.  So just reduce the Assert to a plain elog for now.  Per report
from NikhilS.

Lowercase all mentions of "PostgreSQL" in URLs, per Magnus.

Some small tuptoaster improvements from Greg Stark.  Avoid unnecessary
decompression of an already-compressed external value when we have to copy
it; save a few cycles when a value is too short for compression; and
annotate various lines that are currently unreachable.

Minor improvements in backup and recovery:

- create a separate archive_mode GUC, on which archive_command is dependent

- %r option in recovery.conf sends last restartpoint to recovery command

- %r used in pg_standby, updated README

- minor other code cleanup in pg_standby

- doc on Warm Standby now mentions pg_standby and %r

- log_restartpoints recovery option emits LOG message at each restartpoint

- end of recovery now displays last transaction end time, as requested
  by Warren Little; also shown at each restartpoint

- restart archiver if needed to carry away WAL files at shutdown

Simon Riggs

Rename "PostgreSQL" to "Postgres" in 3 places.

Rename "PostgreSQL" to "Postgres" in 7 places in the FAQ.

Adjust the new memory limit in the lazy vacuum code to use MaxHeapTuplesPerPage
tuples per page instead of fixed 200, to better cope with systems that use a
different block size.

Create a function variable "join_search_hook" to let plugins override the
join search order portion of the planner; this is specifically intended to
simplify developing a replacement for GEQO planning.  Patch by Julius
Stroffek, editorialized on by me.  I renamed make_one_rel_by_joins to
standard_join_search and make_rels_by_joins to join_search_one_level to better
reflect their place within this scheme.

Properly pass -c parameter to ecpg when building regression tests.
Per Michael.

Applied patch by ITAGAKI Takahiro <itagaki.takahiro@oss.ntt.co.jp> to get prepare thread-safe.