Add ALTER INDEX, particularly for moving tablespaces.

Gavin Sherry

> Please find attached a submission to add a "exit on error" option to
> pg_restore, as it seems that some people have scripts that rely on the
> previous "abort on error" default behavior when restoring data with a
> direct connection.
>
> Fabien Coelho

> After all that about numbering centuries and millenia correctly,
> why does CVS tip still give me
>
> regression=# select extract(century from now());
>  date_part
> -----------
>         20
> (1 row)
> [ ... looks in code ... ]
>
> Apparently it's because you fixed only timestamp_part, and not
> timestamptz_part.  I'm not too sure about what timestamp_trunc or
> timestamptz_trunc should do, but they may be wrong as well.

Sigh... as usual, what is not tested does not work:-(

> Could we have a more complete patch?

Please find a submission attached. I hope it really fixes all decade,
century and millenium issues for extract and *_trunc functions on
interval
and other timestamp types. If someone could check that the results
are reasonnable, it would be great.

I indeed overlooked the fact that there were two functions. The patch
fixes the code so that both variants agree.

I added comments to interval extractions, because it relies on the C
division to have a negative remainder: -7/10 = 0 and remains -7.

As for *_trunc functions, I have chosen to put the first year of the
century or millennium: -100, 1, 101... 1001 2001 etc. Indeed, I don't
think it would make sense to put 2000 (last year of the 2nd millennium)
for rounding all years of the third millenium.

I also fixed the code so that all decades last 10 years and decade 199
means the 1990's.

I have added some tests that are relevant to deal with tricky cases. The
formula may be simplified, but all these cases must pass. Please keep
them.

Fabien Coelho

Add:

> * Add a function that returns the 'uptime' of the postmaster

Update wording on commented postgresql.conf values:

>   Another option is to allow commented values to return to their
>   default values.

Repair some issues with column aliases and RowExpr construction in the
presence of dropped columns.  Document the already-presumed fact that
eref aliases in relation RTEs are supposed to have entries for dropped
columns; cause the user alias structs to have such entries too, so that
there's always a one-to-one mapping to the underlying physical attnums.
Adjust expandRTE() and related code to handle the case where a column
that is part of a JOIN has been dropped.  Generalize expandRTE()'s API
so that it can be used in a couple of places that formerly rolled their
own implementation of the same logic.  Fix ruleutils.c to suppress
display of aliases for columns that were dropped since the rule was made.

Use backslash in %USERPROFILE%\.pgpass.

Add:

>  o Allow COPY to optionally include column headings as the first line

Don't use get_home_path so libpq doesn't pull path.c into the library.

Clean up some random departures from project's standard declaration style.

Remove NT4 mention in release notes.

Remove as duplicate:

<  o Allow pg_dumpall to use non-text output formats

Add:

> * Move some /contrib modules out to their own project sites

The enclose patch clarifies and makes a more useful example for the
Global Values in PL/Perl section of the documents.

David Fetter

Add pg_dump section:

< * -Allow pg_dump to dump CREATE CONVERSION (Christopher)
< * -Make pg_restore continue after errors, so it acts more like pg_dump scripts
485,486d482
< * Allow pg_dumpall to use non-text output formats
< * Have pg_dump use multi-statement transactions for INSERT dumps
493,496d488
< * Allow pg_dump to use multiple -t and -n switches
<
<   This should be done by allowing a '-t schema.table' syntax.
<
498a491,512
>
> * pg_dump
>  o Allow pg_dumpall to use non-text output formats
>  o Have pg_dump use multi-statement transactions for INSERT dumps
>  o -Allow pg_dump to dump CREATE CONVERSION (Christopher)
>  o -Make pg_restore continue after errors, so it acts more like pg_dump
>    scripts
>  o Allow pg_dump to use multiple -t and -n switches
>
>    This should be done by allowing a '-t schema.table' syntax.
>
>  o Add dumping of comments on composite type columns
>  o Add dumping of comments on index columns
>  o Replace crude DELETE FROM method of pg_dumpall for cleaning of
>    users and groups with separate DROP commands
>  o Add dumping and restoring of LOB comments
>  o Stop dumping CASCADE on DROP TYPE commands in clean mode
>  o Add full object name to the tag field.  eg. for operators we need
>    '=(integer, integer)', instead of just '='.
>  o Add pg_dumpall custom format dumps. This is probably best done by
>    combining pg_dump and pg_dumpall into a single binary
>  o Add CSV output format

Replace incorrect example for quote_ident

Greg Sabino Mullan

Update release history for 7.4.5.

Add get_home_path() to use USERPROFILE on Win32 and HOME on Unix.

Fix bug introduced into _bt_getstackbuf() on 2003-Feb-21: the initial
value of 'start' could be past the end of the page, if the page was
split by some concurrent inserting process since we visited it.  In
this situation the code could look at bogus entries and possibly find
a match (since after all those entries still contain what they had
before the split).  This would lead to 'specified item offset is too large'
followed by 'PANIC: failed to add item to the page', as reported by Joe
Conway for scenarios involving heavy concurrent insertion activity.

Standardize on the assumption that the arguments of a RowExpr correspond
to the physical layout of the rowtype, ie, there are dummy arguments
corresponding to any dropped columns in the rowtype.  We formerly had a
couple of places that did it this way and several others that did not.
Fixes Gaetano Mendola's "cache lookup failed for type 0" bug of 5-Aug.

Add:

< * -Allow savepoints / nested transactions [transactions] (Alvaro)
> * -Allow savepoints / nested transactions (Alvaro)
348a349,353
> * Add an option to automatically use savepoints for each statement in a
>   multi-statement transaction.
>
>   When enabled, this would allow errors in multi-statement transactions
>   to be automatically ignored.

Remove transactions TODO.detail.

Chain on to SIGPIPE handler rather than just do action on default.

Always create thread-specific variable.

Fix agressive collection of thread flags.

Add comment about portability function name inconsistency.

Fix linking problem when enabling thread safety on Darwin: uninitialized
global variables are problematic on this platform.  Simplest solution
seems to be to initialize pthread key variable to 0.  Also, rename this
variable and check_sigpipe_handler to something involving "pq" to
avoid gratuitous pollution of application namespace.

Add DLLIMPORT for PostGIS.

Move io.h include higher in the file so it doesn't conflict with
rename/unlink defined later.  Problem exists on MS VC.

Andrew Francis

Fix syntax error just introduced.

autoconf for recent thread check changes.

Add trivial NULL statement to plpgsql, for Oracle compatibility.

Add:

>  o Add ALTER INDEX syntax to work like ALTER TABLE indexname

Throw error if initdb -L is not an absolute path.

Add:

> * Set proper permissions on non-system schemas during db creation
>
>   Currently all schemas are owned by the super-user because they are
>   copied from the template1 database.
>

Fix for adding \n for zero-length win32 read_pipe return

Mark server_encoding and integer_datetimes as GUC_REPORT, per previous
proposals by Oliver Jowett.  Update documentation.

Clarify need for \r\n -> \n translation in version checking code.

Dept. of further reflection: I looked around to see if any other callers
of XLogInsert had the same sort of checkpoint interlock problem as
RecordTransactionCommit, and indeed I found some.  Btree index build
and ALTER TABLE SET TABLESPACE write data outside the friendly confines
of the buffer manager, and therefore they have to take their own
responsibility for checkpoint interlock.  The easiest solution seems to
be to force smgrimmedsync at the end of the index build or table copy,
even when the operation is being WAL-logged.  This is sufficient since
the new index or table will be of interest to no one if we don't get
as far as committing the current transaction.

Xmin calculations should consider only top transaction IDs, and
therefore starting with GetCurrentTransactionId is wrong.  Fixes
miscomputation of RecentGlobalXmin leading to bizarre behavior
reported by Gavin Sherry.

Specify SA_NOCLDSTOP when enabling SIGCHLD, per suggestion from
Oliver Jowett.

Quote PERL expansion for Win32 path that might have spaces.

Fix trivial typo.

Minor copy-editing for 7.4.4, 7.3.7, 7.2.5 release notes.

Fix typo, per Andreas Seltenreich.

Add discussion and example about predicate locking and why "serializable"
mode isn't really serializable.  I had thought this was covered already
in our docs, but I sure can't find it.

Minor editorializing.

Translation updates

Add:

>
> * Allow buffered WAL writes and fsync
>
>   Instead of guaranteeing recovery of all committed transactions, this
>   would provide improved performance by delaying WAL writes and fsync
>   so an abrupt operating system restart might lose a few seconds of
>   committed transactions but still be consistent.  We could perhaps
>   remove the 'fsync' parameter (which results in an an inconsistent
>   database) in favor of this capability.

Fix psql's COPY support to deal with \r\n line endings.
Andrew Dunstan, some further hacking by Tom Lane.

Fix breakage with PUBLIC schema.  Try to untwist the remarkably contorted
logic a little bit.

Give a more specific error message for "you can't do that" error cases
in plpgsql, particularly trying to begin/end/rollback a transaction.

Preliminary release notes for 7.4.4, 7.3.7, 7.2.5.  Will add to the
back branches later.

New translations

Translation update

Translation updates

Mention log_statement_stats can not be enabled with the other options.

Clarify trigger function return convention, per Thomas Hallgren.

Fix core dumps, inability to count, etc associated with canonicalize_path
patches.

Change order of operations in ALTER TABLE SET TABLESPACE so that we
don't hold an open file reference to the original table at the end.
This is a good thing in any case, particularly so on Windows which
cannot drop the table file otherwise.

Mention the installer doesn't run on NT4.

Add:

>  o Add ALTER INDEX that works just like ALTER TABLE already does
>    on an index

Allow commas in BEGIN, START TRANSACTION, and SET TRANSACTION, as required
by the SQL standard.  For backwards compatibility, however, continue to
accept the syntax without.  Minor editorialization in the reference pages
for these commands, too.

Add:

> * Allow finer control over the caching of prepared query plans
>
>   Currently, queries prepared via the libpq API are planned on first
>   execute using the supplied parameters --- allow SQL PREPARE to do the
>   same.  Also, allow control over replanning prepared queries either
>   manually or automatically when statistics for execute parameters
>   differ dramatically from those used during planning.
>

dbsize modification to support tablespaces

Gavin Sherry

Allow optional SAVEPOINT keyword in RELEASE and ROLLBACK TO, for greater
compliance with SQL2003 spec syntax.

Oliver Jowett

Cleanup Win32 COPY handling, and move archive examples to SGML.

Add mention of "WIN32" COPY.

Add make_native_path() because Win32 COPY is an internal CMD.EXE command
and doesn't process forward slashes in the same way as external
commands.  Quoting the first argument to COPY does not convert forward
to backward slashes, but COPY does properly process quoted forward
slashes in the second argument.

Win32 COPY works with quoted forward slashes in the first argument only if the
current directory is the same as the directory of the first argument.

Be more aggressive about adding flags to thread compiles.  The configure
test only tests for building a binary, not building a shared library.

On Linux, you can build a binary with -pthread, but you can't build a
binary that uses a threaded shared library unless you also use -pthread
when building the binary, or adding -lpthread to the shared library
build.  This patch has the effect of doing the later by adding both
-pthread and -lpthread when building libpq.

Minor style cleanup of thread test script.

Cause initdb to actually accept -s as intended, and fix some typos in
a comment.

Jon Jensen

Fix broken parsing of CREATE TABLE AS (looks like a mis-sync with main
parser).

Avoid crashing when restoring a saved GUC session_authorization value
that refers to a now-deleted userid.  Per gripe from Chris Ochs.

Add PQserverVersion() to libpq to provide more-convenient access to
the server version number.  This commit also removes bogus DOS line
endings from libpqddll.def.

Greg Sabino Mullane

Work around broken strtod() that's present in many Solaris releases.
Thanks to Michael Fuhr for identifying the problem.

Fix function definition that somehow missed being ANSI-fied, and align
it with previous prototype to suppress complaints from picky compilers,
per report from Scott Bailey.  Also, remove substitute strerror
definition --- not needed, since we link this with libpgport.

Allow compilation when CODESET is not defined (OpenBSD).

Translation update

Add translation from 7.4 branch to head.

Fix failure to guarantee that a checkpoint will write out pg_clog updates
for transaction commits that occurred just before the checkpoint.  This is
an EXTREMELY serious bug --- kudos to Satoshi Okada for creating a
reproducible test case to prove its existence.

Hashed crosstab was dying with an SPI_finish error when the source SQL
produced no rows. Now it returns 0 rows instead. Adjusted regression
test for this case.

Set DYLD_LIBRARY_PATH so that 'make check' works without prior
'make install' on OS X.  Per suggestion from Adam Witney.

New translation

Translation updates

Translation update

Translation updates

Translation update

Update DELETE FROM:

< * Allow DELETE to handle table aliases for self-joins
> * Allow an alias to be provided for the target table in UPDATE/DELETE
276,279c276,282
<   There is no way to create a table alias for the deleted table for use
<   in the DELETE WHERE clause.  The agreed approach is to allow a USING
<   clause to specify additional tables.  UPDATE already has an optional
<   FROM clause for this purpose.
>   This is not SQL-spec but many DBMSs allow it.
>
> * Allow additional tables to be specified in DELETE for joins
>
>   UPDATE already allows this (UPDATE...FROM) but we need similar
>   functionality in DELETE.  It's been agreed that the keyword should
>   be USING, to avoid anything as confusing as DELETE FROM a FROM b.

Update DELETE FROM TODO info.

Add:

* Add COMMENT ON for all cluster global objects (users, groups,
  databases and tablespaces)

Update documentation to reflect the fact that we now know exactly what
time zone names we support.

Removed, shared table:

< * Add COMMENT for tablespaces

Add:

> * Add COMMENT for tablespaces

syslogger.c needs <sys/time.h> on some platforms, per Greg Mullane.

Path-mangling logic was failing to account for paths containing mentions
of '.' or '..'.  Extend canonicalize_path() to trim off trailing occurrences
of these things, and use it to fix up paths where needed (which I think is
only after places where we trim the last path component, but maybe some
others will turn up).  Fixes Josh's complaint that './initdb' does not
work.

When expanding %p in archive_command or restore_command, translate
slashes to backslashes #ifdef WIN32.  This is to cope with the fact
that Windows seems exceedingly unfriendly to slashes in shell commands,
as per recent discussion.

Remove no-longer-relevant information about trying to force your OS
to support PST8PDT time zone for the regression tests.

Fix Win32 pg_dumpall check.

Improve comment.

Stamp libpq.rc with 8.0 version.