]> granicus.if.org Git - postgresql/log
postgresql
12 years agoIn ecpg removed old leftover check for given connection name.
Michael Meskes [Sun, 18 Dec 2011 14:34:33 +0000 (15:34 +0100)]
In ecpg removed old leftover check for given connection name.

Ever since we introduced real prepared statements this should work for
different connections. The old solution just emulating prepared statements,
though, wasn't able to handle this.

Closes: #6309
12 years agoRemove tabs in SGML file.
Bruce Momjian [Sun, 18 Dec 2011 13:32:08 +0000 (08:32 -0500)]
Remove tabs in SGML file.

12 years agoDefend against null scankeys in spgist searches.
Tom Lane [Sun, 18 Dec 2011 00:08:28 +0000 (19:08 -0500)]
Defend against null scankeys in spgist searches.

Should've thought of that one earlier.

12 years agoRemove bogus entries in gist point_ops operator class.
Tom Lane [Sat, 17 Dec 2011 23:51:00 +0000 (18:51 -0500)]
Remove bogus entries in gist point_ops operator class.

These entries could never be matched to an index clause because they don't
have the index datatype on the left-hand side of the operator.  (Their
commutators are in the opclass, which is sensible, but that doesn't mean
these operators should be.)  Spotted by a test that I recently added to
opr_sanity to catch exactly this type of thinko.  AFAICT there is no code
in gistproc.c that is specifically meant to cover these cases, so nothing
to remove at that level.

12 years agoFix some long-obsolete references to XLogOpenRelation.
Tom Lane [Sat, 17 Dec 2011 23:26:52 +0000 (18:26 -0500)]
Fix some long-obsolete references to XLogOpenRelation.

These were missed in commit a213f1ee6c5a1bbe1f074ca201975e76ad2ed50c,
which removed that function.

12 years agoFix compiler warning seen on 64-bit machine.
Tom Lane [Sat, 17 Dec 2011 21:51:36 +0000 (16:51 -0500)]
Fix compiler warning seen on 64-bit machine.

12 years agoAdd SP-GiST (space-partitioned GiST) index access method.
Tom Lane [Sat, 17 Dec 2011 21:41:16 +0000 (16:41 -0500)]
Add SP-GiST (space-partitioned GiST) index access method.

SP-GiST is comparable to GiST in flexibility, but supports non-balanced
partitioned search structures rather than balanced trees.  As described at
PGCon 2011, this new indexing structure can beat GiST in both index build
time and query speed for search problems that it is well matched to.

There are a number of areas that could still use improvement, but at this
point the code seems committable.

Teodor Sigaev and Oleg Bartunov, with considerable revisions by Tom Lane

12 years agoSame clarification for pg_restore.
Andrew Dunstan [Sat, 17 Dec 2011 18:12:01 +0000 (13:12 -0500)]
Same clarification for pg_restore.

12 years agoClarify the post-data status on unvalidated check constraints.
Andrew Dunstan [Sat, 17 Dec 2011 17:38:40 +0000 (12:38 -0500)]
Clarify the post-data status on unvalidated check constraints.

Per gripe from Thom Brown.

12 years agoAdd missing 'static' qualifier.
Tom Lane [Sat, 17 Dec 2011 04:19:06 +0000 (23:19 -0500)]
Add missing 'static' qualifier.

12 years agoVarious micro-optimizations for GetSnapshopData().
Robert Haas [Sat, 17 Dec 2011 02:44:26 +0000 (21:44 -0500)]
Various micro-optimizations for GetSnapshopData().

Heikki Linnakangas had the idea of rearranging GetSnapshotData to
avoid checking for sub-XIDs when no top-level XID is present.  This
patch does that plus further a bit of further, related rearrangement.
Benchmarking show a significant improvement on unlogged tables at
higher concurrency levels, and mostly indifferent result on permanent
tables (which are presumably bottlenecked elsewhere).  Most of the
benefit seems to come from using the new NormalTransactionIdPrecedes()
macro rather than the function call TransactionIdPrecedes().

12 years agoAdd --section option to pg_dump and pg_restore.
Andrew Dunstan [Sat, 17 Dec 2011 00:09:38 +0000 (19:09 -0500)]
Add --section option to pg_dump and pg_restore.

Valid values are --pre-data, data and post-data. The option can be
given more than once. --schema-only is equivalent to
--section=pre-data --section=post-data. --data-only is equivalent
to --section=data.

Andrew Dunstan, reviewed by Joachim Wieland and Josh Berkus.

12 years agoFix reference to "verify-ca" and "verify-full" in a note in the docs.
Heikki Linnakangas [Fri, 16 Dec 2011 13:03:36 +0000 (15:03 +0200)]
Fix reference to "verify-ca" and "verify-full" in a note in the docs.

12 years agoinclude_if_exists facility for config file.
Andrew Dunstan [Fri, 16 Dec 2011 00:40:58 +0000 (19:40 -0500)]
include_if_exists facility for config file.

This works the same as include, except that an error is not thrown
if the file is missing. Instead the fact that it's missing is
logged.

Greg Smith, reviewed by Euler Taveira de Oliveira.

12 years agoImprove behavior of concurrent ALTER <relation> .. SET SCHEMA.
Robert Haas [Fri, 16 Dec 2011 00:02:58 +0000 (19:02 -0500)]
Improve behavior of concurrent ALTER <relation> .. SET SCHEMA.

If the referrent of a name changes while we're waiting for the lock,
we must recheck permissons.  We also now check the relkind before
locking, since it's easy to do that long the way.

Patch by me; review by Noah Misch.

12 years agoImprove behavior of concurrent rename statements.
Robert Haas [Thu, 15 Dec 2011 23:51:46 +0000 (18:51 -0500)]
Improve behavior of concurrent rename statements.

Previously, renaming a table, sequence, view, index, foreign table,
column, or trigger checked permissions before locking the object, which
meant that if permissions were revoked during the lock wait, we would
still allow the operation.  Similarly, if the original object is dropped
and a new one with the same name is created, the operation will be allowed
if we had permissions on the old object; the permissions on the new
object don't matter.  All this is now fixed.

Along the way, attempting to rename a trigger on a foreign table now gives
the same error message as trying to create one there in the first place
(i.e. that it's not a table or view) rather than simply stating that no
trigger by that name exists.

Patch by me; review by Noah Misch.

12 years agoDon't leave regress_test_role_super lying around.
Robert Haas [Thu, 15 Dec 2011 23:45:02 +0000 (18:45 -0500)]
Don't leave regress_test_role_super lying around.

Fixes an oversight in commit fc6d1006bda783cc002c61a5f072905849dbde4b.

Noted by Tom Lane.

12 years agoFix typo.
Robert Haas [Thu, 15 Dec 2011 23:22:29 +0000 (18:22 -0500)]
Fix typo.

12 years agoPL/Python: Refactor subtransaction handling
Peter Eisentraut [Thu, 15 Dec 2011 14:52:57 +0000 (16:52 +0200)]
PL/Python: Refactor subtransaction handling

Lots of repetitive code was moved into new functions
PLy_spi_subtransaction_{begin,commit,abort}.

Jan Urbański

12 years agoDisable excessive FP optimization by recent versions of gcc.
Andrew Dunstan [Wed, 14 Dec 2011 22:15:24 +0000 (17:15 -0500)]
Disable excessive FP optimization by recent versions of gcc.

Suggested solution from Tom Lane. Problem discovered, probably not
for the first time, while testing the mingw-w64 32 bit compiler.

Backpatched to all live branches.

12 years agoFix docs build I inadvertantly broke.
Andrew Dunstan [Wed, 14 Dec 2011 21:49:20 +0000 (16:49 -0500)]
Fix docs build I inadvertantly broke.

12 years agoAdd --exclude-table-data option to pg_dump.
Andrew Dunstan [Wed, 14 Dec 2011 14:23:17 +0000 (09:23 -0500)]
Add --exclude-table-data option to pg_dump.

Andrew Dunstan, reviewed by Josh Berkus, Robert Haas and Peter Geoghegan.

This allows dumping of a table definition but not its data, on a per table basis.
Table name patterns are supported just as for --exclude-table.

12 years agoAdd support for passing cursor parameters in named notation in PL/pgSQL.
Heikki Linnakangas [Wed, 14 Dec 2011 13:55:37 +0000 (15:55 +0200)]
Add support for passing cursor parameters in named notation in PL/pgSQL.

Yeb Havinga, reviewed by Kevin Grittner, with small changes by me.

12 years agoMove BKP_REMOVABLE bit from individual WAL records to WAL page headers.
Tom Lane [Mon, 12 Dec 2011 21:22:14 +0000 (16:22 -0500)]
Move BKP_REMOVABLE bit from individual WAL records to WAL page headers.

Removing this bit from xl_info allows us to restore the old limit of four
(not three) separate pages touched by a WAL record, which is needed for the
upcoming SP-GiST feature, and will likely be useful elsewhere in future.

When we implemented XLR_BKP_REMOVABLE in 2007, we had to do it like that
because no special WAL-visible action was taken when starting a backup.
However, now we force a segment switch when starting a backup, so a
compressing WAL archiver (such as pglesslog) that uses the state shown in
the current page header will not be fooled as to removability of backup
blocks.  The only downside is that the archiver will not return to
compressing mode for up to one WAL page after the backup is over, which is
a small price to pay for getting back the extra xl_info bit.  In any case
the archiver could look for XLOG_BACKUP_END records if it thought it was
worth the trouble to do so.

Bump XLOG_PAGE_MAGIC since this is effectively a change in WAL format.

12 years agoRevert the behavior of inet/cidr functions to not unpack the arguments.
Heikki Linnakangas [Mon, 12 Dec 2011 07:49:47 +0000 (09:49 +0200)]
Revert the behavior of inet/cidr functions to not unpack the arguments.

I forgot to change the functions to use the PG_GETARG_INET_PP() macro,
when I changed DatumGetInetP() to unpack the datum, like Datum*P macros
usually do. Also, I screwed up the definition of the PG_GETARG_INET_PP()
macro, and didn't notice because it wasn't used.

This fixes the memory leak when sorting inet values, as reported
by Jochen Erwied and debugged by Andres Freund. Backpatch to 8.3, like
the previous patch that broke it.

12 years agoMiscellaneous cleanup to silence compiler warnings seen on Mingw.
Andrew Dunstan [Sat, 10 Dec 2011 23:15:15 +0000 (18:15 -0500)]
Miscellaneous cleanup to silence compiler warnings seen on Mingw.

Remove some dead code, conditionally declare some items or call
some code, and fix one or two declarations.

12 years agoRemove define inadvertantly left over from testing.
Andrew Dunstan [Sat, 10 Dec 2011 21:29:37 +0000 (16:29 -0500)]
Remove define inadvertantly left over from testing.

12 years agoEnable compiling with the mingw-w64 32 bit compiler.
Andrew Dunstan [Sat, 10 Dec 2011 20:35:41 +0000 (15:35 -0500)]
Enable compiling with the mingw-w64 32 bit compiler.

Original patch by Lars Kanis, reviewed by Nishiyama Tomoaki and tweaked some by me.

This compiler, or at least the latest version of it, is currently broken, and
only passes the regression tests if built with -O0.

12 years agoIf pg_test_fsync is interrupted, clean up the temp file.
Robert Haas [Fri, 9 Dec 2011 20:05:48 +0000 (15:05 -0500)]
If pg_test_fsync is interrupted, clean up the temp file.

Marti Raudsepp, with additional paranoia by me.

12 years agoAdd ALTER FOREIGN DATA WRAPPER / RENAME and ALTER SERVER / RENAME
Peter Eisentraut [Fri, 9 Dec 2011 18:42:30 +0000 (20:42 +0200)]
Add ALTER FOREIGN DATA WRAPPER / RENAME and ALTER SERVER / RENAME

12 years agoDon't set reachedMinRecoveryPoint during crash recovery. In crash recovery,
Heikki Linnakangas [Fri, 9 Dec 2011 12:32:42 +0000 (14:32 +0200)]
Don't set reachedMinRecoveryPoint during crash recovery. In crash recovery,
we don't reach consistency before replaying all of the WAL. Rename the
variable to reachedConsistency, to make its intention clearer.

In master, that was an active bug because of the recent patch to
immediately PANIC if a reference to a missing page is found in WAL after
reaching consistency, as Tom Lane's test case demonstrated. In 9.1 and 9.0,
the only consequence was a misleading "consistent recovery state reached at
%X/%X" message in the log at the beginning of crash recovery (the database
is not consistent at that point yet). In 8.4, the log message was not
printed in crash recovery, even though there was a similar
reachedMinRecoveryPoint local variable that was also set early. So,
backpatch to 9.1 and 9.0.

12 years agoCancel running query if it is detected that the connection to the client is
Heikki Linnakangas [Fri, 9 Dec 2011 09:37:21 +0000 (11:37 +0200)]
Cancel running query if it is detected that the connection to the client is
lost. The only way we detect that at the moment is when write() fails when
we try to write to the socket.

Florian Pflug with small changes by me, reviewed by Greg Jaskiewicz.

12 years agoAdd const qualifiers to node inspection functions
Peter Eisentraut [Wed, 7 Dec 2011 19:46:56 +0000 (21:46 +0200)]
Add const qualifiers to node inspection functions

Thomas Munro

12 years agoFix corner cases in readlink() usage.
Tom Lane [Wed, 7 Dec 2011 18:34:13 +0000 (13:34 -0500)]
Fix corner cases in readlink() usage.

Make sure all calls are protected by HAVE_READLINK, and get the buffer
overflow tests right.  Be a bit more paranoid about string length in
_tarWriteHeader(), too.

12 years agoBetter error reporting if the link target is too long
Magnus Hagander [Wed, 7 Dec 2011 11:17:55 +0000 (12:17 +0100)]
Better error reporting if the link target is too long

This situation won't set errno, so using %m will give an incorrect
error message.

12 years agoAvoid using readlink() on platforms that don't support it
Magnus Hagander [Wed, 7 Dec 2011 11:09:05 +0000 (12:09 +0100)]
Avoid using readlink() on platforms that don't support it

We don't have any such platforms now, but might in the future.

Also, detect cases when a tablespace symlink points to a path that
is longer than we can handle, and give a warning.

12 years agoRemove spclocation field from pg_tablespace
Magnus Hagander [Wed, 7 Dec 2011 09:35:00 +0000 (10:35 +0100)]
Remove spclocation field from pg_tablespace

Instead, add a function pg_tablespace_location(oid) used to return
the same information, and do this by reading the symbolic link.

Doing it this way makes it possible to relocate a tablespace when the
database is down by simply changing the symbolic link.

12 years agoCreate a "sort support" interface API for faster sorting.
Tom Lane [Wed, 7 Dec 2011 05:18:38 +0000 (00:18 -0500)]
Create a "sort support" interface API for faster sorting.

This patch creates an API whereby a btree index opclass can optionally
provide non-SQL-callable support functions for sorting.  In the initial
patch, we only use this to provide a directly-callable comparator function,
which can be invoked with a bit less overhead than the traditional
SQL-callable comparator.  While that should be of value in itself, the real
reason for doing this is to provide a datatype-extensible framework for
more aggressive optimizations, as in Peter Geoghegan's recent work.

Robert Haas and Tom Lane

12 years agoTypo fixes for commit 2ad36c4e44c8b513f6155656e1b7a8d26715bb94.
Robert Haas [Tue, 6 Dec 2011 20:50:02 +0000 (15:50 -0500)]
Typo fixes for commit 2ad36c4e44c8b513f6155656e1b7a8d26715bb94.

Noted during post-commit review by by Noah Misch.

12 years agoMake command-line tools smarter about finding a DB to connect to.
Robert Haas [Tue, 6 Dec 2011 13:48:15 +0000 (08:48 -0500)]
Make command-line tools smarter about finding a DB to connect to.

If unable to connect to "postgres", try "template1".  This allows things to
work more smoothly in the case where the postgres database has been
dropped.  And just in case that's not good enough, also allow the user to
specify a maintenance database to be used for the initial connection, to
cover the case where neither postgres nor template1 is suitable.

12 years agoAdd missing documentation for function pg_stat_get_wal_senders()
Magnus Hagander [Tue, 6 Dec 2011 10:02:02 +0000 (11:02 +0100)]
Add missing documentation for function pg_stat_get_wal_senders()

Euler Taveira de Oliveira

12 years agoIn pg_upgrade, allow tables using regclass to be upgraded because we
Bruce Momjian [Mon, 5 Dec 2011 21:45:19 +0000 (16:45 -0500)]
In pg_upgrade, allow tables using regclass to be upgraded because we
preserve pg_class oids since PG 9.0.

12 years agoRemove troublesome Asserts in cost_mergejoin().
Tom Lane [Mon, 5 Dec 2011 20:50:06 +0000 (15:50 -0500)]
Remove troublesome Asserts in cost_mergejoin().

While logically correct, these two Asserts could fail depending on the
vagaries of floating-point arithmetic.  In particular, on machines with
floating-point registers wider than standard "double" values, it was
possible for the compiler to compare a rounded-to-double value already
stored in memory with an unrounded long double value still in a register.
Given the preceding checks, these assertions aren't adding much, so let's
just get rid of them rather than try to find a compiler-proof fix.
Per report from Pavel Stehule.

Given the lack of previous complaints, and the fact that only developers
would be likely to trip over it, I'm only going to change this in HEAD,
even though the code has been like this for a long time.

12 years agoplpython: Add SPI cursor support
Peter Eisentraut [Mon, 5 Dec 2011 17:52:15 +0000 (19:52 +0200)]
plpython: Add SPI cursor support

Add a function plpy.cursor that is similar to plpy.execute but uses an
SPI cursor to avoid fetching the entire result set into memory.

Jan Urbański, reviewed by Steve Singer

12 years agoAdd a \setenv command to psql.
Andrew Dunstan [Sun, 4 Dec 2011 16:43:38 +0000 (11:43 -0500)]
Add a \setenv command to psql.

This can be used to set (or unset) environment variables that will
affect programs called by psql (such as the PAGER), probably most
usefully in a .psqlrc file.

Andrew Dunstan, reviewed by Josh Kupershmidt.

12 years agoRemove incorrect instructions to run CREATE FOREIGN DATA WRAPPER
Magnus Hagander [Sun, 4 Dec 2011 15:57:36 +0000 (16:57 +0100)]
Remove incorrect instructions to run CREATE FOREIGN DATA WRAPPER

The CREATE EXTENSION step does this automatically. Doing it again will
cause an error.

12 years agoApplied another patch by Zoltan to fix memory alignement issues in ecpg's sqlda
Michael Meskes [Sat, 3 Dec 2011 20:03:57 +0000 (21:03 +0100)]
Applied another patch by Zoltan to fix memory alignement issues in ecpg's sqlda
code.

12 years agoTreat ENOTDIR as ENOENT when looking for client certificate file
Magnus Hagander [Sat, 3 Dec 2011 14:02:53 +0000 (15:02 +0100)]
Treat ENOTDIR as ENOENT when looking for client certificate file

This makes it possible to use a libpq app with home directory set
to /dev/null, for example - treating it the same as if the file
doesn't exist (which it doesn't).

Per bug #6302, reported by Diego Elio Petteno

12 years agoUpdate documentation suggestions for debugging the backend.
Bruce Momjian [Fri, 2 Dec 2011 22:08:33 +0000 (17:08 -0500)]
Update documentation suggestions for debugging the backend.

Tom Lane, with minor adjustments by me.

12 years agopsql: Make temporary editor files have .sql extension
Peter Eisentraut [Fri, 2 Dec 2011 21:38:33 +0000 (23:38 +0200)]
psql: Make temporary editor files have .sql extension

This gives editors a better chance to treat these files as the SQL
files that they are.

12 years agoAdd some weasel wording about threaded usage of PGresults.
Tom Lane [Fri, 2 Dec 2011 16:33:53 +0000 (11:33 -0500)]
Add some weasel wording about threaded usage of PGresults.

PGresults used to be read-only from the application's viewpoint, but now
that we've exposed various functions that allow modification of a PGresult,
that sweeping statement is no longer accurate.  Noted by Dmitriy Igrishin.

12 years agoDuring recovery, if we reach consistent state and still have entries in the
Heikki Linnakangas [Fri, 2 Dec 2011 08:49:54 +0000 (10:49 +0200)]
During recovery, if we reach consistent state and still have entries in the
invalid-page hash table, PANIC immediately. Immediate PANIC is much better
than waiting for end-of-recovery, which is what we did before, because the
end-of-recovery might not come until months later if this is a standby
server.

Also refrain from creating a restartpoint if there are invalid-page entries
in the hash table. Restarting recovery from such a restartpoint would not
see the invalid references, and wouldn't be able to cross-check them when
consistency is reached. That wouldn't matter when things are going smoothly,
but the more sanity checks you have the better.

Fujii Masao

12 years agoClarify documentation about SQL:2008 variant of LIMIT/OFFSET syntax.
Tom Lane [Thu, 1 Dec 2011 21:38:59 +0000 (16:38 -0500)]
Clarify documentation about SQL:2008 variant of LIMIT/OFFSET syntax.

The point that you need parentheses for non-constant expressions apparently
needs to be brought out a bit more clearly, per bug #6315.

12 years agoMake pgcompinclude/pgrminclude less tied to Bruce's personal machine.
Robert Haas [Thu, 1 Dec 2011 19:44:32 +0000 (14:44 -0500)]
Make pgcompinclude/pgrminclude less tied to Bruce's personal machine.

Not everyone has /pg linked to the src subdirectory of their PostgreSQL
tree.  Also, cc isn't the way to invoke the compiler everywhere.

12 years agoFix getTypeIOParam to support type record[].
Tom Lane [Thu, 1 Dec 2011 17:44:16 +0000 (12:44 -0500)]
Fix getTypeIOParam to support type record[].

Since record[] uses array_in, it needs to have its element type passed
as typioparam.  In HEAD and 9.1, this fix essentially reverts commit
9bc933b2125a5358722490acbc50889887bf7680, which was a hack that is no
longer needed since domains don't set their typelem anymore.  Before
that, adjust the logic so that only domains are excluded from being
treated like arrays, rather than assuming that only base types should
be included.  Add a regression test to demonstrate the need for this.
Per report from Maxim Boguk.

Back-patch to 8.4, where type record[] was added.

12 years agoAdd file-fdw documentation example.
Bruce Momjian [Thu, 1 Dec 2011 14:33:59 +0000 (09:33 -0500)]
Add file-fdw documentation example.

Josh Berkus

12 years agoAdd documentation mention that 7 != NULL also returns NULL.
Bruce Momjian [Thu, 1 Dec 2011 14:17:51 +0000 (09:17 -0500)]
Add documentation mention that 7 != NULL also returns NULL.

12 years agoUpdate information about configuring SysV IPC parameters on NetBSD.
Tom Lane [Thu, 1 Dec 2011 01:55:00 +0000 (20:55 -0500)]
Update information about configuring SysV IPC parameters on NetBSD.

Per Emmanuel Kasper, sysctl works fine as of NetBSD 5.0.

12 years agoDraft release notes for 9.1.2, 9.0.6, 8.4.10, 8.3.17, 8.2.23.
Tom Lane [Thu, 1 Dec 2011 00:34:47 +0000 (19:34 -0500)]
Draft release notes for 9.1.2, 9.0.6, 8.4.10, 8.3.17, 8.2.23.

12 years agoUpdate time zone data files to tzdata release 2011n.
Tom Lane [Wed, 30 Nov 2011 16:48:05 +0000 (11:48 -0500)]
Update time zone data files to tzdata release 2011n.

DST law changes in Brazil, Cuba, Fiji, Palestine, Russia, Samoa.
Historical corrections for Alaska and British East Africa.

12 years agoImprove table locking behavior in the face of current DDL.
Robert Haas [Wed, 30 Nov 2011 15:12:27 +0000 (10:12 -0500)]
Improve table locking behavior in the face of current DDL.

In the previous coding, callers were faced with an awkward choice:
look up the name, do permissions checks, and then lock the table; or
look up the name, lock the table, and then do permissions checks.
The first choice was wrong because the results of the name lookup
and permissions checks might be out-of-date by the time the table
lock was acquired, while the second allowed a user with no privileges
to interfere with access to a table by users who do have privileges
(e.g. if a malicious backend queues up for an AccessExclusiveLock on
a table on which AccessShareLock is already held, further attempts
to access the table will be blocked until the AccessExclusiveLock
is obtained and the malicious backend's transaction rolls back).

To fix, allow callers of RangeVarGetRelid() to pass a callback which
gets executed after performing the name lookup but before acquiring
the relation lock.  If the name lookup is retried (because
invalidation messages are received), the callback will be re-executed
as well, so we get the best of both worlds.  RangeVarGetRelid() is
renamed to RangeVarGetRelidExtended(); callers not wishing to supply
a callback can continue to invoke it as RangeVarGetRelid(), which is
now a macro.  Since the only one caller that uses nowait = true now
passes a callback anyway, the RangeVarGetRelid() macro defaults nowait
as well.  The callback can also be used for supplemental locking - for
example, REINDEX INDEX needs to acquire the table lock before the index
lock to reduce deadlock possibilities.

There's a lot more work to be done here to fix all the cases where this
can be a problem, but this commit provides the general infrastructure
and fixes the following specific cases: REINDEX INDEX, REINDEX TABLE,
LOCK TABLE, and and DROP TABLE/INDEX/SEQUENCE/VIEW/FOREIGN TABLE.

Per discussion with Noah Misch and Alvaro Herrera.

12 years agoTweak previous patch to ensure edata->filename always gets initialized.
Tom Lane [Wed, 30 Nov 2011 05:37:06 +0000 (00:37 -0500)]
Tweak previous patch to ensure edata->filename always gets initialized.

On a platform that isn't supplying __FILE__, previous coding would either
crash or give a stale result for the filename string.  Not sure how likely
that is, but the original code catered for it, so let's keep doing so.

12 years agoStrip file names reported in error messages in vpath builds
Peter Eisentraut [Tue, 29 Nov 2011 20:04:59 +0000 (22:04 +0200)]
Strip file names reported in error messages in vpath builds

In vpath builds, the __FILE__ macro that is used in verbose error
reports contains the full absolute file name, which makes the error
messages excessively verbose.  So keep only the base name, thus
matching the behavior of non-vpath builds.

12 years agoRemove duplicate definition of 'progname'.
Tom Lane [Wed, 30 Nov 2011 04:19:46 +0000 (23:19 -0500)]
Remove duplicate definition of 'progname'.

Per buildfarm.

12 years agoMove EXTRA_CLEAN to where it actually works.
Tom Lane [Wed, 30 Nov 2011 04:14:16 +0000 (23:14 -0500)]
Move EXTRA_CLEAN to where it actually works.

12 years agoPrevent autovacuum transactions from running in serializable mode.
Tom Lane [Wed, 30 Nov 2011 03:39:16 +0000 (22:39 -0500)]
Prevent autovacuum transactions from running in serializable mode.

Force the transaction isolation level to READ COMMITTED in autovacuum
worker and launcher processes.  There is no benefit to using a higher
isolation level, and doing so could result in delaying foreground
transactions (or maybe even causing unnecessary serialization failures?).
Noted by Dan Ports.

Also, make sure we disable zero_damaged_pages and statement_timeout in
the autovac launcher, not only workers.  Now that the launcher can run
transactions, these settings could affect its behavior, and it seems
like the same arguments apply to the launcher as the workers.

12 years agoChange installation docs to mention general debugging options.
Bruce Momjian [Wed, 30 Nov 2011 03:35:06 +0000 (22:35 -0500)]
Change installation docs to mention general debugging options.

12 years agoClean up after recent pg_dump patches.
Tom Lane [Wed, 30 Nov 2011 01:41:06 +0000 (20:41 -0500)]
Clean up after recent pg_dump patches.

Fix entirely broken handling of va_list printing routines, update some
out-of-date comments, fix some bogus inclusion orders, fix NLS declarations,
fix missed realloc calls.

12 years agoIn docs, suggest "-O0 -g" only if using a debugger.
Bruce Momjian [Wed, 30 Nov 2011 00:11:53 +0000 (19:11 -0500)]
In docs, suggest "-O0 -g" only if using a debugger.

12 years agoWhen a row fails a not-null constraint, show row's contents in errdetail.
Tom Lane [Tue, 29 Nov 2011 23:29:18 +0000 (18:29 -0500)]
When a row fails a not-null constraint, show row's contents in errdetail.

Simple extension of previous patch for CHECK constraints.

12 years agoSimplify the pg_dump/pg_restore error reporting macros, and allow
Bruce Momjian [Tue, 29 Nov 2011 21:34:45 +0000 (16:34 -0500)]
Simplify the pg_dump/pg_restore error reporting macros, and allow
pg_dumpall to use the same memory allocation functions as the others.

12 years agoSuggest configure options for server developers.
Bruce Momjian [Tue, 29 Nov 2011 21:31:53 +0000 (16:31 -0500)]
Suggest configure options for server developers.

Greg Smith

12 years agoWhen a row fails a CHECK constraint, show row's contents in errdetail.
Tom Lane [Tue, 29 Nov 2011 20:02:10 +0000 (15:02 -0500)]
When a row fails a CHECK constraint, show row's contents in errdetail.

This should make it easier to identify which row is problematic when an
insert or update is processing many rows.

The formatting is similar to that for unique-index violation messages,
except that we limit field widths to 64 bytes since otherwise the message
could get unreasonably long.  (In particular, there's currently no attempt
to quote or escape field values that contain commas etc.)

Jan Kundrát, reviewed by Royce Ausburn, somewhat rewritten by me.

12 years agopg_dump: Add gettext plural support to error message
Peter Eisentraut [Tue, 29 Nov 2011 17:49:02 +0000 (19:49 +0200)]
pg_dump: Add gettext plural support to error message

12 years agopg_upgrade: Disable installcheck
Peter Eisentraut [Tue, 29 Nov 2011 04:57:09 +0000 (06:57 +0200)]
pg_upgrade: Disable installcheck

Disabled for now because some build farm members with low resources
are not prepared to handle it.

12 years agopython.m4: Remove useless "import string" calls
Peter Eisentraut [Tue, 29 Nov 2011 04:50:11 +0000 (06:50 +0200)]
python.m4: Remove useless "import string" calls

They have been unneeded since the use of the string module has been
removed in a65ed83f8a6fbf35c1e563533ab8293a788b1fce.

12 years agoplpython: Fix sed expression in python3 build
Peter Eisentraut [Tue, 29 Nov 2011 04:39:05 +0000 (06:39 +0200)]
plpython: Fix sed expression in python3 build

The old expression sed 's,$(srcdir),python3,' would normally resolve
as sed 's,.,python3,', which is not really what we wanted.  While it
doesn't actually break anything right now, it's still wrong, so put in
a bit more work to make it more robust.

12 years agopg_dump: Fix clean rule after file renaming shuffle
Peter Eisentraut [Tue, 29 Nov 2011 04:30:45 +0000 (06:30 +0200)]
pg_dump: Fix clean rule after file renaming shuffle

12 years agoDocument that perl needs to be indented during the pgindent run.
Bruce Momjian [Tue, 29 Nov 2011 02:56:19 +0000 (21:56 -0500)]
Document that perl needs to be indented during the pgindent run.

12 years agoMake some minor formatting improvements to what pgindent did.
Tom Lane [Tue, 29 Nov 2011 01:19:29 +0000 (20:19 -0500)]
Make some minor formatting improvements to what pgindent did.

Moving the code two full tab stops to the right requires rethinking of
cosmetic code layout choices, which pgindent isn't really able to do for
us.  Whitespace and comment adjustments only, no code changes.

12 years agoDisallow deletion of CurrentExtensionObject while running extension script.
Tom Lane [Tue, 29 Nov 2011 00:12:17 +0000 (19:12 -0500)]
Disallow deletion of CurrentExtensionObject while running extension script.

While the deletion in itself wouldn't break things, any further creation
of objects in the script would result in dangling pg_depend entries being
added by recordDependencyOnCurrentExtension().  An example from Phil
Sorber convinced me that this is just barely likely enough to be worth
expending a couple lines of code to defend against.  The resulting error
message might be confusing, but it's better than leaving corrupted catalog
contents for the user to deal with.

12 years agoPgindent clauses.c, per request from Tom.
Bruce Momjian [Mon, 28 Nov 2011 21:47:43 +0000 (16:47 -0500)]
Pgindent clauses.c, per request from Tom.

12 years agoConvert eval_const_expressions's long series of IsA tests into a switch.
Tom Lane [Mon, 28 Nov 2011 19:21:40 +0000 (14:21 -0500)]
Convert eval_const_expressions's long series of IsA tests into a switch.

This function has now grown enough cases that a switch seems appropriate.
This results in a measurable speed improvement on some platforms, and
should certainly not hurt.  The code's in need of a pgindent run now,
though.

Andres Freund

12 years agoRemove erroneous claim about use of pg_locks.objid for advisory locks.
Tom Lane [Mon, 28 Nov 2011 18:51:58 +0000 (13:51 -0500)]
Remove erroneous claim about use of pg_locks.objid for advisory locks.

The correct information appears in the text, so just remove the statement
in the table, where it did not fit nicely anyway.  (Curiously, the correct
info has been there much longer than the erroneous table entry.)
Resolves problem noted by Daniele Varrazzo.

In HEAD and 9.1, also do a bit of wordsmithing on other text on the page.

12 years agoFix some bogosities in pg_dump's foreign-table support.
Tom Lane [Mon, 28 Nov 2011 17:51:41 +0000 (12:51 -0500)]
Fix some bogosities in pg_dump's foreign-table support.

The server name for a foreign table was not quoted at need, as per report
from Ronan Dunklau.  Also, queries related to FDW options were inadequately
schema-qualified in places where the search path isn't just pg_catalog, and
were inconsistently formatted everywhere, and we didn't always check that
we got the expected number of rows from them.

12 years agoAdd libpq connection option to disable SSL compression
Magnus Hagander [Mon, 28 Nov 2011 12:13:42 +0000 (13:13 +0100)]
Add libpq connection option to disable SSL compression

This can be used to remove the overhead of SSL compression on
fast networks.

Laurenz Albe

12 years agoEnsure that whole-row junk Vars are always of composite type.
Tom Lane [Mon, 28 Nov 2011 03:27:24 +0000 (22:27 -0500)]
Ensure that whole-row junk Vars are always of composite type.

The EvalPlanQual machinery assumes that whole-row Vars generated for the
outputs of non-table RTEs will be of composite types.  However, for the
case where the RTE is a function call returning a scalar type, we were
doing the wrong thing, as a result of sharing code with a parser case
where the function's scalar output is wanted.  (Or at least, that's what
that case has done historically; it does seem a bit inconsistent.)

To fix, extend makeWholeRowVar's API so that it can support both use-cases.
This fixes Belinda Cussen's report of crashes during concurrent execution
of UPDATEs involving joins to the result of UNNEST() --- in READ COMMITTED
mode, we'd run the EvalPlanQual machinery after a conflicting row update
commits, and it was expecting to get a HeapTuple not a scalar datum from
the "wholerowN" variable referencing the function RTE.

Back-patch to 9.0 where the current EvalPlanQual implementation appeared.

In 9.1 and up, this patch also fixes failure to attach the correct
collation to the Var generated for a scalar-result case.  An example:
regression=# select upper(x.*) from textcat('ab', 'cd') x;
ERROR:  could not determine which collation to use for upper() function

12 years agoMake pg_dumpall build with the right object files under MSVC.
Andrew Dunstan [Mon, 28 Nov 2011 01:14:47 +0000 (20:14 -0500)]
Make pg_dumpall build with the right object files under MSVC.

This fixes a longstanding but up to now benign bug in the way pg_dumpall
was built. The bug was exposed by recent code adjustments. The Makefile
does not use $(OBJS) to build pg_dumpall, so this fix removes their source
files from the pg_dumpall object and adds in the one source file it
consequently needs.

12 years agoUse IEEE infinity, not 1e10, for null-and-not-null case in gistpenalty().
Tom Lane [Sun, 27 Nov 2011 22:12:54 +0000 (17:12 -0500)]
Use IEEE infinity, not 1e10, for null-and-not-null case in gistpenalty().

Use of a randomly chosen large value was never exactly graceful, and
now that there are penalty functions that are intentionally using infinity,
it doesn't seem like a good idea for null-vs-not-null to be using something
less.

12 years agoImprove GiST range-contained-by searches by adding a flag for empty ranges.
Tom Lane [Sun, 27 Nov 2011 21:50:37 +0000 (16:50 -0500)]
Improve GiST range-contained-by searches by adding a flag for empty ranges.

In the original implementation, a range-contained-by search had to scan
the entire index because an empty range could be lurking anywhere.
Improve that by adding a flag to upper GiST entries that says whether the
represented subtree contains any empty ranges.

Also, make a simple mod to the penalty function to discourage empty ranges
from getting pushed into subtrees without any.  This needs more work, and
the picksplit function should be taught about it too, but that code can be
improved without causing an on-disk compatibility break; so we'll leave it
for another day.

Since we're breaking on-disk compatibility of range values anyway, I took
the opportunity to reorganize the range flags bits; the unused
RANGE_xB_NULL bits are now adjacent, which might open the door for using
them in some other way later.

In passing, remove the GiST range opclass entry for <>, which doesn't seem
like it can really be indexed usefully.

Alexander Korotkov, with some editorializing by Tom

12 years agoAdd pg_upgrade test suite
Peter Eisentraut [Sun, 27 Nov 2011 20:42:32 +0000 (22:42 +0200)]
Add pg_upgrade test suite

It runs the regression tests, runs pg_upgrade on the populated
database, and compares the before and after dumps.  While not actually
a cross-version upgrade, this does detect omissions and bugs in the
involved tools from time to time.  It's also possible to do a
cross-version upgrade by manually supplying parameters.

12 years agoUse the proper macro to convert a bool to a Datum.
Tom Lane [Sun, 27 Nov 2011 17:57:11 +0000 (12:57 -0500)]
Use the proper macro to convert a bool to a Datum.

The original coding was
var->value = (Datum) state;
which is bogus, and then in commit 2f0f7b4bce13e68394543728801ef011fd82fac6
it was "corrected" to
var->value = PointerGetDatum(state);
which is a faithful translation but still wrong.

This seems purely cosmetic, though, so no need for a back-patch.

Pavel Stehule

12 years agoFix MSVC builds broken by xsubpp change
Andrew Dunstan [Sun, 27 Nov 2011 06:23:47 +0000 (01:23 -0500)]
Fix MSVC builds broken by xsubpp change

12 years agoMove pg_dump memory routines into pg_dumpmem.c/h and restore common.c
Bruce Momjian [Sun, 27 Nov 2011 03:34:36 +0000 (22:34 -0500)]
Move pg_dump memory routines into pg_dumpmem.c/h and restore common.c
with its original functions.  The previous function migration would
cause too many difficulties in back-patching.

12 years agoUse the right interpreter for encoding test.
Andrew Dunstan [Sat, 26 Nov 2011 23:39:12 +0000 (18:39 -0500)]
Use the right interpreter for encoding test.

12 years agoUse the preferred version of xsubpp, not necessarily the one that came with the
Andrew Dunstan [Sat, 26 Nov 2011 20:22:32 +0000 (15:22 -0500)]
Use the preferred version of xsubpp, not necessarily the one that came with the
distro version of perl.

David Wheeler and Alex Hunsaker.

Backpatch to 9.1 where it applies cleanly. A simple workaround is available for earlier
branches, and further effort doesn't seem warranted.

12 years agoMake GiST index searches smarter about queries against empty ranges.
Tom Lane [Sat, 26 Nov 2011 19:27:05 +0000 (14:27 -0500)]
Make GiST index searches smarter about queries against empty ranges.

In the cases where the result of the called proc is negated, we should
explicitly test both inputs for empty, to ensure we'll never return "true"
for an unsatisfiable query.  In other cases we can rely on the called proc
to say the right thing.

12 years agoFix overly-aggressive and inconsistent quoting in OS X start script.
Tom Lane [Sat, 26 Nov 2011 18:01:02 +0000 (13:01 -0500)]
Fix overly-aggressive and inconsistent quoting in OS X start script.

Sidar Lopez, per bug #6310, with some additional improvements by me.
Back-patch to 9.0, where the issue was introduced.

12 years agoEnsure plperl strings are always correctly UTF8 encoded.
Andrew Dunstan [Sat, 26 Nov 2011 17:19:38 +0000 (12:19 -0500)]
Ensure plperl strings are always correctly UTF8 encoded.

Amit Khandekar and Alex Hunsaker.

Backpatched to 9.1 where the problem first occurred.

12 years agoFix join_path_components() to not add a leading slash when joining to an
Bruce Momjian [Sat, 26 Nov 2011 14:27:11 +0000 (09:27 -0500)]
Fix join_path_components() to not add a leading slash when joining to an
initial null string.

Per report from Robert Haas in testing psql \ir.