Minor code cleanup: make the WIN32 case less gratuitously different from
the other platform-specific cases in ps_status.

Code review for psql multiline history patch(es).  Fix memory leak,
failure to enter commands in history if canceled by control-C, other
infelicities.

Fix Assert failure when a fastpath function call is attempted inside an
already-aborted transaction block.  GetSnapshotData throws an Assert if
not in a valid transaction; hence we mustn't attempt to set a snapshot
for the function until after checking for aborted transaction.  This is
harmless AFAICT if Asserts aren't enabled (GetSnapshotData will compute
a bogus snapshot, but it doesn't matter since HandleFunctionRequest will
throw an error shortly anywy).  Hence, not a major bug.

Along the way, add some ability to log fastpath calls when statement
logging is turned on.  This could probably stand to be improved further,
but not logging anything is clearly undesirable.

Backpatched as far as 8.0; bug doesn't exist before that.

Remove pg_dump wording:

<   pg_get_tabledef(), pg_get_domaindef(), pg_get_functiondef(), and
<   make use of them in pg_dump
>   pg_get_tabledef(), pg_get_domaindef(), pg_get_functiondef()

Add:

<   pg_get_tabledef(), pg_get_domaindef(), pg_get_functiondef()
>   pg_get_tabledef(), pg_get_domaindef(), pg_get_functiondef(), and
>   make use of them in pg_dump

Repair remarkably-inefficient query for dumping large object comments: it
was invoking obj_description() for each large object chunk, instead of once
per large object.  This code is new as of 8.1, which may explain why the
problem hadn't been noticed already.

Revert sampling patch for EXPLAIN ANALYZE; it turns out to be too unreliable
because node timing is much less predictable than the patch expects.  I kept
the API change for InstrStopNode, however.

Cleanups:

<  * Allow a database in tablespace t1 with tables created in
>  o Allow a database in tablespace t1 with tables created in
125c125
<  * Allow reporting of which objects are in which tablespaces
>  o Allow reporting of which objects are in which tablespaces
593c593
<  * Allow GRANT/REVOKE permissions to be inherited by objects based on
>  o Allow GRANT/REVOKE permissions to be inherited by objects based on
596c596
<  * Allow SERIAL sequences to inherit permissions from the base table?
>  o Allow SERIAL sequences to inherit permissions from the base table?

Cleanup:

< * -%Allow pooled connections to list all prepared statements
> * -Allow pooled connections to list all prepared statements

Now ispell dictionary can eat dictionaries in MySpell format,
used by OpenOffice. Dictionaries are placed at
http://lingucomponent.openoffice.org/spell_dic.html
Dictionary automatically recognizes format of files.

Warning. MySpell's format has limitation with compound
word support: it's impossible to mark affix as
compound-only affix. So for norwegian, german etc
languages it's recommended to use original ispell format.
For that reason I don't want to remove my2ispell
scripts, it's has workaround at least for norwegian language.

Add URL.

<    the insert.
>    the insert.
>    http://archives.postgresql.org/pgsql-patches/2005-07/msg00568.php

Update:

< * Add SQL99 WITH RECURSIVE (hierarchical) queries to SELECT
> * Add SQL:2003 WITH RECURSIVE (hierarchical) queries to SELECT

Remove:

< * Rename some /contrib modules from pg* to pg_*

Fix bootstrap.c so that database startup process and bgwriter properly release
LWLocks during a panic exit.  This avoids the possible self-deadlock pointed
out by Qingqing Zhou.  Also, I noted that an error during LoadFreeSpaceMap()
or BuildFlatFiles() would result in exit(0) which would leave the postmaster
thinking all is well.  Added a critical section to ensure such errors don't
allow startup to proceed.

Backpatched to 8.1.  The 8.0 code is a bit different and I'm not sure if the
problem exists there; given we've not seen this reported from the field, I'm
going to be conservative about backpatching any further.

Add URL.

>
>   http://archives.postgresql.org/pgsql-patches/2006-06/msg00096.php
>

Use simple URL's rather than text and a URL:

<      recovery.  See http://archives.postgresql.org/pgsql-patches/2005-04/msg00121.php.
>      recovery.
>      http://archives.postgresql.org/pgsql-patches/2005-04/msg00121.php
<   Right now only one encoding is allowed per database.  For a partial
<   patch, see http://archives.postgresql.org/pgsql-hackers/2005-03/msg00932.php.
>   Right now only one encoding is allowed per database.
>   http://archives.postgresql.org/pgsql-hackers/2005-03/msg00932.php
459c460
<   notify the protocol when a RESET CONNECTION command is used.  See
>   notify the protocol when a RESET CONNECTION command is used.
461d461
<   for a partial implementation.
515c515
<    See http://archives.postgresql.org/pgsql-patches/2006-02/msg00168.php.
>    http://archives.postgresql.org/pgsql-patches/2006-02/msg00168.php
535c535
<    See http://archives.postgresql.org/pgsql-hackers/2006-05/msg00988.php.
>    http://archives.postgresql.org/pgsql-hackers/2006-05/msg00988.php
821c821
<   See http://archives.postgresql.org/pgsql-patches/2005-07/msg00107.php.
>   http://archives.postgresql.org/pgsql-patches/2005-07/msg00107.php
877c877
<   Details at http://archives.postgresql.org/pgsql-hackers/2004-04/msg00818.php.
>   http://archives.postgresql.org/pgsql-hackers/2004-04/msg00818.php
<   See partially completed patch and additional work required at
<   http://archives.postgresql.org/pgsql-patches/2006-06/msg00025.php.
>   http://archives.postgresql.org/pgsql-patches/2006-06/msg00025.php
1297c1296
<   See http://archives.postgresql.org/pgsql-patches/2006-05/msg00040.php.
>   http://archives.postgresql.org/pgsql-patches/2006-05/msg00040.php
1311c1310,1311
<  o Improve signal handling,
>  o Improve signal handling
>
1312a1313
>

Add URL.

< * Support triggers on columns (Greg Sabino Mullane)
> * Support triggers on columns
>
>   See http://archives.postgresql.org/pgsql-patches/2005-07/msg00107.php.
>

Remove Win32 file, moved to /port.

Magnus Hagander

Remove obsolete comment about VACUUM FULL: it takes buffer content locks
now, and must do so to ensure bgwriter doesn't write a page that is in
process of being compacted.

/contrib/adminpack:  More clearly identify renaming of existing backend
functions.  I also found that pg_file_length was incorrectly documented
in the README as pg_file_size.

Spelling fix.

Robert Treat

Done:

> * -Allow server logs to be remotely read and removed using SQL commands

Add missing 3rd argument to open().

Add missing 3rd parameter to open().

Add URL for ALTER CONSTRAINT RENAME.

>
>    See http://archives.postgresql.org/pgsql-patches/2006-02/msg00168.php.
>

Add URL.

<      recovery.
>      recovery.  See http://archives.postgresql.org/pgsql-patches/2005-04/msg00121.php.

Add URL:

<   Right now only one encoding is allowed per database.
>   Right now only one encoding is allowed per database.  For a partial
>   patch, see http://archives.postgresql.org/pgsql-hackers/2005-03/msg00932.php.

Update wording:

< * Add SQL99 WITH RECURSIVE to SELECT
> * Add SQL99 WITH RECURSIVE (hierarchical) queries to SELECT

Add entry:

> * Consider allowing control of upper/lower case folding of unquoted
>   identifiers
>
>   Details at http://archives.postgresql.org/pgsql-hackers/2004-04/msg00818.php.

Allow timezone names in SQL strings,

'2006-05-24 21:11 Americas/New_York'::timestamptz

Joachim Wieland

Done:

<  o Allow timezone names in SQL strings, '2006-05-24 21:11
>  o -Allow timezone names in SQL strings, '2006-05-24 21:11

Prepare code to be built by MSVC:

o  remove many WIN32_CLIENT_ONLY defines
o  add WIN32_ONLY_COMPILER define
o  add 3rd argument to open() for portability
o  add include/port/win32_msvc directory for
   system includes

Magnus Hagander

Spelling fix.

Robert Treat

Mention GPL /contrib modules:

>
>   Particularly, move GPL-licensed /contrib/userlock and
>   /contrib/dbmirror/clean_pending.pl.
>

Per previous analysis, the most correct notion of SampleOverhead is that
it is just the total time to do INSTR_TIME_SET_CURRENT(), and not any of
the other code involved in InstrStartNode/InstrStopNode.  Even though I
fear we may end up reverting this patch altogether, we may as well have
the most correct version in our CVS archive.

Have REINDEX use NOTICE, so it can be suppressed.

Remove "fuzzy comparison" logic in qsort comparison function for
choose_bitmap_and().  It was way too fuzzy --- per comment, it was meant to be
1% relative difference, but was actually coded as 0.01 absolute difference,
thus causing selectivities of say 0.001 and 0.000000000001 to be treated as
equal.  I believe this thinko explains Maxim Boguk's recent complaint.  While
we could change it to a relative test coded like compare_fuzzy_path_costs(),
there's a bigger problem here, which is that any fuzziness at all renders the
comparison function non-transitive, which could confuse qsort() to the point
of delivering completely wrong results.  So forget the whole thing and just
do an exact comparison.

For psql multi-line history, save backslash commands right away, rather
than only if there is already history.

Use INFO-level for reindex messages, rather than NOTICE.

Euler Taveira de Oliveira

Update comment.

On Win32, return original patch if GetShortPathName() fails (no short
name, path does not exist), rather than returning nothing.

Backpatch to 8.1.X.

Make the planner estimate costs for nestloop inner indexscans on the basis
that the Mackert-Lohmann formula applies across all the repetitions of the
nestloop, not just each scan independently.  We use the M-L formula to
estimate the number of pages fetched from the index as well as from the table;
that isn't what it was designed for, but it seems reasonably applicable
anyway.  This makes large numbers of repetitions look much cheaper than
before, which accords with many reports we've received of overestimation
of the cost of a nestloop.  Also, change the index access cost model to
charge random_page_cost per index leaf page touched, while explicitly
not counting anything for access to metapage or upper tree pages.  This
may all need tweaking after we get some field experience, but in simple
tests it seems to be giving saner results than before.  The main thing
is to get the infrastructure in place to let cost_index() and amcostestimate
functions take repeated scans into account at all.  Per my recent proposal.

Note: this patch changes pg_proc.h, but I did not force initdb because
the changes are basically cosmetic --- the system does not look into
pg_proc to decide how to call an index amcostestimate function, and
there's no way to call such a function from SQL at all.

Add URL to RESET CONNECTION:

<   notify the protocol when a RESET CONNECTION command is used.
>   notify the protocol when a RESET CONNECTION command is used.  See
>   http://archives.postgresql.org/pgsql-patches/2006-04/msg00192.php
>   for a partial implementation.

Allow do not lexize words in substitution.
 Docs will be submitted some later, now it's at
 http://www.sai.msu.su/~megera/oddmuse/index.cgi/Thesaurus_dictionary

Remove ancient AIX structure workaround.

Have BCC echo output "" using "", rather than \"\" as earlier reported.
Backpatch to 8.1.X.

Done:

< * %Make postmater and postgres options distinct so the postmaster -o
> * -Make postmater and postgres options distinct so the postmaster -o

Fixed two more memory leaks in ecpglib.
Synced parser.

Remove extremely old, incomplete, broken example code.
Per my proposal a few days ago.

While making the seq_page_cost changes, I was struck by the fact that
cost_nonsequential_access() is really totally inappropriate for its only
remaining use, namely estimating I/O costs in cost_sort().  The routine
was designed on the assumption that disk caching might eliminate the need
for some re-reads on a random basis, but there's nothing very random in
that sense about sort's access pattern --- it'll always be picking up the
oldest outputs.  If we had a good fix on the effective cache size we
might consider charging zero for I/O unless the sort temp file size
exceeds it, but that's probably putting much too much faith in the
parameter.  Instead just drop the logic in favor of a fixed compromise
between seq_page_cost and random_page_cost per page of sort I/O.

Avoid longjump/vfork warning about line_saved_in_history variable by
making it volatile.

Have psql history save \e editor contents to history before query is
executed, like it did for 8.1.X.

Increase the default value of cpu_index_tuple_cost from 0.001 to 0.005.
This shouldn't affect simple indexscans much, while for bitmap scans that
are touching a lot of index rows, this seems to bring the estimates more
in line with reality.  Per recent discussion.

Add a GUC parameter seq_page_cost, and use that everywhere we formerly
assumed that a sequential page fetch has cost 1.0.  This patch doesn't
in itself change the system's behavior at all, but it opens the door to
people adopting other units of measurement for EXPLAIN costs.  Also, if
we ever decide it's worth inventing per-tablespace access cost settings,
this change provides a workable intellectual framework for that.

Fix psql so \r is separate in the  history, rather than at the end of
the query.

Update:

< * %Remove behavior of postmaster -o
> * %Make postmater and postgres options distinct so the postmaster -o
>   option is no longer needed

Update:

<  o Allow COPY to output from views
>  o Allow COPY to output from SELECT
570c570
<    Another idea would be to allow actual SELECT statements in a COPY.
>    COPY should also be able to output views.

Update GUI FAQ.

Robert Treat

Update GUI tools URL.

Robert Treat

Don't choke during startup if the environment offers an invalid value
for LC_MESSAGES; instead, just press forward, leaving the effective setting
at 'C'.  There is not any very good reason to complain when we are going
to replace the value soon with whatever postgresql.conf says.  This change
should solve the occasionally-reported problem of initdb failing with
'failed to initialize lc_messages'; the current theory is that that is
a reflection of either wrong LANG/LC_MESSAGES or completely broken locale
support.

Fix copy-and-pasteo in Russian translation: message complaining about
HAVE_INT64_TIMESTAMP was mentioning PG_CONTROL_VERSION instead.
Victor Snezhko

Record location of partial patch :

>  * Allow WAL information to recover corrupted pg_controldata
>
>   See partially completed patch and additional work required at
>   http://archives.postgresql.org/pgsql-patches/2006-06/msg00025.php.
>

Mention pg_encoding_to_char in entry for pg_database.encoding.
Per Larry Rosenman.

Patch reverted, not done:

> * Allow WAL information to recover corrupted pg_controldata

Remove pg_resetxlog -r flag, feature has problems referenced here:

http://archives.postgresql.org/pgsql-hackers/2006-05/msg00002.php

Fix wrong NOTICE/ERROR levels

Distinguish between stop-word recognized in thesaurus_lexize()

Add more strict check of stop and non-recognized words,
allow only recognized words in thezaurus configuration file.

Got tired of waiting for spoonbill's compiler to get fixed.  Let's
see if using an intermediate variable avoids the gcc bug.

Respond to Jeremy Drake's original gripe that \copy needs to recognize
E'...' syntax for strings in order to track the backend.

Fix up hack to suppress escape_string_warning so that it actually works
and there's only one place that's a kluge, ie, appendStringLiteralConn.
Note that pg_dump itself doesn't use appendStringLiteralConn, so its
behavior is not affected; only the other utility programs care.

Fix example of how to escape data in psql backslash commands.

Fix seriously broken patch for psql '' ... per its comment, emit() is
NOT the thing to use here.

Make PG_MODULE_MAGIC required in shared libraries that are loaded into
the server.  Per discussion, there seems no point in a waiting period
before making this required.

fix comparison with SPI_processed

Add thesaurus dictionary which can replace N>0 lexemes by M>0 lexemes.
It required some changes in lexize algorithm, but interface with
dictionaries stays compatible with old dictionaries.

Funded by Georgia Public Library Service and LibLime, Inc.

Recommend '' for literal ', rather than \', in psql documentation.

Support '' for literal ' in psql single-quote strings, documentation update.

Escape processing patch:

        o  turns off escape_string_warning in pg_dumpall.c
        o  optionally use E'' for \password (undocumented option?)
        o  honor standard_conforming-strings for \copy (but not
           support literal E'' strings)
        o  optionally use E'' for \d commands
        o  turn off escape_string_warning for createdb, createuser,
           droplang

Update cvs command example to show diff -rBASE.

Somehow a ";" got lost which changed the logic. This btw is the first fix resulting from SoC.

Magic blocks don't do us any good unless we use 'em ... so install one
in every shared library.

Un-DOS-ify newly added files.

Code review for magic-block patch.  Remove separate header file pgmagic.h,
as this seems only likely to create headaches for module developers.  Put
the macro in the pre-existing fmgr.h file instead.  Avoid being too cute
about how many fields we can cram into a word, and avoid trying to fetch
from a library we've already unlinked.
Along the way, it occurred to me that the magic block really ought to be
'const' so it can be stored in the program text area.  Do the same for
the existing data blocks for PG_FUNCTION_INFO_V1 functions.

Code review for EXPLAIN patch.  Fix some typos, make it behave sanely
across multiple loops, get rid of the shaky assumption that exactly one
tuple is returned per node iteration.

PostmasterIsAlive test really ought to be in the inner loop for safety.

Remove pqsignalinquire(), which is unused and has portability issues.

Update ppport.h to not cause warnings with newest Perl versions.
This is just the minimal necessary change; we might want to adopt
later PPPort output instead.

Add pgmagic header block to store compile-time constants:

It now only checks four things:

Major version number (7.4 or 8.1 for example)
NAMEDATALEN
FUNC_MAX_ARGS
INDEX_MAX_KEYS

The three constants were chosen because:

1. We document them in the config page in the docs
2. We mark them as changable in pg_config_manual.h
3. Changing any of these will break some of the more popular modules:

FUNC_MAX_ARGS changes fmgr interface, every module uses this NAMEDATALEN
changes syscache interface, every PL as well as tsearch uses this
INDEX_MAX_KEYS breaks tsearch and anything using GiST.

Martijn van Oosterhout

Make EXPLAIN sampling smarter, to avoid excessive sampling delay.

Martijn van Oosterhout

Clarify the HINT for 'checkpoint request failed', per recent complaint
demonstrating that its intent wasn't obvious.

Emit warnings for unknown configure options.

Martijn van Oosterhout

Revert patch, needs more work:

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

Add dynamic record inspection to PL/PgSQL, useful for generic triggers:

  tval2 := r.(cname);

or

  columns := r.(*);

Titus von Boxberg

Move conversion dependency patch to the proper branch, out of 8.1.X,
into HEAD.

Prevent multiple archivers from starting.  Backpatch to 8.1.X.

Simon Riggs

Add /contrib/adminpack to contrib/Makefile, do case-folding change.

Fix printf mask for SizeVfdCache

Qingqing Zhou

Re-defines SHA2 symbols so that they would not conflict with certain
versions of OpenSSL.  If your OpenSSL does not contain SHA2, then there
should be no conflict.  But ofcourse, if someone upgrades OpenSSL,
server starts crashing.

Backpatched to 8.1.X.

Marko Kreen

Add "inline" compile fix for MSVC/BCC:

#define inline __inline

Backpatch to 8.1.X.

Hiroshi Saito

Re-add btree_gist description.