]> granicus.if.org Git - postgresql/log
postgresql
12 years agoFix name collision between concurrent regression tests.
Tom Lane [Sun, 22 Jul 2012 04:01:19 +0000 (00:01 -0400)]
Fix name collision between concurrent regression tests.

Commit f5bcd398addcbeb785f0513cf28cba5d1ecd2c8a introduced a test using
a table named "circles" in inherit.sql.  Unfortunately, the concurrently
executed constraints test was already using that table name, so the
parallel regression tests would sometimes fail.  Rename table to dodge
the problem.  Per buildfarm.

12 years agoImprove copydir() code for the case that fsync is off.
Tom Lane [Sun, 22 Jul 2012 00:10:29 +0000 (20:10 -0400)]
Improve copydir() code for the case that fsync is off.

We should avoid calling sync_file_range or posix_fadvise in this case,
since (a) we don't really care if the data gets synced, and might as
well save the kernel calls; (b) at least on Linux we know that the
kernel might block us until it's scheduled the write.

Also, avoid making a useless second traversal of the directory tree
if we're not actually going to call fsync(2) after all.

12 years agoUse --nosync during make check's initdb call.
Tom Lane [Sat, 21 Jul 2012 23:56:22 +0000 (19:56 -0400)]
Use --nosync during make check's initdb call.

We left this out of commit b966dd6c4228d696b291c1cdcb5ab8c8475fefa8
so as to get some more buildfarm testing of the new fsync code in initdb.
But since no problems have turned up, it's probably time to save the
cycles.

12 years agoSuppress volatile-related warning seen in some compilers.
Tom Lane [Sat, 21 Jul 2012 23:39:03 +0000 (19:39 -0400)]
Suppress volatile-related warning seen in some compilers.

Antique versions of gcc complain about vars that are initialized outside
PG_TRY and then modified within it.  Rather than marking the var volatile,
expend one more line of code.

12 years agoAccount for SRFs in targetlists in planner rowcount estimates.
Tom Lane [Sat, 21 Jul 2012 21:45:07 +0000 (17:45 -0400)]
Account for SRFs in targetlists in planner rowcount estimates.

We made use of the ROWS estimate for set-returning functions used in FROM,
but not for those used in SELECT targetlists; which is a bit of an
oversight considering there are common usages that require the latter
approach.  Improve that.  (I had initially thought it might be worth
folding this into cost_qual_eval, but after investigation concluded that
that wouldn't be very helpful, so just do it separately.)  Per complaint
from David Johnston.

Back-patch to 9.2, but not further, for fear of destabilizing plan choices
in existing releases.

12 years agoRevert temporary patch to debug Windows breakage.
Robert Haas [Sat, 21 Jul 2012 02:31:19 +0000 (22:31 -0400)]
Revert temporary patch to debug Windows breakage.

This reverts commit 0a248208a038d7847ef98996c6b70209d782415c.

12 years agoRepair plpgsql_validator breakage.
Robert Haas [Sat, 21 Jul 2012 01:25:26 +0000 (21:25 -0400)]
Repair plpgsql_validator breakage.

Commit 3a0e4d36ebd7f477822d5bae41ba121a40d22ccc arranged to
reference stack-allocated variables after they were out of scope.
That's no good, so let's arrange to not do that after all.

12 years agoRemove now unneeded results file for disabled prepared transactions case.
Andrew Dunstan [Fri, 20 Jul 2012 20:30:34 +0000 (16:30 -0400)]
Remove now unneeded results file for disabled prepared transactions case.

12 years agoTemporary patch to try to debug why event trigger patch broke Windows.
Robert Haas [Fri, 20 Jul 2012 20:20:15 +0000 (16:20 -0400)]
Temporary patch to try to debug why event trigger patch broke Windows.

Apologies for the ugliness.

12 years agoRemove prepared transactions from main isolation test schedule.
Andrew Dunstan [Fri, 20 Jul 2012 19:51:40 +0000 (15:51 -0400)]
Remove prepared transactions from main isolation test schedule.

There is no point in running this test when prepared transactions are disabled,
which is the default. New make targets that include the test are provided. This
will save some useless waste of cycles on buildfarm machines.

Backpatch to 9.1 where these tests were introduced.

12 years agopg_dump: Simplify mkdir() error checking
Peter Eisentraut [Fri, 20 Jul 2012 19:34:11 +0000 (22:34 +0300)]
pg_dump: Simplify mkdir() error checking

mkdir() can check for errors itself.  We don't need to code that
ourselves again.

12 years agoconnoinherit may be true only for CHECK constraints
Alvaro Herrera [Fri, 20 Jul 2012 16:33:34 +0000 (12:33 -0400)]
connoinherit may be true only for CHECK constraints

The code was setting it true for other constraints, which is
bogus.  Doing so caused bogus catalog entries for such constraints, and
in particular caused an error to be raised when trying to drop a
constraint of types other than CHECK from a table that has children,
such as reported in bug #6712.

In 9.2, additionally ignore connoinherit=true for other constraint
types, to avoid having to force initdb; existing databases might already
contain bogus catalog entries.

Includes a catversion bump (in HEAD only).

Bug report from Miroslav Šulc
Analysis from Amit Kapila and Noah Misch; Amit also contributed the patch.

12 years agoFix whole-row Var evaluation to cope with resjunk columns (again).
Tom Lane [Fri, 20 Jul 2012 17:09:00 +0000 (13:09 -0400)]
Fix whole-row Var evaluation to cope with resjunk columns (again).

When a whole-row Var is reading the result of a subquery, we need it to
ignore any "resjunk" columns that the subquery might have evaluated for
GROUP BY or ORDER BY purposes.  We've hacked this area before, in commit
68e40998d058c1f6662800a648ff1e1ce5d99cba, but that fix only covered
whole-row Vars of named composite types, not those of RECORD type; and it
was mighty klugy anyway, since it just assumed without checking that any
extra columns in the result must be resjunk.  A proper fix requires getting
hold of the subquery's targetlist so we can actually see which columns are
resjunk (whereupon we can use a JunkFilter to get rid of them).  So bite
the bullet and add some infrastructure to make that possible.

Per report from Andrew Dunstan and additional testing by Merlin Moncure.
Back-patch to all supported branches.  In 8.3, also back-patch commit
292176a118da6979e5d368a4baf27f26896c99a5, which for some reason I had
not done at the time, but it's a prerequisite for this change.

12 years agoMake new event trigger facility actually do something.
Robert Haas [Fri, 20 Jul 2012 15:38:47 +0000 (11:38 -0400)]
Make new event trigger facility actually do something.

Commit 3855968f328918b6cd1401dd11d109d471a54d40 added syntax, pg_dump,
psql support, and documentation, but the triggers didn't actually fire.
With this commit, they now do.  This is still a pretty basic facility
overall because event triggers do not get a whole lot of information
about what the user is trying to do unless you write them in C; and
there's still no option to fire them anywhere except at the very
beginning of the execution sequence, but it's better than nothing,
and a good building block for future work.

Along the way, add a regression test for ALTER LARGE OBJECT, since
testing of event triggers reveals that we haven't got one.

Dimitri Fontaine and Robert Haas

12 years agoRethink checkpointer's fsync-request table representation.
Tom Lane [Thu, 19 Jul 2012 23:28:22 +0000 (19:28 -0400)]
Rethink checkpointer's fsync-request table representation.

Instead of having one hash table entry per relation/fork/segment, just have
one per relation, and use bitmapsets to represent which specific segments
need to be fsync'd.  This eliminates the need to scan the whole hash table
to implement FORGET_RELATION_FSYNC, which fixes the O(N^2) behavior
recently demonstrated by Jeff Janes for cases involving lots of TRUNCATE or
DROP TABLE operations during a single checkpoint cycle.  Per an idea from
Robert Haas.

(FORGET_DATABASE_FSYNC still sucks, but since dropping a database is a
pretty expensive operation anyway, we'll live with that.)

In passing, improve the delayed-unlink code: remove the pass over the list
in mdpreckpt, since it wasn't doing anything for us except supporting a
useless Assert in mdpostckpt, and fix mdpostckpt so that it will absorb
fsync requests every so often when clearing a large backlog of deletion
requests.

12 years agoSend only one FORGET_RELATION_FSYNC request when dropping a relation.
Tom Lane [Thu, 19 Jul 2012 17:07:33 +0000 (13:07 -0400)]
Send only one FORGET_RELATION_FSYNC request when dropping a relation.

We were sending one per fork, but a little bit of refactoring allows us
to send just one request with forknum == InvalidForkNumber.  This not only
reduces pressure on the shared-memory request queue, but saves repeated
traversals of the checkpointer's hash table.

12 years agoRefactor the way code is shared between some range type functions.
Heikki Linnakangas [Wed, 18 Jul 2012 19:34:06 +0000 (22:34 +0300)]
Refactor the way code is shared between some range type functions.

Functions like range_eq, range_before etc. are exposed at the SQL-level, but
they're also used internally by the GiST consistent support function. The
code sharing was done by a hack, TrickFunctionCall2, which relied on the
knowledge that all the functions used fn_extra the same way. This commit
splits the functions into internal versions that take a TypeCacheEntry as
argument, and thin wrappers to expose the functions at the SQL-level. The
internal versions can then be called directly and in a less hacky way from
the GiST consistent function.

This is just cosmetic, but backpatch to 9.2 anyway, to avoid having a
different version of this code in the 9.2 branch. That would make
backpatching fixes in this area more difficult.

Alexander Korotkov

12 years agoFix statistics breakage from bgwriter/checkpointer process split.
Tom Lane [Wed, 18 Jul 2012 19:40:31 +0000 (15:40 -0400)]
Fix statistics breakage from bgwriter/checkpointer process split.

ForwardFsyncRequest() supposed that it could only be called in regular
backends, which used to be true; but since the splitup of bgwriter and
checkpointer, it is also called in the bgwriter.  We do not want to count
such calls in pg_stat_bgwriter.buffers_backend statistics, so fix things
so that they aren't.

(It's worth noting here that this implies an alarmingly large increase in
the expected amount of cross-process fsync request traffic, which may well
mean that the process splitup was not such a hot idea.)

12 years agoFix management of pendingOpsTable in auxiliary processes.
Tom Lane [Wed, 18 Jul 2012 19:28:10 +0000 (15:28 -0400)]
Fix management of pendingOpsTable in auxiliary processes.

mdinit() was misusing IsBootstrapProcessingMode() to decide whether to
create an fsync pending-operations table in the current process.  This led
to creating a table not only in the startup and checkpointer processes as
intended, but also in the bgwriter process, not to mention other auxiliary
processes such as walwriter and walreceiver.  Creation of the table in the
bgwriter is fatal, because it absorbs fsync requests that should have gone
to the checkpointer; instead they just sit in bgwriter local memory and are
never acted on.  So writes performed by the bgwriter were not being fsync'd
which could result in data loss after an OS crash.  I think there is no
live bug with respect to walwriter and walreceiver because those never
perform any writes of shared buffers; but the potential is there for
future breakage in those processes too.

To fix, make AuxiliaryProcessMain() export the current process's
AuxProcType as a global variable, and then make mdinit() test directly for
the types of aux process that should have a pendingOpsTable.  Having done
that, we might as well also get rid of the random bool flags such as
am_walreceiver that some of the aux processes had grown.  (Note that we
could not have fixed the bug by examining those variables in mdinit(),
because it's called from BaseInit() which is run by AuxiliaryProcessMain()
before entering any of the process-type-specific code.)

Back-patch to 9.2, where the problem was introduced by the split-up of
bgwriter and checkpointer processes.  The bogus pendingOpsTable exists
in walwriter and walreceiver processes in earlier branches, but absent
any evidence that it causes actual problems there, I'll leave the older
branches alone.

12 years agoSyntax support and documentation for event triggers.
Robert Haas [Wed, 18 Jul 2012 14:16:16 +0000 (10:16 -0400)]
Syntax support and documentation for event triggers.

They don't actually do anything yet; that will get fixed in a
follow-on commit.  But this gets the basic infrastructure in place,
including CREATE/ALTER/DROP EVENT TRIGGER; support for COMMENT,
SECURITY LABEL, and ALTER EXTENSION .. ADD/DROP EVENT TRIGGER;
pg_dump and psql support; and documentation for the anticipated
initial feature set.

Dimitri Fontaine, with review and a bunch of additional hacking by me.
Thom Brown extensively reviewed earlier versions of this patch set,
but there's not a whole lot of that code left in this commit, as it
turns out.

12 years agoGet rid of useless global variable in pg_upgrade.
Tom Lane [Wed, 18 Jul 2012 05:23:12 +0000 (01:23 -0400)]
Get rid of useless global variable in pg_upgrade.

Since the scandir() emulation was taken out of pg_upgrade, there's
no longer any need for scandir_file_pattern to exist as a global
variable.  Replace it with a local in the one remaining function
that was making use of it.

12 years agoImprove pg_upgrade's load_directory() function.
Tom Lane [Wed, 18 Jul 2012 05:13:20 +0000 (01:13 -0400)]
Improve pg_upgrade's load_directory() function.

Error out on out-of-memory, rather than returning -1, which the sole
existing caller wasn't checking for anyway.  There doesn't seem to be
any use-case for making the caller check for failure here.

Detect failure return from readdir().

Use a less platform-dependent method of calculating the entrysize.
It's possible, but not yet confirmed, that this explains bug #6733,
in which Mike Wilson reports a pg_upgrade crash that did not occur
in 9.1.  (Note that load_directory is effectively new code in 9.2,
at least on platforms that have scandir().)

Fix up comments, avoid uselessly using two counters, reduce the number
of realloc calls to something sane.

12 years agoImprove coding around the fsync request queue.
Tom Lane [Tue, 17 Jul 2012 20:55:39 +0000 (16:55 -0400)]
Improve coding around the fsync request queue.

In all branches back to 8.3, this patch fixes a questionable assumption in
CompactCheckpointerRequestQueue/CompactBgwriterRequestQueue that there are
no uninitialized pad bytes in the request queue structs.  This would only
cause trouble if (a) there were such pad bytes, which could happen in 8.4
and up if the compiler makes enum ForkNumber narrower than 32 bits, but
otherwise would require not-currently-planned changes in the widths of
other typedefs; and (b) the kernel has not uniformly initialized the
contents of shared memory to zeroes.  Still, it seems a tad risky, and we
can easily remove any risk by pre-zeroing the request array for ourselves.
In addition to that, we need to establish a coding rule that struct
RelFileNode can't contain any padding bytes, since such structs are copied
into the request array verbatim.  (There are other places that are assuming
this anyway, it turns out.)

In 9.1 and up, the risk was a bit larger because we were also effectively
assuming that struct RelFileNodeBackend contained no pad bytes, and with
fields of different types in there, that would be much easier to break.
However, there is no good reason to ever transmit fsync or delete requests
for temp files to the bgwriter/checkpointer, so we can revert the request
structs to plain RelFileNode, getting rid of the padding risk and saving
some marginal number of bytes and cycles in fsync queue manipulation while
we are at it.  The savings might be more than marginal during deletion of
a temp relation, because the old code transmitted an entirely useless but
nonetheless expensive-to-process ForgetRelationFsync request to the
background process, and also had the background process perform the file
deletion even though that can safely be done immediately.

In addition, make some cleanup of nearby comments and small improvements to
the code in CompactCheckpointerRequestQueue/CompactBgwriterRequestQueue.

12 years agoPL/Python: Remove PLy_result_ass_item
Peter Eisentraut [Tue, 17 Jul 2012 20:26:49 +0000 (23:26 +0300)]
PL/Python: Remove PLy_result_ass_item

It is apparently no longer used after the new slicing support was
implemented (a97207b6908f1d4a7d19b37b818367bb0171039f), so let's
remove the dead code and see if anything cares.

12 years agoShow step titles in the pg_upgrade man page
Peter Eisentraut [Tue, 17 Jul 2012 18:34:22 +0000 (21:34 +0300)]
Show step titles in the pg_upgrade man page

The upstream XSLT stylesheets missed that case.

found by Álvaro Herrera

12 years agoRemove recently added PL/Perl encoding tests
Alvaro Herrera [Tue, 17 Jul 2012 16:31:48 +0000 (12:31 -0400)]
Remove recently added PL/Perl encoding tests

These only pass cleanly on UTF8 and SQL_ASCII encodings, besides the
Japanese encoding in which they were originally written, which is clearly
not good enough.  Since the functionality they test has not ever been
tested from PL/Perl, the best answer seems to be to remove the new tests
completely.

Per buildfarm results and ensuing discussion.

12 years agoPut back storage/proc.h in postmaster.c.
Tom Lane [Tue, 17 Jul 2012 14:14:06 +0000 (10:14 -0400)]
Put back storage/proc.h in postmaster.c.

I took this out thinking it wasn't needed anymore, but the EXEC_BACKEND
code still needs it.  Per buildfarm.

12 years agoIntroduce timeout handling framework
Alvaro Herrera [Mon, 16 Jul 2012 22:43:21 +0000 (18:43 -0400)]
Introduce timeout handling framework

Management of timeouts was getting a little cumbersome; what we
originally had was more than enough back when we were only concerned
about deadlocks and query cancel; however, when we added timeouts for
standby processes, the code got considerably messier.  Since there are
plans to add more complex timeouts, this seems a good time to introduce
a central timeout handling module.

External modules register their timeout handlers during process
initialization, and later enable and disable them as they see fit using
a simple API; timeout.c is in charge of keeping track of which timeouts
are in effect at any time, installing a common SIGALRM signal handler,
and calling setitimer() as appropriate to ensure timely firing of
external handlers.

timeout.c additionally supports pluggable modules to add their own
timeouts, though this capability isn't exercised anywhere yet.

Additionally, as of this commit, walsender processes are aware of
timeouts; we had a preexisting bug there that made those ignore SIGALRM,
thus being subject to unhandled deadlocks, particularly during the
authentication phase.  This has already been fixed in back branches in
commit 0bf8eb2a, which see for more details.

Main author: Zoltán Böszörményi
Some review and cleanup by Álvaro Herrera
Extensive reworking by Tom Lane

12 years agoRemove unreachable code
Peter Eisentraut [Mon, 16 Jul 2012 19:15:03 +0000 (22:15 +0300)]
Remove unreachable code

The Solaris Studio compiler warns about these instances, unlike more
mainstream compilers such as gcc.  But manual inspection showed that
the code is clearly not reachable, and we hope no worthy compiler will
complain about removing this code.

12 years agoAdd comment why seemingly dead code is necessary
Peter Eisentraut [Mon, 16 Jul 2012 19:08:04 +0000 (22:08 +0300)]
Add comment why seemingly dead code is necessary

12 years agoAvoid pre-determining index names during CREATE TABLE LIKE parsing.
Tom Lane [Mon, 16 Jul 2012 17:25:18 +0000 (13:25 -0400)]
Avoid pre-determining index names during CREATE TABLE LIKE parsing.

Formerly, when trying to copy both indexes and comments, CREATE TABLE LIKE
had to pre-assign names to indexes that had comments, because it made up an
explicit CommentStmt command to apply the comment and so it had to know the
name for the index.  This creates bad interactions with other indexes, as
shown in bug #6734 from Daniele Varrazzo: the preassignment logic couldn't
take any other indexes into account so it could choose a conflicting name.

To fix, add a field to IndexStmt that allows it to carry a comment to be
assigned to the new index.  (This isn't a user-exposed feature of CREATE
INDEX, only an internal option.)  Now we don't need preassignment of index
names in any situation.

I also took the opportunity to refactor DefineIndex to accept the IndexStmt
as such, rather than passing all its fields individually in a mile-long
parameter list.

Back-patch to 9.2, but no further, because it seems too dangerous to change
IndexStmt or DefineIndex's API in released branches.  The bug exists back
to 9.0 where CREATE TABLE LIKE grew the ability to copy comments, but given
the lack of prior complaints we'll just let it go unfixed before 9.2.

12 years agoPrevent corner-case core dump in rfree().
Tom Lane [Sun, 15 Jul 2012 17:27:54 +0000 (13:27 -0400)]
Prevent corner-case core dump in rfree().

rfree() failed to cope with the case that pg_regcomp() had initialized the
regex_t struct but then failed to allocate any memory for re->re_guts (ie,
the first malloc call in pg_regcomp() failed).  It would try to touch the
guts struct anyway, and thus dump core.  This is a sufficiently narrow
corner case that it's not surprising it's never been seen in the field;
but still a bug is a bug, so patch all active branches.

Noted while investigating whether we need to call pg_regfree after a
failure return from pg_regcomp.  Other than this bug, it turns out we
don't, so adjust comments appropriately.

12 years agoDon't initialize TLI variable to -1, as TimeLineID is unsigned.
Heikki Linnakangas [Sat, 14 Jul 2012 18:02:04 +0000 (21:02 +0300)]
Don't initialize TLI variable to -1, as TimeLineID is unsigned.

This was causing a compiler warning with Solaris compiler. Use 0 instead.
The variable is initialized just for the sake of tidyness  and/or debugging,
it's not used for anything before setting it to a real value.

Per report and suggestion from Peter Eisentraut.

12 years agoPrint the name of the WAL file containing latest REDO ptr in pg_controldata.
Heikki Linnakangas [Sat, 14 Jul 2012 11:17:43 +0000 (14:17 +0300)]
Print the name of the WAL file containing latest REDO ptr in pg_controldata.

This makes it easier to determine how far back you need to keep archived WAL
files, to restore from a backup.

Fujii Masao

12 years agoAdd link to PEP 394 regarding python2 vs python3 naming
Peter Eisentraut [Sat, 14 Jul 2012 10:06:09 +0000 (13:06 +0300)]
Add link to PEP 394 regarding python2 vs python3 naming

12 years agoAdd fsync capability to initdb, and use sync_file_range() if available.
Tom Lane [Fri, 13 Jul 2012 21:16:58 +0000 (17:16 -0400)]
Add fsync capability to initdb, and use sync_file_range() if available.

Historically we have not worried about fsync'ing anything during initdb
(in fact, initdb intentionally passes -F to each backend launch to prevent
it from fsync'ing).  But with filesystems getting more aggressive about
caching data, that's not such a good plan anymore.  Make initdb do a pass
over the finished data directory tree to fsync everything.  For testing
purposes, the -N/--nosync flag can be used to restore the old behavior.

Also, testing shows that on Linux, sync_file_range() is much faster than
posix_fadvise() for hinting to the kernel that an fsync is coming,
apparently because the latter blocks on a rather small request queue while
the former doesn't.  So use this function if available in initdb, and also
in the backend's pg_flush_data() (where it currently will affect only the
speed of CREATE DATABASE's cloning step).

We will later make pg_regress invoke initdb with the --nosync flag
to avoid slowing down cases such as "make check" in contrib.  But
let's not do so until we've shaken out any portability issues in this
patch.

Jeff Davis, reviewed by Andres Freund

12 years agoCosmetic cleanup of ginInsertValue().
Tom Lane [Fri, 13 Jul 2012 15:37:39 +0000 (11:37 -0400)]
Cosmetic cleanup of ginInsertValue().

Make it clearer that the passed stack mustn't be empty, and that we
are not supposed to fall off the end of the stack in the main loop.
Tighten the loop that extracts the root block number, too.

Markus Wanner and Tom Lane

12 years agoAvoid extra newlines in XML mapping in table forest mode
Peter Eisentraut [Thu, 12 Jul 2012 20:47:33 +0000 (23:47 +0300)]
Avoid extra newlines in XML mapping in table forest mode

found by P. Broennimann

12 years agoSkip text->binary conversion of unnecessary columns in contrib/file_fdw.
Tom Lane [Thu, 12 Jul 2012 20:26:59 +0000 (16:26 -0400)]
Skip text->binary conversion of unnecessary columns in contrib/file_fdw.

When reading from a text- or CSV-format file in file_fdw, the datatype
input routines can consume a significant fraction of the runtime.
Often, the query does not need all the columns, so we can get a useful
speed boost by skipping I/O conversion for unnecessary columns.

To support this, add a "convert_selectively" option to the core COPY code.
This is undocumented and not accessible from SQL (for now, anyway).

Etsuro Fujita, reviewed by KaiGai Kohei

12 years agoRemove 'x =- 1' check for pgindent, not needed, per report from Andrew
Bruce Momjian [Thu, 12 Jul 2012 18:37:35 +0000 (14:37 -0400)]
Remove 'x =- 1' check for pgindent, not needed, per report from Andrew
Dunstan.

12 years agoFix memory and file descriptor leaks in pg_receivexlog/pg_basebackup
Magnus Hagander [Thu, 12 Jul 2012 11:31:19 +0000 (13:31 +0200)]
Fix memory and file descriptor leaks in pg_receivexlog/pg_basebackup

When the internal loop mode was added, freeing memory and closing
filedescriptors before returning became important, and a few cases
in the code missed that.

Fujii Masao

12 years agoAdd array_remove() and array_replace() functions.
Tom Lane [Wed, 11 Jul 2012 17:59:35 +0000 (13:59 -0400)]
Add array_remove() and array_replace() functions.

These functions support removing or replacing array element value(s)
matching a given search value.  Although intended mainly to support a
future array-foreign-key feature, they seem useful in their own right.

Marco Nenciarini and Gabriele Bartolini, reviewed by Alex Hunsaker

12 years agoDocument that Log-Shipping Standby Servers cannot be upgraded by
Bruce Momjian [Wed, 11 Jul 2012 03:08:19 +0000 (23:08 -0400)]
Document that Log-Shipping Standby Servers cannot be upgraded by
pg_upgrade.

Backpatch to 9.2.

12 years agoFix bogus macro definition.
Tom Lane [Wed, 11 Jul 2012 02:36:11 +0000 (22:36 -0400)]
Fix bogus macro definition.

Per buildfarm complaints.

12 years agoAdd comments about additional mule-internal charsets from emacs's
Tatsuo Ishii [Tue, 10 Jul 2012 23:10:50 +0000 (08:10 +0900)]
Add comments about additional mule-internal charsets from emacs's
source code(lisp/international/mule-conf.el).  These charsets have not
been supported up to now anyway, so this is just for adding
commentary.  Also add mention that we follow emacs's implementation,
not xemacs's.

12 years agoFix ASCII case in pg_wchar2mule_with_len.
Tom Lane [Tue, 10 Jul 2012 19:58:36 +0000 (15:58 -0400)]
Fix ASCII case in pg_wchar2mule_with_len.

Also some cosmetic improvements for wchar-to-mblen patch.

12 years agoplperl: Skip setting UTF8 flag when in SQL_ASCII encoding
Alvaro Herrera [Mon, 9 Jul 2012 21:36:29 +0000 (17:36 -0400)]
plperl: Skip setting UTF8 flag when in SQL_ASCII encoding

When in SQL_ASCII encoding, strings passed around are not necessarily
UTF8-safe.  We had already fixed this in some places, but it looks like
we missed some.

I had to backpatch Peter Eisentraut's a8b92b60 to 9.1 in order for this
patch to cherry-pick more cleanly.

Patch from Alex Hunsaker, tweaked by Kyotaro HORIGUCHI and myself.

Some desultory cleanup and comment addition by me, during patch review.

Per bug report from Christoph Berg in
20120209102116.GA14429@msgid.df7cb.de

12 years agoperltidy adjustments to new file
Alvaro Herrera [Mon, 9 Jul 2012 21:35:57 +0000 (17:35 -0400)]
perltidy adjustments to new file

12 years agoRe-implement extraction of fixed prefixes from regular expressions.
Tom Lane [Tue, 10 Jul 2012 18:54:37 +0000 (14:54 -0400)]
Re-implement extraction of fixed prefixes from regular expressions.

To generate btree-indexable conditions from regex WHERE conditions (such as
WHERE indexed_col ~ '^foo'), we need to be able to identify any fixed
prefix that a regex might have; that is, find any string that must be a
prefix of all strings satisfying the regex.  We used to do that with
entirely ad-hoc code that looked at the source text of the regex.  It
didn't know very much about regex syntax, which mostly meant that it would
fail to identify some optimizable cases; but Viktor Rosenfeld reported that
it would produce actively wrong answers for quantified parenthesized
subexpressions, such as '^(foo)?bar'.  Rather than trying to extend the
ad-hoc code to cover this, let's get rid of it altogether in favor of
identifying prefixes by examining the compiled form of a regex.

To do this, I've added a new entry point "pg_regprefix" to the regex library;
hopefully it is defined in a sufficiently general fashion that it can remain
in the library when/if that code gets split out as a standalone project.

Since this bug has been there for a very long time, this fix needs to get
back-patched.  However it depends on some other recent commits (particularly
the addition of wchar-to-database-encoding conversion), so I'll commit this
separately and then go to work on back-porting the necessary fixes.

12 years agoRefactor pattern_fixed_prefix() to avoid dealing in incomplete patterns.
Tom Lane [Tue, 10 Jul 2012 03:22:55 +0000 (23:22 -0400)]
Refactor pattern_fixed_prefix() to avoid dealing in incomplete patterns.

Previously, pattern_fixed_prefix() was defined to return whatever fixed
prefix it could extract from the pattern, plus the "rest" of the pattern.
That definition was sensible for LIKE patterns, but not so much for
regexes, where reconstituting a valid pattern minus the prefix could be
quite tricky (certainly the existing code wasn't doing that correctly).
Since the only thing that callers ever did with the "rest" of the pattern
was to pass it to like_selectivity() or regex_selectivity(), let's cut out
the middle-man and just have pattern_fixed_prefix's subroutines do this
directly.  Then pattern_fixed_prefix can return a simple selectivity
number, and the question of how to cope with partial patterns is removed
from its API specification.

While at it, adjust the API spec so that callers who don't actually care
about the pattern's selectivity (which is a lot of them) can pass NULL for
the selectivity pointer to skip doing the work of computing a selectivity
estimate.

This patch is only an API refactoring that doesn't actually change any
processing, other than allowing a little bit of useless work to be skipped.
However, it's necessary infrastructure for my upcoming fix to regex prefix
extraction, because after that change there won't be any simple way to
identify the "rest" of the regex, not even to the low level of fidelity
needed by regex_selectivity.  We can cope with that if regex_fixed_prefix
and regex_selectivity communicate directly, but not if we have to work
within the old API.  Hence, back-patch to all active branches.

12 years agoFix planner to pass correct collation to operator selectivity estimators.
Tom Lane [Mon, 9 Jul 2012 03:51:08 +0000 (23:51 -0400)]
Fix planner to pass correct collation to operator selectivity estimators.

We can do this without creating an API break for estimation functions
by passing the collation using the existing fmgr functionality for
passing an input collation as a hidden parameter.

The need for this was foreseen at the outset, but we didn't get around to
making it happen in 9.1 because of the decision to sort all pg_statistic
histograms according to the database's default collation.  That meant that
selectivity estimators generally need to use the default collation too,
even if they're estimating for an operator that will do something
different.  The reason it's suddenly become more interesting is that
regexp interpretation also uses a collation (for its LC_TYPE not LC_COLLATE
property), and we no longer want to use the wrong collation when examining
regexps during planning.  It's not that the selectivity estimate is likely
to change much from this; rather that we are thinking of caching compiled
regexps during planner estimation, and we won't get the intended benefit
if we cache them with a different collation than the executor will use.

Back-patch to 9.1, both because the regexp change is likely to get
back-patched and because we might as well get this right in all
collation-supporting branches, in case any third-party code wants to
rely on getting the collation.  The patch turns out to be minuscule
now that I've done it ...

12 years agoSimplify and document regex library's compact-NFA representation.
Tom Lane [Sat, 7 Jul 2012 21:39:50 +0000 (17:39 -0400)]
Simplify and document regex library's compact-NFA representation.

The previous coding abused the first element of a cNFA state's arcs list
to hold a per-state flag bit, which was confusing, undocumented, and not
even particularly efficient.  Get rid of that in favor of a separate
"stflags" vector.  Since there's only one bit in use, I chose to allocate a
char per state; we could possibly replace this with a bitmap at some point,
but that would make accesses a little slower.  It's already about 8X
smaller than before, so let's not get overly tense.

Also document the representation better than it was before, which is to say
not at all.

This patch is a byproduct of investigations towards extracting a "fixed
prefix" string from the compact-NFA representation of regex patterns.
Might need to back-patch it if we decide to back-patch that fix, but for
now it's just code cleanup so I'll just put it in HEAD.

12 years agoConvert libpq regress script to Perl
Alvaro Herrera [Fri, 6 Jul 2012 19:38:11 +0000 (15:38 -0400)]
Convert libpq regress script to Perl

This should ease its use on the Windows build environment.

12 years agoUpdate libpq test expected output
Alvaro Herrera [Fri, 6 Jul 2012 20:23:30 +0000 (16:23 -0400)]
Update libpq test expected output

Commit 2b443063 changed wording for some of the error messages, but
neglected updating the regress output to match.

12 years agoRun updated copyright.pl on HEAD and 9.2 trees, updating the psql
Bruce Momjian [Fri, 6 Jul 2012 16:28:18 +0000 (12:28 -0400)]
Run updated copyright.pl on HEAD and 9.2 trees, updating the psql
\copyright output to 2012.

Backpatch to 9.2.

12 years agoHave copyright.pl skip updating something that is just the current year,
Bruce Momjian [Fri, 6 Jul 2012 16:21:43 +0000 (12:21 -0400)]
Have copyright.pl skip updating something that is just the current year,
to avoid producing dups, e.g. 2012-2012

Backpatch to 9.2.

12 years agoModify copyright.pl so all lines are processed, not just the first
Bruce Momjian [Fri, 6 Jul 2012 15:58:55 +0000 (11:58 -0400)]
Modify copyright.pl so all lines are processed, not just the first
match, so files that contain embedded copyrights are updated, e.g.
pgsql/help.c.

Backpatch to 9.2.

12 years agoFix copyright.pl to properly skip the .git directory by adding a
Bruce Momjian [Fri, 6 Jul 2012 15:43:59 +0000 (11:43 -0400)]
Fix copyright.pl to properly skip the .git directory by adding a
basename() qualification.

12 years agoFix spacing in copyright.pl after being run with missing regex slash
Bruce Momjian [Fri, 6 Jul 2012 14:57:08 +0000 (10:57 -0400)]
Fix spacing in copyright.pl after being run with missing regex slash
(now added).

Backpatch to 9.2.

12 years agoUpdate pg_upgrade comments for recent configpath fix.
Bruce Momjian [Fri, 6 Jul 2012 13:39:22 +0000 (09:39 -0400)]
Update pg_upgrade comments for recent configpath fix.

12 years agoFix failure of new wchar->mb functions to advance from pointer.
Robert Haas [Fri, 6 Jul 2012 03:47:53 +0000 (23:47 -0400)]
Fix failure of new wchar->mb functions to advance from pointer.

Bug spotted by Tom Lane.

12 years agoFix PGDATAOLD and PGDATANEW to properly set pgconfig location, per
Bruce Momjian [Fri, 6 Jul 2012 03:36:30 +0000 (23:36 -0400)]
Fix PGDATAOLD and PGDATANEW to properly set pgconfig location, per
report from Tom.

Backpatch to 9.2.

12 years agoDon't try to trim "../" in join_path_components().
Tom Lane [Thu, 5 Jul 2012 21:15:05 +0000 (17:15 -0400)]
Don't try to trim "../" in join_path_components().

join_path_components() tried to remove leading ".." components from its
tail argument, but it was not nearly bright enough to do so correctly
unless the head argument was (a) absolute and (b) canonicalized.
Rather than try to fix that logic, let's just get rid of it: there is no
correctness reason to remove "..", and cosmetic concerns can be taken
care of by a subsequent canonicalize_path() call.  Per bug #6715 from
Greg Davidson.

Back-patch to all supported branches.  It appears that pre-9.2, this
function is only used with absolute paths as head arguments, which is why
we'd not noticed the breakage before.  However, third-party code might be
expecting this function to work in more general cases, so it seems wise
to back-patch.

In HEAD and 9.2, also make some minor cosmetic improvements to callers.

12 years agoRevert part of the previous patch that avoided using PLy_elog().
Heikki Linnakangas [Thu, 5 Jul 2012 20:40:25 +0000 (23:40 +0300)]
Revert part of the previous patch that avoided using PLy_elog().

That caused the plpython_unicode regression test to fail on SQL_ASCII
encoding, as evidenced by the buildfarm. The reason is that with the patch,
you don't get the detail in the error message that you got before. That
detail is actually very informative, so rather than just adjust the expected
output, let's revert that part of the patch for now to make the buildfarm
green again, and figure out some other way to avoid the recursion of
PLy_elog() that doesn't lose the detail.

12 years agoFix mapping of PostgreSQL encodings to Python encodings.
Heikki Linnakangas [Thu, 5 Jul 2012 18:45:24 +0000 (21:45 +0300)]
Fix mapping of PostgreSQL encodings to Python encodings.

Windows encodings, "win1252" and so forth, are named differently in Python,
like "cp1252". Also, if the PyUnicode_AsEncodedString() function call fails
for some reason, use a plain ereport(), not a PLy_elog(), to report that
error. That avoids recursion and crash, if PLy_elog() tries to call
PLyUnicode_Bytes() again.

This fixes bug reported by Asif Naeem. Backpatch down to 9.0, before that
plpython didn't even try these conversions.

Jan Urbański, with minor comment improvements by me.

12 years agoRemove support for using wait3() in place of waitpid().
Tom Lane [Thu, 5 Jul 2012 18:00:40 +0000 (14:00 -0400)]
Remove support for using wait3() in place of waitpid().

All Unix-oid platforms that we currently support should have waitpid(),
since it's in V2 of the Single Unix Spec.  Our git history shows that
the wait3 code was added to support NextStep, which we officially dropped
support for as of 9.2.  So get rid of the configure test, and simplify the
macro spaghetti in reaper().  Per suggestion from Fujii Masao.

12 years agopg_upgrade: abstract out copying of files from old cluster to new
Alvaro Herrera [Thu, 5 Jul 2012 15:38:42 +0000 (11:38 -0400)]
pg_upgrade: abstract out copying of files from old cluster to new

Currently only pg_clog is copied, but some other directories could need
the same treatment as well, so create a subroutine to do it.

Extracted from my (somewhat larger) FOR KEY SHARE patch.

12 years agoFix function argument tab completion for schema-qualified or quoted function names
Magnus Hagander [Thu, 5 Jul 2012 12:03:49 +0000 (14:03 +0200)]
Fix function argument tab completion for schema-qualified or quoted function names

Dean Rasheed, reviewed by Josh Kupershmidt

12 years agoFix missing regex slash that caused perltidy to get confused on
Bruce Momjian [Thu, 5 Jul 2012 01:58:48 +0000 (21:58 -0400)]
Fix missing regex slash that caused perltidy to get confused on
copyright.pl.

Backpatch to 9.2.

12 years agoRun newly-configured perltidy script on Perl files.
Bruce Momjian [Thu, 5 Jul 2012 01:47:49 +0000 (21:47 -0400)]
Run newly-configured perltidy script on Perl files.

Run on HEAD and 9.2.

12 years agoReduce messages about implicit indexes and sequences to DEBUG1.
Robert Haas [Thu, 5 Jul 2012 00:34:24 +0000 (20:34 -0400)]
Reduce messages about implicit indexes and sequences to DEBUG1.

Per recent discussion on pgsql-hackers, these messages are too
chatty for most users.

12 years agoHave pg_dump in binary-upgrade mode properly drop user-created
Bruce Momjian [Wed, 4 Jul 2012 21:36:50 +0000 (17:36 -0400)]
Have pg_dump in binary-upgrade mode properly drop user-created
extensions that might exist in the new empty cluster databases, like
plpgsql.

Backpatch to 9.2.

12 years agoFix sample INSTR function to return 0 if third arg is 0.
Robert Haas [Wed, 4 Jul 2012 21:19:23 +0000 (17:19 -0400)]
Fix sample INSTR function to return 0 if third arg is 0.

Albe Laurenz, per a report by Greg Smith that our sample function
doesn't quite match Oracle's behavior.

12 years agoAdd wchar -> mb conversion routines.
Robert Haas [Wed, 4 Jul 2012 21:10:10 +0000 (17:10 -0400)]
Add wchar -> mb conversion routines.

This is infrastructure for Alexander Korotkov's work on indexing regular
expression searches.

Alexander Korotkov, with a bit of further hackery on the MULE conversion
by me

12 years agoMore doc cleanups for recent shared memory changes.
Robert Haas [Wed, 4 Jul 2012 19:57:48 +0000 (15:57 -0400)]
More doc cleanups for recent shared memory changes.

Josh Kupershmidt

12 years agoDocumentation cleanups for recent shared memory changes.
Robert Haas [Wed, 4 Jul 2012 19:56:12 +0000 (15:56 -0400)]
Documentation cleanups for recent shared memory changes.

12 years agoIncrease the maximum initdb-configured value for shared_buffers to 128MB.
Robert Haas [Wed, 4 Jul 2012 19:52:35 +0000 (15:52 -0400)]
Increase the maximum initdb-configured value for shared_buffers to 128MB.

The old value of 32MB has been around for a very long time, and in the
meantime typical system memories have become vastly larger.  Also, now
that we no longer depend on being able to fit the entirety of our
shared memory segment into the system's limit on System V shared
memory, there's a much better chance of the higher limit actually
proving productive.

Per recent discussion on pgsql-hackers.

12 years agoMake oid2name, pgbench, and vacuumlo set fallback_application_name.
Robert Haas [Wed, 4 Jul 2012 19:39:33 +0000 (15:39 -0400)]
Make oid2name, pgbench, and vacuumlo set fallback_application_name.

Amit Kapila, reviewed by Shigeru Hanada and Peter Eisentraut,
with some modifications by me.

12 years agoRemove duplicate, unnecessary, variable declaration
Magnus Hagander [Wed, 4 Jul 2012 14:16:54 +0000 (16:16 +0200)]
Remove duplicate, unnecessary, variable declaration

12 years agoSet the write location in the pg_receivexlog status messages
Magnus Hagander [Wed, 4 Jul 2012 13:13:09 +0000 (15:13 +0200)]
Set the write location in the pg_receivexlog status messages

This makes it possible for the master to track how much data has
actually been written my pg_receivexlog - and not just how much
has been sent towards it.

12 years agoAlways treat a standby returning an an invalid flush location as async
Magnus Hagander [Wed, 4 Jul 2012 13:10:46 +0000 (15:10 +0200)]
Always treat a standby returning an an invalid flush location as async

This ensures that a standby such as pg_receivexlog will not be selected
as sync standby - which would cause the master to block waiting for
a location that could never happen.

Fujii Masao

12 years agoRemove reference to default wal_buffers being 8
Magnus Hagander [Wed, 4 Jul 2012 07:22:21 +0000 (09:22 +0200)]
Remove reference to default wal_buffers being 8

This hasn't been true since 9.1, when the default was changed to -1.
Remove the reference completely, keeping the discussion of the parameter
and it's shared memory effects on the config page.

12 years agoRemove references to pgfoundry as recommended hosting platform
Magnus Hagander [Wed, 4 Jul 2012 06:59:35 +0000 (08:59 +0200)]
Remove references to pgfoundry as recommended hosting platform

pgfoundry is deprectaed and no longer accepting new projects,
so we really shouldn't be directing people there.

12 years agoRemove references to PostgreSQL bundled on Solaris
Magnus Hagander [Wed, 4 Jul 2012 06:58:31 +0000 (08:58 +0200)]
Remove references to PostgreSQL bundled on Solaris

Also remove special references to downloads off pgfoundry since they are
not correct - downloads are done through the main website.

12 years agoImprove documentation about MULE encoding.
Tom Lane [Wed, 4 Jul 2012 04:29:57 +0000 (00:29 -0400)]
Improve documentation about MULE encoding.

This commit improves the comments in pg_wchar.h and creates #define symbols
for some formerly hard-coded values.  No substantive code changes.

Tatsuo Ishii and Tom Lane

12 years agoForgot an #include in the previous patch :-(
Alvaro Herrera [Tue, 3 Jul 2012 20:40:15 +0000 (16:40 -0400)]
Forgot an #include in the previous patch :-(

12 years agoHave REASSIGN OWNED work on extensions, too
Alvaro Herrera [Tue, 3 Jul 2012 19:09:59 +0000 (15:09 -0400)]
Have REASSIGN OWNED work on extensions, too

Per bug #6593, REASSIGN OWNED fails when the affected role has created
an extension.  Even though the user related to the extension is not
nominally the owner, its OID appears on pg_shdepend and thus causes
problems when the user is to be dropped.

This commit adds code to change the "ownership" of the extension itself,
not of the contained objects.  This is fine because it's currently only
called from REASSIGN OWNED, which would also modify the ownership of the
contained objects.  However, this is not sufficient for a working ALTER
OWNER implementation extension.

Back-patch to 9.1, where extensions were introduced.

Bug #6593 reported by Emiliano Leporati.

12 years agoHave copyright tool mention that certain files should be updated in back branches.
Bruce Momjian [Tue, 3 Jul 2012 16:01:38 +0000 (12:01 -0400)]
Have copyright tool mention that certain files should be updated in back branches.

12 years agoRemove misleading hints about reducing the System V request size.
Robert Haas [Tue, 3 Jul 2012 14:07:47 +0000 (10:07 -0400)]
Remove misleading hints about reducing the System V request size.

Since the request size will now be ~48 bytes regardless of how
shared_buffers et. al. are set, much of this advice is no longer
relevant.

12 years agoFix a stupid bug I introduced into XLogFlush().
Robert Haas [Mon, 2 Jul 2012 19:33:59 +0000 (15:33 -0400)]
Fix a stupid bug I introduced into XLogFlush().

Commit f11e8be3e812cdbbc139c1b4e49141378b118dee broke this; it was right
in Peter's original patch, but I messed it up before committing.

12 years agoFix position of WalSndWakeupRequest call.
Robert Haas [Mon, 2 Jul 2012 18:43:10 +0000 (14:43 -0400)]
Fix position of WalSndWakeupRequest call.

This avoids discriminating against wal_sync_method = open_sync or
open_datasync.

Fujii Masao, reviewed by Andres Freund

12 years agoAssorted message style improvements
Peter Eisentraut [Mon, 2 Jul 2012 18:12:46 +0000 (21:12 +0300)]
Assorted message style improvements

12 years agoFix to_date's handling of year 519.
Tom Lane [Mon, 2 Jul 2012 15:35:21 +0000 (11:35 -0400)]
Fix to_date's handling of year 519.

A thinko in commit 029dfdf1157b6d837a7b7211cd35b00c6bcd767c caused the year
519 to be handled differently from either adjacent year, which was not the
intention AFAICS.  Report and diagnosis by Marc Cousin.

In passing, remove redundant re-tests of year value.

12 years agoWork a little harder on comments for walsender wakeup patch.
Robert Haas [Mon, 2 Jul 2012 15:28:53 +0000 (11:28 -0400)]
Work a little harder on comments for walsender wakeup patch.

Per gripe from Tom Lane.

12 years agoMake commit_delay much smarter.
Robert Haas [Mon, 2 Jul 2012 14:26:31 +0000 (10:26 -0400)]
Make commit_delay much smarter.

Instead of letting every backend participating in a group commit wait
independently, have the first one that becomes ready to flush WAL wait
for the configured delay, and let all the others wait just long enough
for that first process to complete its flush.  This greatly increases
the chances of being able to configure a commit_delay setting that
actually improves performance.

As a side consequence of this change, commit_delay now affects all WAL
flushes, rather than just commits.  There was some discussion on
pgsql-hackers about whether to rename the GUC to, say, wal_flush_delay,
but in the absence of consensus I am leaving it alone for now.

Peter Geoghegan, with some changes, mostly to the documentation, by me.

12 years agoMake walsender more responsive.
Robert Haas [Mon, 2 Jul 2012 13:36:34 +0000 (09:36 -0400)]
Make walsender more responsive.

Per testing by Andres Freund, this improves replication performance
and reduces replication latency and latency jitter.  I was a bit
concerned about moving more work into XLogInsert, but testing seems
to show that it's not a problem in practice.

Along the way, improve comments for WaitLatchOrSocket.

Andres Freund.  Review and stylistic cleanup by me.

12 years agoFix race condition in enum value comparisons.
Tom Lane [Sun, 1 Jul 2012 21:12:49 +0000 (17:12 -0400)]
Fix race condition in enum value comparisons.

When (re) loading the typcache comparison cache for an enum type's values,
use an up-to-date MVCC snapshot, not the transaction's existing snapshot.
This avoids problems if we encounter an enum OID that was created since our
transaction started.  Per report from Andres Freund and diagnosis by Robert
Haas.

To ensure this is safe even if enum comparison manages to get invoked
before we've set a transaction snapshot, tweak GetLatestSnapshot to
redirect to GetTransactionSnapshot instead of throwing error when
FirstSnapshotSet is false.  The existing uses of GetLatestSnapshot (in
ri_triggers.c) don't care since they couldn't be invoked except in a
transaction that's already done some work --- but it seems just conceivable
that this might not be true of enums, especially if we ever choose to use
enums in system catalogs.

Note that the comparable coding in enum_endpoint and enum_range_internal
remains GetTransactionSnapshot; this is perhaps debatable, but if we
changed it those functions would have to be marked volatile, which doesn't
seem attractive.

Back-patch to 9.1 where ALTER TYPE ADD VALUE was added.

12 years agoSuppress compiler warnings in readfuncs.c.
Tom Lane [Sun, 1 Jul 2012 02:27:49 +0000 (22:27 -0400)]
Suppress compiler warnings in readfuncs.c.

Commit 7357558fc8866e3a449aa9473c419b593d67b5b6 introduced "(void) token;"
into the READ_TEMP_LOCALS() macro, to suppress complaints from gcc 4.6
when the value of token was not used anywhere in a particular node-read
function.  However, this just moved the warning around: inspection of
buildfarm results shows that some compilers are now complaining that token
is being read before it's set.  Revert the READ_TEMP_LOCALS() macro change
and instead put "(void) token;" into READ_NODE_FIELD(), which is the
principal culprit for cases where the warning might occur.  In principle we
might need the same in READ_BITMAPSET_FIELD() and/or READ_LOCATION_FIELD(),
but it seems unlikely that a node would consist only of such fields, so
I'll leave them alone for now.

12 years agoRemove inappropriate semicolons after function definitions.
Tom Lane [Sat, 30 Jun 2012 21:29:39 +0000 (17:29 -0400)]
Remove inappropriate semicolons after function definitions.

Solaris Studio warns about this, and some compilers might think it's an
outright syntax error.

12 years agoDeclare AnonymousShmem pointer as "void *".
Tom Lane [Sat, 30 Jun 2012 21:19:46 +0000 (17:19 -0400)]
Declare AnonymousShmem pointer as "void *".

The original coding had it as "PGShmemHeader *", but that doesn't offer any
notational benefit because we don't dereference it.  And it was resulting
in compiler warnings on some platforms, notably buildfarm member
castoroides, where mmap() and munmap() are evidently declared to take and
return "char *".