]> granicus.if.org Git - postgresql/log
postgresql
13 years agoDon't leak the temporary PLyProcedure struct we create for inline plpython
Heikki Linnakangas [Thu, 31 Mar 2011 09:37:11 +0000 (12:37 +0300)]
Don't leak the temporary PLyProcedure struct we create for inline plpython
blocks.

Investigation by Jan UrbaƄski, though I didn't use his patch.

13 years agoReword the phrase on zero replication_timeout in the docs.
Heikki Linnakangas [Thu, 31 Mar 2011 07:19:29 +0000 (10:19 +0300)]
Reword the phrase on zero replication_timeout in the docs.

13 years agoImprove error message when WAL ends before reaching end of online backup.
Heikki Linnakangas [Thu, 31 Mar 2011 07:09:49 +0000 (10:09 +0300)]
Improve error message when WAL ends before reaching end of online backup.

13 years agoMention pg_upgrade sets autovacuum_freeze_max_age to maximum.
Bruce Momjian [Wed, 30 Mar 2011 21:45:54 +0000 (17:45 -0400)]
Mention pg_upgrade sets autovacuum_freeze_max_age to maximum.

13 years agoExpand comment on how pg_upgrade is turning off autovacuum.
Bruce Momjian [Wed, 30 Mar 2011 21:36:24 +0000 (17:36 -0400)]
Expand comment on how pg_upgrade is turning off autovacuum.

13 years agoAttempt to unbreak windows builds broken by commit 754baa2.
Andrew Dunstan [Wed, 30 Mar 2011 20:43:31 +0000 (16:43 -0400)]
Attempt to unbreak windows builds broken by commit 754baa2.

13 years agoCheck that we've reached end-of-backup also when we're not performing
Heikki Linnakangas [Wed, 30 Mar 2011 07:36:58 +0000 (10:36 +0300)]
Check that we've reached end-of-backup also when we're not performing
archive recovery.

It's possible to restore an online backup without recovery.conf, by simply
copying all the necessary WAL files to pg_xlog. "pg_basebackup -x" does that
too. That's the use case where this cross-check is useful.

Backpatch to 9.0. We used to do this in earlier versins, but in 9.0 the code
was inadvertently changed so that the check is only performed after archive
recovery.

Fujii Masao.

13 years agoAutomatically terminate replication connections that are idle for more
Heikki Linnakangas [Wed, 30 Mar 2011 07:10:32 +0000 (10:10 +0300)]
Automatically terminate replication connections that are idle for more
than replication_timeout (a new GUC) milliseconds. The TCP timeout is often
too long, you want the master to notice a dead connection much sooner.
People complained about that in 9.0 too, but with synchronous replication
it's even more important to notice dead connections promptly.

Fujii Masao and Heikki Linnakangas

13 years agoAdjust error message, now that we expect other message types than connection
Heikki Linnakangas [Wed, 30 Mar 2011 05:54:28 +0000 (08:54 +0300)]
Adjust error message, now that we expect other message types than connection
close at this point. Fix PQsetnonblocking() comment.

Fujii Masao

13 years agoUpdate SQL features list
Peter Eisentraut [Tue, 29 Mar 2011 20:23:50 +0000 (23:23 +0300)]
Update SQL features list

Feature F692 "Extended collation support" is now also supported.  This
refers to allowing the COLLATE clause anywhere in a column or domain
definition instead of just directly after the type.

Also correct the name of the feature in accordance with the latest SQL
standard.

13 years agoIn pg_upgrade, add C comment about how autovacuum is disabled.
Bruce Momjian [Tue, 29 Mar 2011 15:11:12 +0000 (11:11 -0400)]
In pg_upgrade, add C comment about how autovacuum is disabled.

13 years agoAdd maintainer-check target
Peter Eisentraut [Mon, 28 Mar 2011 19:53:55 +0000 (22:53 +0300)]
Add maintainer-check target

This can do various source code checks that are not appropriate for
either the build or the regression tests.  Currently: duplicate_oids,
SGML syntax and tabs check, NLS syntax check.

13 years agoMake duplicate_oids return nonzero exit status if duplicates were found
Peter Eisentraut [Mon, 28 Mar 2011 19:52:23 +0000 (22:52 +0300)]
Make duplicate_oids return nonzero exit status if duplicates were found

Automatic detection of errors is easier that way.

13 years agoPrevent a rowtype from being included in itself.
Tom Lane [Mon, 28 Mar 2011 19:44:54 +0000 (15:44 -0400)]
Prevent a rowtype from being included in itself.

Eventually we might be able to allow that, but it's not clear how many
places need to be fixed to prevent infinite recursion when there's a direct
or indirect inclusion of a rowtype in itself.  One such place is
CheckAttributeType(), which will recurse to stack overflow in cases such as
those exhibited in bug #5950 from Alex Perepelica.  If we were sure it was
the only such place, we could easily modify the code added by this patch to
stop the recursion without a complaint ... but it probably isn't the only
such place.  Hence, throw error until such time as someone is excited
enough about this type of usage to put work into making it safe.

Back-patch as far as 8.3.  8.2 doesn't have the recursive call in
CheckAttributeType in the first place, so I see no need to add code there
in the absence of clear evidence of a problem elsewhere.

13 years agoAdd 9.1 release note link for synchronous replication.
Bruce Momjian [Mon, 28 Mar 2011 18:50:08 +0000 (14:50 -0400)]
Add 9.1 release note link for synchronous replication.

We already had links to the GUC variables that control it.

13 years agoMark up release notes using <link>. REL9_1_ALPHA5
Robert Haas [Mon, 28 Mar 2011 13:42:09 +0000 (09:42 -0400)]
Mark up release notes using <link>.

Unlike <xref>, this actually works.

13 years agoAdd missing #include
Alvaro Herrera [Sat, 26 Mar 2011 03:09:17 +0000 (00:09 -0300)]
Add missing #include

13 years agoGet rid of links that don't work when building HISTORY.
Tom Lane [Mon, 28 Mar 2011 13:34:56 +0000 (09:34 -0400)]
Get rid of links that don't work when building HISTORY.

13 years agoTypo fix in SSI docs.
Robert Haas [Mon, 28 Mar 2011 01:35:15 +0000 (21:35 -0400)]
Typo fix in SSI docs.

Kevin Grittner

13 years agoCorrect "characters" to "bytes" in createdb docs.
Robert Haas [Mon, 28 Mar 2011 01:28:15 +0000 (21:28 -0400)]
Correct "characters" to "bytes" in createdb docs.

Susanne Ebrecht

13 years agofix up a couple non-prototypes of the form foo() to be foo(void) -- found using ...
Greg Stark [Sun, 27 Mar 2011 20:31:41 +0000 (21:31 +0100)]
fix up a couple non-prototypes of the form foo() to be foo(void) -- found using -Wstrict-prototypes

13 years agoFix check_exclusion_constraint() to insert correct collations in ScanKeys.
Tom Lane [Sun, 27 Mar 2011 17:29:52 +0000 (13:29 -0400)]
Fix check_exclusion_constraint() to insert correct collations in ScanKeys.

13 years agoFix plpgsql to release SPI plans when a function or DO block is freed.
Tom Lane [Sun, 27 Mar 2011 16:51:04 +0000 (12:51 -0400)]
Fix plpgsql to release SPI plans when a function or DO block is freed.

This fixes the gripe I made a few months ago about DO blocks getting
slower with repeated use.  At least, it fixes it for the case where
the DO block isn't aborted by an error.  We could try running
plpgsql_free_function_memory() even during error exit, but that seems
a bit scary since it makes a lot of presumptions about the data
structures being in good shape.  It's probably reasonable to assume
that repeated failures of DO blocks isn't a performance-critical case.

13 years agoRemove disclaimer stating that fsync=off slows down sync rep.
Robert Haas [Sun, 27 Mar 2011 01:18:53 +0000 (21:18 -0400)]
Remove disclaimer stating that fsync=off slows down sync rep.

The underlying problem that caused this phenomenon was fixed by commit
92f4786fa9b730fd12cbfe973eb96addc6e98924.

13 years agoUpdate release notes for latest commits.
Robert Haas [Sun, 27 Mar 2011 01:11:15 +0000 (21:11 -0400)]
Update release notes for latest commits.

In addition, correct the entry for pg_object_description, and some other
minor cleanup.

13 years agoClean up cruft around collation initialization for tupdescs and scankeys.
Tom Lane [Sat, 26 Mar 2011 22:28:40 +0000 (18:28 -0400)]
Clean up cruft around collation initialization for tupdescs and scankeys.

I found actual bugs in GiST and plpgsql; the rest of this is cosmetic
but meant to decrease the odds of future bugs of omission.

13 years agoMore collations cleanup, from trawling for missed collation assignments.
Tom Lane [Sat, 26 Mar 2011 20:35:25 +0000 (16:35 -0400)]
More collations cleanup, from trawling for missed collation assignments.

Mostly cosmetic, though I did find that generateClonedIndexStmt failed
to clone the index's collations.

13 years agoClean up a few failures to set collation fields in expression nodes.
Tom Lane [Sat, 26 Mar 2011 18:25:48 +0000 (14:25 -0400)]
Clean up a few failures to set collation fields in expression nodes.

I'm not sure these have any non-cosmetic implications, but I'm not sure
they don't, either.  In particular, ensure the CaseTestExpr generated
by transformAssignmentIndirection to represent the base target column
carries the correct collation, because parse_collate.c won't fix that.
Tweak lsyscache.c API so that we can get the appropriate collation
without an extra syscache lookup.

13 years agoAdditional test for each commit in sync rep path to plug minute
Simon Riggs [Sat, 26 Mar 2011 10:09:37 +0000 (10:09 +0000)]
Additional test for each commit in sync rep path to plug minute
possibility of race condition that would effect performance only.
Requested by Robert Haas. Re-arrange related comments.

13 years agoPass collation to makeConst() instead of looking it up internally.
Tom Lane [Sat, 26 Mar 2011 00:10:42 +0000 (20:10 -0400)]
Pass collation to makeConst() instead of looking it up internally.

In nearly all cases, the caller already knows the correct collation, and
in a number of places, the value the caller has handy is more correct than
the default for the type would be.  (In particular, this patch makes it
significantly less likely that eval_const_expressions will result in
changing the exposed collation of an expression.)  So an internal lookup
is both expensive and wrong.

13 years agoFix failure to propagate collation in negate_clause().
Tom Lane [Fri, 25 Mar 2011 22:44:47 +0000 (18:44 -0400)]
Fix failure to propagate collation in negate_clause().

Turns out it was this, and not so much plpgsql, that was at fault in Stefan
Huehner's collation-error-in-a-trigger bug report of a couple weeks ago.

13 years agoDocument collation handling in SQL and plpgsql functions.
Tom Lane [Fri, 25 Mar 2011 22:21:25 +0000 (18:21 -0400)]
Document collation handling in SQL and plpgsql functions.

This is pretty minimal but covers the bare facts.

13 years agoFix collation handling in plpgsql functions.
Tom Lane [Fri, 25 Mar 2011 19:06:36 +0000 (15:06 -0400)]
Fix collation handling in plpgsql functions.

Make plpgsql treat the input collation as a polymorphism variable, so
that we cache separate plans for each input collation that's used in a
particular session, as per recent discussion.  Propagate the input
collation to all collatable input parameters.

I chose to also propagate the input collation to all declared variables of
collatable types, which is a bit more debatable but seems to be necessary
for non-astonishing behavior.  (Copying a parameter into a separate local
variable shouldn't result in a change of behavior, for example.)  There is
enough infrastructure here to support declaring a collation for each local
variable to override that default, but I thought we should wait to see what
the field demand is before adding such a feature.

In passing, remove exec_get_rec_fieldtype(), which wasn't used anywhere.

Documentation patch to follow.

13 years agoRemove alpha release notes.
Robert Haas [Fri, 25 Mar 2011 15:52:11 +0000 (11:52 -0400)]
Remove alpha release notes.

Temporarily move some of the alpha release note disclaimers into the regular
release notes, for the sake of alpha5.

13 years agoMake walreceiver send a reply after receiving data but before flushing it.
Robert Haas [Fri, 25 Mar 2011 15:23:39 +0000 (11:23 -0400)]
Make walreceiver send a reply after receiving data but before flushing it.

It originally worked this way, but was changed by commit
a8a8a3e0965201df88bdfdff08f50e5c06c552b7, since which time it's been impossible
for walreceiver to ever send a reply with write_location and flush_location
set to different values.

13 years agoFix broken markup, and remove tabs
Alvaro Herrera [Fri, 25 Mar 2011 15:20:14 +0000 (12:20 -0300)]
Fix broken markup, and remove tabs

13 years agoDocumented some ecpg command line options that were missing:
Michael Meskes [Thu, 24 Mar 2011 21:18:11 +0000 (22:18 +0100)]
Documented some ecpg command line options that were missing:

-r no_indicator
-r prepare
-r questionsmarks

13 years agoFix handling of collation in SQL-language functions.
Tom Lane [Fri, 25 Mar 2011 00:30:14 +0000 (20:30 -0400)]
Fix handling of collation in SQL-language functions.

Ensure that parameter symbols receive collation from the function's
resolved input collation, and fix inlining to behave properly.

BTW, this commit lays about 90% of the infrastructure needed to support
use of argument names in SQL functions.  Parsing of parameters is now
done via the parser-hook infrastructure ... we'd just need to supply
a column-ref hook ...

13 years agoAdd post-creation hook for extensions, consistent with other object types.
Robert Haas [Thu, 24 Mar 2011 22:44:49 +0000 (18:44 -0400)]
Add post-creation hook for extensions, consistent with other object types.

KaiGai Kohei

13 years agoEdits to 9.1 release notes.
Robert Haas [Thu, 24 Mar 2011 20:57:12 +0000 (16:57 -0400)]
Edits to 9.1 release notes.

Add some new items and some additional details to existing items, mostly
by cribbing from the 9.1alpha notes.  Some additional clarifications and
corrections elsewhere, and a few typo fixes.

13 years agoClean up handling of COLLATE clauses in index column definitions.
Tom Lane [Thu, 24 Mar 2011 19:29:52 +0000 (15:29 -0400)]
Clean up handling of COLLATE clauses in index column definitions.

Ensure that COLLATE at the top level of an index expression is treated the
same as a grammatically separate COLLATE.  Fix bogus reverse-parsing logic
in pg_get_indexdef.

13 years agoImprove user-defined-aggregates documentation.
Tom Lane [Wed, 23 Mar 2011 20:56:55 +0000 (16:56 -0400)]
Improve user-defined-aggregates documentation.

On closer inspection, that two-element initcond value seems to have been
a little white lie to avoid explaining the full behavior of float8_accum.
But if people are going to expect the examples to be exactly correct,
I suppose we'd better explain.  Per comment from Thom Brown.

13 years agoMinor changes to recovery pause behaviour.
Simon Riggs [Wed, 23 Mar 2011 19:35:53 +0000 (19:35 +0000)]
Minor changes to recovery pause behaviour.
Change location LOG message so it works each time we pause, not
just for final pause.
Ensure that we pause only if we are in Hot Standby and can connect
to allow us to run resume function. This change supercedes the
code to override parameter recoveryPauseAtTarget to false if not
attempting to enter Hot Standby, which is now removed.

13 years agoFix ancient typo in user-defined-aggregates documentation.
Tom Lane [Wed, 23 Mar 2011 16:33:14 +0000 (12:33 -0400)]
Fix ancient typo in user-defined-aggregates documentation.

The description of the initcond value for the built-in avg(float8)
aggregate has been wrong since it was written.  Noted by Disc Magnet.

13 years agoRemove synchronous_replication/max_wal_senders cross-check.
Robert Haas [Wed, 23 Mar 2011 15:44:27 +0000 (11:44 -0400)]
Remove synchronous_replication/max_wal_senders cross-check.

This is no longer necessary, and might result in a situation where the
configuration file is reloaded (and everything seems OK) but a subsequent
restart of the database fails.

Per an observation from Fujii Masao.

13 years agoPrevent intermittent hang in recovery from bgwriter interaction.
Simon Riggs [Wed, 23 Mar 2011 13:30:05 +0000 (13:30 +0000)]
Prevent intermittent hang in recovery from bgwriter interaction.
Startup process waited for cleanup lock but when hot_standby = off
the pid was not registered, so that the bgwriter would not wake
the waiting process as intended.

13 years agoCosmetic capitalization fix
Peter Eisentraut [Tue, 22 Mar 2011 15:44:38 +0000 (17:44 +0200)]
Cosmetic capitalization fix

13 years agoMake FKs valid at creation when added as column constraints.
Simon Riggs [Tue, 22 Mar 2011 23:10:35 +0000 (23:10 +0000)]
Make FKs valid at creation when added as column constraints.

Bug report from Alvaro Herrera

13 years agoMake initdb ignore locales for client-only encodings.
Tom Lane [Tue, 22 Mar 2011 21:20:50 +0000 (17:20 -0400)]
Make initdb ignore locales for client-only encodings.

While putting such entries into pg_collation is harmless (since backends
will ignore entries that don't match the database encoding), it's also
useless.

13 years agoImprove reporting of run-time-detected indeterminate-collation errors.
Tom Lane [Tue, 22 Mar 2011 20:55:32 +0000 (16:55 -0400)]
Improve reporting of run-time-detected indeterminate-collation errors.

pg_newlocale_from_collation does not have enough context to give an error
message that's even a little bit useful, so move the responsibility for
complaining up to its callers.  Also, reword ERRCODE_INDETERMINATE_COLLATION
error messages in a less jargony, more message-style-guide-compliant
fashion.

13 years agoThrow error for indeterminate collation of an ORDER/GROUP/DISTINCT target.
Tom Lane [Tue, 22 Mar 2011 19:58:03 +0000 (15:58 -0400)]
Throw error for indeterminate collation of an ORDER/GROUP/DISTINCT target.

This restores a parse error that was thrown (though only in the ORDER BY
case) by the original collation patch.  I had removed it in my recent
revisions because it was thrown at a place where collations now haven't
been computed yet; but I thought of another way to handle it.

Throwing the error at parse time, rather than leaving it to be done at
runtime, is good because a syntax error pointer is helpful for localizing
the problem.  We can reasonably assume that the comparison function for a
collatable datatype will complain if it doesn't have a collation to use.
Now the planner might choose to implement GROUP or DISTINCT via hashing,
in which case no runtime error would actually occur, but it seems better
to throw error consistently rather than let the error depend on what the
planner chooses to do.  Another possible objection is that the user might
specify a nondefault sort operator that doesn't care about collation
... but that's surely an uncommon usage, and it wouldn't hurt him to throw
in a COLLATE clause anyway.  This change also makes the ORDER BY/GROUP
BY/DISTINCT case more consistent with the UNION/INTERSECT/EXCEPT case,
which was already coded to throw this error even though the same objections
could be raised there.

13 years agoAvoid potential deadlock in InitCatCachePhase2().
Tom Lane [Tue, 22 Mar 2011 17:00:24 +0000 (13:00 -0400)]
Avoid potential deadlock in InitCatCachePhase2().

Opening a catcache's index could require reading from that cache's own
catalog, which of course would acquire AccessShareLock on the catalog.
So the original coding here risks locking index before heap, which could
deadlock against another backend trying to get exclusive locks in the
normal order.  Because InitCatCachePhase2 is only called when a backend
has to start up without a relcache init file, the deadlock was seldom seen
in the field.  (And by the same token, there's no need to worry about any
performance disadvantage; so not much point in trying to distinguish
exactly which catalogs have the risk.)

Bug report, diagnosis, and patch by Nikhil Sontakke.  Additional commentary
by me.  Back-patch to all supported branches.

13 years agoUpdate docs to say you need fsync to make sync rep work fast.
Simon Riggs [Tue, 22 Mar 2011 16:23:54 +0000 (16:23 +0000)]
Update docs to say you need fsync to make sync rep work fast.

13 years agoReimplement planner's handling of MIN/MAX aggregate optimization (again).
Tom Lane [Tue, 22 Mar 2011 04:34:31 +0000 (00:34 -0400)]
Reimplement planner's handling of MIN/MAX aggregate optimization (again).

Instead of playing cute games with pathkeys, just build a direct
representation of the intended sub-select, and feed it through
query_planner to get a Path for the index access.  This is a bit slower
than 9.1's previous method, since we'll duplicate most of the overhead of
query_planner; but since the whole optimization only applies to rather
simple single-table queries, that probably won't be much of a problem in
practice.  The advantage is that we get to do the right thing when there's
a partial index that needs the implicit IS NOT NULL clause to be usable.
Also, although this makes planagg.c be a bit more closely tied to the
ordering of operations in grouping_planner, we can get rid of some coupling
to lower-level parts of the planner.  Per complaint from Marti Raudsepp.

13 years agoWhen two base backups are started at the same time with pg_basebackup,
Heikki Linnakangas [Mon, 21 Mar 2011 09:25:25 +0000 (11:25 +0200)]
When two base backups are started at the same time with pg_basebackup,
ensure that they use different checkpoints as the starting point. We use
the checkpoint redo location as a unique identifier for the base backup in
the end-of-backup record, and in the backup history file name.

Bug spotted by Fujii Masao.

13 years agoMarkup and wording improvements for 9.1 release notes.
Bruce Momjian [Mon, 21 Mar 2011 03:17:21 +0000 (23:17 -0400)]
Markup and wording improvements for 9.1 release notes.

13 years agoAnother 9.1 release note typo fix.
Bruce Momjian [Sun, 20 Mar 2011 23:36:44 +0000 (19:36 -0400)]
Another 9.1 release note typo fix.

13 years agoFix snapshot isolation level typo in 9.1 release notes.
Bruce Momjian [Sun, 20 Mar 2011 23:35:19 +0000 (19:35 -0400)]
Fix snapshot isolation level typo in 9.1 release notes.

13 years agoReorder some 9.1 release item documentation entries, fix some awkward wording.
Bruce Momjian [Sun, 20 Mar 2011 21:43:04 +0000 (17:43 -0400)]
Reorder some 9.1 release item documentation entries, fix some awkward wording.

13 years agoAdd some platform-independent tests for the collation feature.
Tom Lane [Sun, 20 Mar 2011 18:35:39 +0000 (14:35 -0400)]
Add some platform-independent tests for the collation feature.

There's a lot we can't test very well without platform dependencies,
but the C/POSIX collations should now work the same way everywhere.

13 years agoSuppress platform-dependent unused-variable warning.
Tom Lane [Sun, 20 Mar 2011 17:34:31 +0000 (13:34 -0400)]
Suppress platform-dependent unused-variable warning.

The local variable "sock" can be unused depending on compilation flags.
But there seems no particular need for it, since the kernel calls can
just as easily say port->sock instead.

13 years agoFix up handling of C/POSIX collations.
Tom Lane [Sun, 20 Mar 2011 16:43:39 +0000 (12:43 -0400)]
Fix up handling of C/POSIX collations.

Install just one instance of the "C" and "POSIX" collations into
pg_collation, rather than one per encoding.  Make these instances exist
and do something useful even in machines without locale_t support: to wit,
it's now possible to force comparisons and case-folding functions to use C
locale in an otherwise non-C database, whether or not the platform has
support for using any additional collations.

Fix up severely broken upper/lower/initcap functions, too: the C/POSIX
fastpath now does what it is supposed to, and non-default collations are
handled correctly in single-byte database encodings.

Merge the two separate collation hashtables that were being maintained in
pg_locale.c, and be more wary of the possibility that we fail partway
through filling a cache entry.

13 years agoMove PITR and StreamingRep up one level of heading in the 9.1 release
Bruce Momjian [Sun, 20 Mar 2011 13:33:37 +0000 (09:33 -0400)]
Move PITR and StreamingRep up one level of heading in the 9.1 release
notes.

Remove excessive linking to pg_ctl manual page.

Reorder incompatibility sections.

13 years agoMisc minor fixes to 9.1 release notes
Magnus Hagander [Sun, 20 Mar 2011 12:28:05 +0000 (13:28 +0100)]
Misc minor fixes to 9.1 release notes

Thom Brown

13 years agoWord-wrap 9.1 release note lines.
Bruce Momjian [Sun, 20 Mar 2011 04:27:23 +0000 (00:27 -0400)]
Word-wrap 9.1 release note lines.

13 years agoAdd links to documentation from 9.1 release notes.
Bruce Momjian [Sun, 20 Mar 2011 03:58:53 +0000 (23:58 -0400)]
Add links to documentation from 9.1 release notes.

13 years agoRevise collation derivation method and expression-tree representation.
Tom Lane [Sun, 20 Mar 2011 00:29:08 +0000 (20:29 -0400)]
Revise collation derivation method and expression-tree representation.

All expression nodes now have an explicit output-collation field, unless
they are known to only return a noncollatable data type (such as boolean
or record).  Also, nodes that can invoke collation-aware functions store
a separate field that is the collation value to pass to the function.
This avoids confusion that arises when a function has collatable inputs
and noncollatable output type, or vice versa.

Also, replace the parser's on-the-fly collation assignment method with
a post-pass over the completed expression tree.  This allows us to use
a more complex (and hopefully more nearly spec-compliant) assignment
rule without paying for it in extra storage in every expression node.

Fix assorted bugs in the planner's handling of collations by making
collation one of the defining properties of an EquivalenceClass and
by converting CollateExprs into discardable RelabelType nodes during
expression preprocessing.

13 years agoAdd SGML markup for keywords, terms, functions, commands, etc.
Bruce Momjian [Sat, 19 Mar 2011 21:21:49 +0000 (17:21 -0400)]
Add SGML markup for keywords, terms, functions, commands, etc.

13 years agoInitial version of PG 9.1 release notes.
Bruce Momjian [Sat, 19 Mar 2011 20:39:17 +0000 (16:39 -0400)]
Initial version of PG 9.1 release notes.

13 years agoRename ident authentication over local connections to peer
Magnus Hagander [Sat, 19 Mar 2011 17:44:35 +0000 (18:44 +0100)]
Rename ident authentication over local connections to peer

This removes an overloading of two authentication options where
one is very secure (peer) and one is often insecure (ident). Peer
is also the name used in libpq from 9.1 to specify the same type
of authentication.

Also make initdb select peer for local connections when ident is
chosen, and ident for TCP connections when peer is chosen.

ident keyword in pg_hba.conf is still accepted and maps to peer
authentication.

13 years agoProperly clear the verbose output of pg_basebackup
Magnus Hagander [Sat, 19 Mar 2011 15:38:50 +0000 (16:38 +0100)]
Properly clear the verbose output of pg_basebackup

Don't leave partial filename information on the last status or
dangling characters if the previous filename was longer than
the current one.

13 years agoRemove "all balls" mention of network address; change IPv6 address to ::/0.
Bruce Momjian [Sat, 19 Mar 2011 14:04:34 +0000 (10:04 -0400)]
Remove "all balls" mention of network address;  change IPv6 address to ::/0.

13 years agoFix possible "tuple concurrently updated" error in ALTER TABLE.
Robert Haas [Sat, 19 Mar 2011 02:09:57 +0000 (22:09 -0400)]
Fix possible "tuple concurrently updated" error in ALTER TABLE.

When adding an inheritance parent to a table, an AccessShareLock on the
parent isn't strong enough to prevent trouble, so take
ShareUpdateExclusiveLock instead.  Since this is a behavior change,
albeit a fairly unobtrusive one, and since we have only one report
from the field, no back-patch.

Report by Jon Nelson, analysis by Alvaro Herrera, fix by me.

13 years agoMove synchronous_standbys_defined updates from WAL writer to BG writer.
Robert Haas [Sat, 19 Mar 2011 01:43:45 +0000 (21:43 -0400)]
Move synchronous_standbys_defined updates from WAL writer to BG writer.

This is advantageous because the BG writer is alive until much later in
the shutdown sequence than WAL writer; we want to make sure that it's
possible to shut off synchronous replication during a smart shutdown,
else it might not be possible to complete the shutdown at all.

Per very reasonable gripes from Fujii Masao and Simon Riggs.

13 years agoWoops. Previous patch to remove -X options missed getopt_long calls().
Robert Haas [Sat, 19 Mar 2011 01:22:43 +0000 (21:22 -0400)]
Woops.  Previous patch to remove -X options missed getopt_long calls().

Noted by Tom Lane.

13 years agoDocument that the timestamp returned by pg_last_xact_replay_timestamp is
Bruce Momjian [Fri, 18 Mar 2011 23:18:08 +0000 (19:18 -0400)]
Document that the timestamp returned by pg_last_xact_replay_timestamp is
the time on the primary that the transaction was generated.

13 years agoDocument the all-balls IPv6 address.
Bruce Momjian [Fri, 18 Mar 2011 22:41:02 +0000 (18:41 -0400)]
Document the all-balls IPv6 address.

13 years agoMake synchronous replication query cancel/die messages more consistent.
Robert Haas [Fri, 18 Mar 2011 14:20:22 +0000 (10:20 -0400)]
Make synchronous replication query cancel/die messages more consistent.

Per a gripe from Thom Brown about my previous commit in this area,
commit 9a56dc3389b9470031e9ef8e45c95a680982e01a.

13 years agoRemove ancient -X options to pg_dump, pg_dumpall, pg_restore.
Robert Haas [Fri, 18 Mar 2011 13:44:44 +0000 (09:44 -0400)]
Remove ancient -X options to pg_dump, pg_dumpall, pg_restore.

The last version in which these options were documented is now EOL, so
it's time to get rid of them for real.  We now use GNU-style long
options instead.

13 years agoRemove bogus semicolons in recoveryPausesHere.
Robert Haas [Fri, 18 Mar 2011 12:09:09 +0000 (08:09 -0400)]
Remove bogus semicolons in recoveryPausesHere.

Without this, the startup process goes into a tight loop, consuming
100% of one CPU and failing to respond to interrupts.

13 years agoMinor fixes for high availability documentation.
Robert Haas [Thu, 17 Mar 2011 19:46:44 +0000 (15:46 -0400)]
Minor fixes for high availability documentation.

Erik Rijkers and me

13 years agoProtocol documentation for Hot Standby feedback messages.
Robert Haas [Thu, 17 Mar 2011 19:10:55 +0000 (15:10 -0400)]
Protocol documentation for Hot Standby feedback messages.

13 years agoRemove bogus comment.
Robert Haas [Thu, 17 Mar 2011 18:43:14 +0000 (14:43 -0400)]
Remove bogus comment.

13 years agoFix PL/Python memory leak involving array slices
Alvaro Herrera [Thu, 17 Mar 2011 15:28:46 +0000 (12:28 -0300)]
Fix PL/Python memory leak involving array slices

Report and patch from Daniel Popowich, bug #5842
(with some debugging help from Alex Hunsaker)

13 years agoRaise maximum value of several timeout parameters
Peter Eisentraut [Thu, 17 Mar 2011 18:19:51 +0000 (20:19 +0200)]
Raise maximum value of several timeout parameters

The maximum value of deadlock_timeout, max_standby_archive_delay,
max_standby_streaming_delay, log_min_duration_statement, and
log_autovacuum_min_duration was INT_MAX/1000 milliseconds, which is
about 35min, which is too short for some practical uses.  Raise the
maximum value to INT_MAX; the code that uses the parameters already
supports that just fine.

13 years agoAdd pause_at_recovery_target to recovery.conf.sample; improve docs.
Robert Haas [Thu, 17 Mar 2011 18:02:41 +0000 (14:02 -0400)]
Add pause_at_recovery_target to recovery.conf.sample; improve docs.

Fujii Masao, but with the proposed behavior change reverted, and the
rest adjusted accordingly.

13 years agoFix various possible problems with synchronous replication.
Robert Haas [Thu, 17 Mar 2011 17:10:42 +0000 (13:10 -0400)]
Fix various possible problems with synchronous replication.

1. Don't ignore query cancel interrupts.  Instead, if the user asks to
cancel the query after we've already committed it, but before it's on
the standby, just emit a warning and let the COMMIT finish.

2. Don't ignore die interrupts (pg_terminate_backend or fast shutdown).
Instead, emit a warning message and close the connection without
acknowledging the commit.  Other backends will still see the effect of
the commit, but there's no getting around that; it's too late to abort
at this point, and ignoring die interrupts altogether doesn't seem like
a good idea.

3. If synchronous_standby_names becomes empty, wake up all backends
waiting for synchronous replication to complete.  Without this, someone
attempting to shut synchronous replication off could easily wedge the
entire system instead.

4. Avoid depending on the assumption that if a walsender updates
MyProc->syncRepState, we'll see the change even if we read it without
holding the lock.  The window for this appears to be quite narrow (and
probably doesn't exist at all on machines with strong memory ordering)
but protecting against it is practically free, so do that.

5. Remove useless state SYNC_REP_MUST_DISCONNECT, which isn't needed and
doesn't actually do anything.

There's still some further work needed here to make the behavior of fast
shutdown plausible, but that looks complex, so I'm leaving it for a
separate commit.  Review by Fujii Masao.

13 years agoDocument guc context values, and reference them from the config doc section.
Bruce Momjian [Thu, 17 Mar 2011 04:26:03 +0000 (00:26 -0400)]
Document guc context values, and reference them from the config doc section.

Tom Lane

13 years agoUse correct PATH separator for Cygwin in pg_regress.c.
Andrew Dunstan [Thu, 17 Mar 2011 04:06:52 +0000 (00:06 -0400)]
Use correct PATH separator for Cygwin in pg_regress.c.

This has been broken for years, and I'm not sure why it has not been
noticed before, but now a very modern Cygwin breaks on it, and the fix
is clearly correct. Backpatching to all live branches.

13 years agoTab completion for \pset format and \pset linestyle.
Robert Haas [Wed, 16 Mar 2011 21:46:15 +0000 (17:46 -0400)]
Tab completion for \pset format and \pset linestyle.

Pavel Stehule

13 years agoCorrections to CREATE FOREIGN TABLE documentation.
Robert Haas [Wed, 16 Mar 2011 12:11:24 +0000 (08:11 -0400)]
Corrections to CREATE FOREIGN TABLE documentation.

Omit incorrect = sign, and properly document server_name parameter.

Shigeru Hanada

13 years agoImprove handling of unknown-type literals in UNION/INTERSECT/EXCEPT.
Tom Lane [Wed, 16 Mar 2011 01:51:36 +0000 (21:51 -0400)]
Improve handling of unknown-type literals in UNION/INTERSECT/EXCEPT.

This patch causes unknown-type Consts to be coerced to the resolved output
type of the set operation at parse time.  Formerly such Consts were left
alone until late in the planning stage.  The disadvantage of that approach
is that it disables some optimizations, because the planner sees the set-op
leaf query as having different output column types than the overall set-op.
We saw an example of that in a recent performance gripe from Claudio
Freire.

Fixing such a Const requires scribbling on the leaf query in
transformSetOperationTree, but that should be all right since if the leaf
query's semantics depended on that output column, it would already have
resolved the unknown to something else.

Most of the bulk of this patch is a simple adjustment of
transformSetOperationTree's API so that upper levels can get at the
TargetEntry containing a Const to be replaced: it now returns a list of
TargetEntries, instead of just the bare expressions.

13 years agoRemove unused copy_dir() function from pg_upgrade.
Bruce Momjian [Wed, 16 Mar 2011 00:52:35 +0000 (20:52 -0400)]
Remove unused copy_dir() function from pg_upgrade.

13 years agoAdjust doc wording about the casting exceptions for composite types.
Bruce Momjian [Wed, 16 Mar 2011 00:47:31 +0000 (20:47 -0400)]
Adjust doc wording about the casting exceptions for composite types.

13 years agoAdd hint suggesting the use of EXECUTE CREATE TABLE ... AS for EXECUTE INTO.
Bruce Momjian [Wed, 16 Mar 2011 00:18:35 +0000 (20:18 -0400)]
Add hint suggesting the use of EXECUTE CREATE TABLE ... AS for EXECUTE INTO.

13 years agoFix spelling error in fuzzystrmatch docs.
Bruce Momjian [Tue, 15 Mar 2011 23:50:02 +0000 (19:50 -0400)]
Fix spelling error in fuzzystrmatch docs.

13 years agoWording adjustment for restart_after_crash entry
Bruce Momjian [Tue, 15 Mar 2011 16:43:39 +0000 (12:43 -0400)]
Wording adjustment for restart_after_crash entry

Specifically, mention that "restart" is disabled by this parameter.

13 years agoAdd database comments to template0 and postgres databases, and improve
Bruce Momjian [Tue, 15 Mar 2011 15:26:20 +0000 (11:26 -0400)]
Add database comments to template0 and postgres databases, and improve
the comments on the template1 database.  No catalog version bump because
they are just comments.

13 years agoMinor sync rep documentation improvements.
Robert Haas [Tue, 15 Mar 2011 15:25:04 +0000 (11:25 -0400)]
Minor sync rep documentation improvements.

- Make the name of the ID tag for the GUC entry match the GUC name.
- Clarify that synchronous_replication waits for xlog flush, not receipt.
- Mention that synchronous_replication won't wait if max_wal_senders=0.

13 years agoRemove 13 keywords that are used only for ROLE options.
Robert Haas [Tue, 15 Mar 2011 14:22:58 +0000 (10:22 -0400)]
Remove 13 keywords that are used only for ROLE options.

Review by Tom Lane.