]> granicus.if.org Git - postgresql/log
postgresql
14 years agoFix plpgsql's handling of "simple" expression evaluation.
Tom Lane [Thu, 28 Oct 2010 17:00:54 +0000 (13:00 -0400)]
Fix plpgsql's handling of "simple" expression evaluation.

In general, expression execution state trees aren't re-entrantly usable,
since functions can store private state information in them.
For efficiency reasons, plpgsql tries to cache and reuse state trees for
"simple" expressions.  It can get away with that most of the time, but it
can fail if the state tree is dirty from a previous failed execution (as
in an example from Alvaro) or is being used recursively (as noted by me).

Fix by tracking whether a state tree is in use, and falling back to the
"non-simple" code path if so.  This results in a pretty considerable speed
hit when the non-simple path is taken, but the available alternatives seem
even more unpleasant because they add overhead in the simple path.  Per
idea from Heikki.

Back-patch to all supported branches.

14 years agoRelease notes for 9.1alpha2
Peter Eisentraut [Thu, 28 Oct 2010 14:42:58 +0000 (17:42 +0300)]
Release notes for 9.1alpha2

14 years agoRemove obsolete release-alpha.sgml
Peter Eisentraut [Thu, 28 Oct 2010 14:40:56 +0000 (17:40 +0300)]
Remove obsolete release-alpha.sgml

This was only used while the final release notes for 9.0 were being prepared.
The alpha release notes are now in release-9.1.sgml.

14 years agoRemove tabs from SGML
Peter Eisentraut [Thu, 28 Oct 2010 14:40:27 +0000 (17:40 +0300)]
Remove tabs from SGML

14 years agoRevert "Correct WAL space calculation formula in docs."
Robert Haas [Thu, 28 Oct 2010 01:24:02 +0000 (21:24 -0400)]
Revert "Correct WAL space calculation formula in docs."

This reverts commit 915116bc62db2aaec7001bde6610128f4cbd29f9.

Per discussion, the previous formula was in fact correct.

http://archives.postgresql.org/pgsql-docs/2010-10/msg00038.php

14 years agoReorganize OS-specific details about write caching into a list.
Robert Haas [Thu, 28 Oct 2010 01:19:11 +0000 (21:19 -0400)]
Reorganize OS-specific details about write caching into a list.

Along the way, clarify that sdparm can be used on Linux as well as FreeBSD.

14 years agoPrevious patch had no detectable virtue other than being a one-liner.
Tom Lane [Wed, 27 Oct 2010 19:26:24 +0000 (15:26 -0400)]
Previous patch had no detectable virtue other than being a one-liner.

Try to make the code look self-consistent again, so it doesn't confuse
future developers.

14 years agoFix long-standing segfault when accept() or one of the calls made right
Heikki Linnakangas [Wed, 27 Oct 2010 17:03:00 +0000 (20:03 +0300)]
Fix long-standing segfault when accept() or one of the calls made right
after accepting a connection fails, and the server is compiled with GSSAPI
support. Report and patch by Alexander V. Chernikov, bug #5731.

14 years agoFix up some oversights in psql's Unicode-escape support.
Tom Lane [Wed, 27 Oct 2010 02:23:04 +0000 (22:23 -0400)]
Fix up some oversights in psql's Unicode-escape support.

Original patch failed to include new exclusive states in a switch that
needed to include them; and also was guilty of very fuzzy thinking
about how to handle error cases.  Per bug #5729 from Alan Choi.

14 years agoNote that effective_io_concurrency only affects bitmap heap scans.
Robert Haas [Wed, 27 Oct 2010 01:44:14 +0000 (21:44 -0400)]
Note that effective_io_concurrency only affects bitmap heap scans.

Josh Kupershmidt

14 years agoAdd a client authentication hook.
Robert Haas [Wed, 27 Oct 2010 01:20:02 +0000 (21:20 -0400)]
Add a client authentication hook.

KaiGai Kohei, with minor cleanup of the comments by me.

14 years agoMinor fixups for psql's process_file() function.
Robert Haas [Tue, 26 Oct 2010 23:28:18 +0000 (19:28 -0400)]
Minor fixups for psql's process_file() function.

- Avoid closing stdin, since we didn't open it.  Previously multiple
inclusions of stdin would be terminated with a single quit, now a separate
quit is needed for each invocation. Previous behavior also accessed stdin
after it was fclose()d, which is undefined behavior per ANSI C.

- Properly restore pset.inputfile, since the caller expects to be able
to free that memory.

Marti Raudsepp

14 years agoNote explicitly that hash indexes are also not replicated because they're not
Heikki Linnakangas [Tue, 26 Oct 2010 19:50:31 +0000 (22:50 +0300)]
Note explicitly that hash indexes are also not replicated because they're not
WAL-logged. Make the notice about the lack of WAL-logging more visible by
making it a <caution>. Also remove the false statement from hot standby
caveats section that hash indexes are not used during hot standby.

14 years agoFix dumb typo in SECURITY LABEL error message.
Robert Haas [Tue, 26 Oct 2010 18:54:31 +0000 (14:54 -0400)]
Fix dumb typo in SECURITY LABEL error message.

Report by Peter Eisentraut.

14 years agoBefore removing backup_label and irrevocably changing pg_control file, check
Heikki Linnakangas [Tue, 26 Oct 2010 18:15:42 +0000 (21:15 +0300)]
Before removing backup_label and irrevocably changing pg_control file, check
that WAL file containing the checkpoint redo-location can be found. This
avoids making the cluster irrecoverable if the redo location is in an earlie
WAL file than the checkpoint record.

Report, analysis and patch by Jeff Davis, with small changes by me.

14 years agoAdd missing newlines at end of files
Peter Eisentraut [Tue, 26 Oct 2010 17:11:43 +0000 (20:11 +0300)]
Add missing newlines at end of files

14 years agoFix typos "are are".
Itagaki Takahiro [Tue, 26 Oct 2010 08:15:17 +0000 (17:15 +0900)]
Fix typos "are are".

14 years agoAdd .gitignore for contrib/uuid-ossp.
Itagaki Takahiro [Tue, 26 Oct 2010 03:25:34 +0000 (12:25 +0900)]
Add .gitignore for contrib/uuid-ossp.

14 years agoRefactor typenameTypeId()
Peter Eisentraut [Mon, 25 Oct 2010 18:40:46 +0000 (21:40 +0300)]
Refactor typenameTypeId()

Split the old typenameTypeId() into two functions: A new typenameTypeId() that
returns only a type OID, and typenameTypeIdAndMod() that returns type OID and
typmod.  This isolates call sites better that actually care about the typmod.

14 years agoFix overly-enthusiastic Assert in printing of Param reference expressions.
Tom Lane [Mon, 25 Oct 2010 18:25:10 +0000 (14:25 -0400)]
Fix overly-enthusiastic Assert in printing of Param reference expressions.

A NestLoopParam's value can only be a Var or Aggref, but this isn't the
case in general for SubPlan parameters, so print_parameter_expr had better
be prepared to cope.  Brain fade in my recent patch to print the referenced
expression instead of just printing $N for PARAM_EXEC Params.  Per report
from Pavel Stehule.

14 years agoFix inline_set_returning_function() to preserve the invalItems list properly.
Tom Lane [Mon, 25 Oct 2010 17:04:37 +0000 (13:04 -0400)]
Fix inline_set_returning_function() to preserve the invalItems list properly.

This avoids a possible crash when inlining a SRF whose argument list
contains a reference to an inline-able user function.  The crash is quite
reproducible with CLOBBER_FREED_MEMORY enabled, but would be less certain
in a production build.  Problem introduced in 9.0 by the named-arguments
patch, which requires invoking eval_const_expressions() before we can try
to inline a SRF.  Per report from Brendan Jurd.

14 years agofind -path is not portable, so use grep -v instead.
Alvaro Herrera [Mon, 25 Oct 2010 12:24:02 +0000 (09:24 -0300)]
find -path is not portable, so use grep -v instead.

Per previous failure of buildfarm member koi (which is no longer
failing, alas).

14 years agoWork around rounding misbehavior exposed by buildfarm.
Tom Lane [Mon, 25 Oct 2010 05:13:22 +0000 (01:13 -0400)]
Work around rounding misbehavior exposed by buildfarm.

14 years agoRemove unnecessary use of trigger flag to hash plperl functions
Andrew Dunstan [Mon, 25 Oct 2010 03:53:44 +0000 (23:53 -0400)]
Remove unnecessary use of trigger flag to hash plperl functions

14 years agoAllow new values to be added to an existing enum type.
Tom Lane [Mon, 25 Oct 2010 03:04:37 +0000 (23:04 -0400)]
Allow new values to be added to an existing enum type.

After much expenditure of effort, we've got this to the point where the
performance penalty is pretty minimal in typical cases.

Andrew Dunstan, reviewed by Brendan Jurd, Dean Rasheed, and Tom Lane

14 years agoSupport suffix matching of host names in pg_hba.conf
Peter Eisentraut [Sun, 24 Oct 2010 12:54:00 +0000 (15:54 +0300)]
Support suffix matching of host names in pg_hba.conf

A name starting with a dot can be used to match a suffix of the actual
host name (e.g., .example.com matches foo.example.com).

14 years agoCorrect a mistake in levenshtein_less_equal() multibyte character handling.
Robert Haas [Fri, 22 Oct 2010 19:27:39 +0000 (15:27 -0400)]
Correct a mistake in levenshtein_less_equal() multibyte character handling.

Spotted by Alexander Korotkov.

Along the way, remove a misleading comment line.

14 years agoAdd semicolon, missed in previous patch. And update the keyword list in
Heikki Linnakangas [Fri, 22 Oct 2010 15:38:31 +0000 (18:38 +0300)]
Add semicolon, missed in previous patch. And update the keyword list in
the docs to reflect that OFF is now unreserved. Spotted by Tom Lane.

14 years agoMake OFF keyword unreserved. It's not hard to imagine wanting to use 'off'
Heikki Linnakangas [Fri, 22 Oct 2010 14:37:38 +0000 (17:37 +0300)]
Make OFF keyword unreserved. It's not hard to imagine wanting to use 'off'
as a variable or column name, and it's not reserved in recent versions of
the SQL spec either. This became particularly annoying in 9.0, before that
PL/pgSQL replaced variable names in queries with parameter markers, so
it was possible to use OFF and many other backend parser keywords as
variable names. Because of that, backpatch to 9.0.

14 years agoMention limited usefulness of .pgpass database field.
Bruce Momjian [Thu, 21 Oct 2010 21:46:42 +0000 (21:46 +0000)]
Mention limited usefulness of .pgpass database field.

14 years agoImprove handling of domains over arrays.
Tom Lane [Thu, 21 Oct 2010 20:07:17 +0000 (16:07 -0400)]
Improve handling of domains over arrays.

This patch eliminates various bizarre behaviors caused by sloppy thinking
about the difference between a domain type and its underlying array type.
In particular, the operation of updating one element of such an array
has to be considered as yielding a value of the underlying array type,
*not* a value of the domain, because there's no assurance that the
domain's CHECK constraints are still satisfied.  If we're intending to
store the result back into a domain column, we have to re-cast to the
domain type so that constraints are re-checked.

For similar reasons, such a domain can't be blindly matched to an ANYARRAY
polymorphic parameter, because the polymorphic function is likely to apply
array-ish operations that could invalidate the domain constraints.  For the
moment, we just forbid such matching.  We might later wish to insert an
automatic downcast to the underlying array type, but such a change should
also change matching of domains to ANYELEMENT for consistency.

To ensure that all such logic is rechecked, this patch removes the original
hack of setting a domain's pg_type.typelem field to match its base type;
the typelem will always be zero instead.  In those places where it's really
okay to look through the domain type with no other logic changes, use the
newly added get_base_element_type function in place of get_element_type.
catversion bumped due to change in pg_type contents.

Per bug #5717 from Richard Huxton and subsequent discussion.

14 years agoRemove obsolete comment, per Josh Kupershmidt.
Tom Lane [Wed, 20 Oct 2010 21:05:15 +0000 (17:05 -0400)]
Remove obsolete comment, per Josh Kupershmidt.

14 years agoClean up pg_upgrade cache lookup code; remove useless NULL pointer tests.
Bruce Momjian [Wed, 20 Oct 2010 20:02:00 +0000 (20:02 +0000)]
Clean up pg_upgrade cache lookup code;  remove useless NULL pointer tests.

14 years agoIf pk is NULL, the backend would segfault when accessing ->algo and the
Heikki Linnakangas [Wed, 20 Oct 2010 19:20:33 +0000 (22:20 +0300)]
If pk is NULL, the backend would segfault when accessing ->algo and the
following NULL check was never reached.

This problem was found by Coccinelle (null_ref.cocci from coccicheck).

Marti Raudsepp

14 years agoDon't try to fetch database name when SetTransactionIdLimit() is executed
Tom Lane [Wed, 20 Oct 2010 16:48:51 +0000 (12:48 -0400)]
Don't try to fetch database name when SetTransactionIdLimit() is executed
outside a transaction.

This repairs brain fade in my patch of 2009-08-30: the reason we had been
storing oldest-database name, not OID, in ShmemVariableCache was of course
to avoid having to do a catalog lookup at times when it might be unsafe.

This error explains why Aleksandr Dushein is having trouble getting out of
an XID wraparound state in bug #5718, though not how he got into that state
in the first place.  I suspect pg_upgrade is at fault there.

14 years agoRemove AtStart_Cache() call in CommandCounterIncrement().
Alvaro Herrera [Mon, 18 Oct 2010 02:08:30 +0000 (23:08 -0300)]
Remove AtStart_Cache() call in CommandCounterIncrement().

This call was present in the aboriginal code from Berkeley, and has
never been touched; it may very well be that it was there to mask
effects of bugs in other places and it may no longer be necessary.
The removal has been foreseen in a code comment since 2007; this seems
to be a good time to test this hypothesis.

14 years agoFix ecpg test building process to not generate *.dSYM junk on Macs.
Tom Lane [Wed, 20 Oct 2010 04:54:58 +0000 (00:54 -0400)]
Fix ecpg test building process to not generate *.dSYM junk on Macs.

The trick is to not try to build executables directly from .c files,
but to always build the intermediate .o files.  For obscure reasons,
Darwin's version of gcc will leave debug cruft behind in the first
case but not the second.  Per complaint from Robert Haas.

14 years agoAdd some caveats to the contrib/isn docs.
Robert Haas [Wed, 20 Oct 2010 02:46:55 +0000 (22:46 -0400)]
Add some caveats to the contrib/isn docs.

14 years agoUpdates to contrib/isn ISBN tables.
Robert Haas [Wed, 20 Oct 2010 02:24:33 +0000 (22:24 -0400)]
Updates to contrib/isn ISBN tables.

Jan Otto, reviewed by Peter Geoghegan

14 years agoRename pg_upgrade 'log' to 'log_opts', to avoid platform naming conflict.
Bruce Momjian [Wed, 20 Oct 2010 02:31:17 +0000 (02:31 +0000)]
Rename pg_upgrade 'log' to 'log_opts', to avoid platform naming conflict.

14 years agoFurther speed up pg_upgrade lookups.
Bruce Momjian [Wed, 20 Oct 2010 02:05:43 +0000 (02:05 +0000)]
Further speed up pg_upgrade lookups.

14 years agoUpdate storage.sgml to describe the 9.0 tablespace directory layout.
Tom Lane [Wed, 20 Oct 2010 01:52:59 +0000 (21:52 -0400)]
Update storage.sgml to describe the 9.0 tablespace directory layout.

14 years agoFix for new pg_upgrade cache code.
Bruce Momjian [Wed, 20 Oct 2010 01:24:31 +0000 (01:24 +0000)]
Fix for new pg_upgrade cache code.

14 years agoCache most recent relfilenode lookups, for speed, after report of
Bruce Momjian [Wed, 20 Oct 2010 00:55:16 +0000 (00:55 +0000)]
Cache most recent relfilenode lookups, for speed, after report of
pg_upgrade slowness for 150k tables.

14 years agoIn pg_upgrade, use cached copy of directory listing, rather than calling
Bruce Momjian [Wed, 20 Oct 2010 00:33:19 +0000 (00:33 +0000)]
In pg_upgrade, use cached copy of directory listing, rather than calling
scandir() with a pattern for every table.

Optimization after report of pg_upgrade slowness with 150k tables.

14 years agoPgindent run on pg_upgrade source after restructuring.
Bruce Momjian [Tue, 19 Oct 2010 22:37:04 +0000 (22:37 +0000)]
Pgindent run on pg_upgrade source after restructuring.

14 years agoRestructure the pg_upgrade code to use several global structures rather
Bruce Momjian [Tue, 19 Oct 2010 21:38:16 +0000 (21:38 +0000)]
Restructure the pg_upgrade code to use several global structures rather
than packing everything into 'ctx' and passing that to every function.

14 years agoFix incorrect generation of whole-row variables in planner.
Tom Lane [Tue, 19 Oct 2010 19:08:37 +0000 (15:08 -0400)]
Fix incorrect generation of whole-row variables in planner.

A couple of places in the planner need to generate whole-row Vars, and were
cutting corners by setting vartype = RECORDOID in the Vars, even in cases
where there's an identifiable named composite type for the RTE being
referenced.  While we mostly got away with this, it failed when there was
also a parser-generated whole-row reference to the same RTE, because the
two Vars weren't equal() due to the difference in vartype.  Fix by
providing a subroutine the planner can call to generate whole-row Vars
the same way the parser does.

Per bug #5716 from Andrew Tipton.  Back-patch to 9.0 where one of the bogus
calls was introduced (the other one is new in HEAD).

14 years agoIn pg_upgrade, rename SHELL_EXT to SCRIPT_EXT, for clarity.
Bruce Momjian [Tue, 19 Oct 2010 15:57:55 +0000 (15:57 +0000)]
In pg_upgrade, rename SHELL_EXT to SCRIPT_EXT, for clarity.

14 years agoAdd removal of PG_VERSION to optional old cluster deletion script.
Bruce Momjian [Tue, 19 Oct 2010 15:52:43 +0000 (15:52 +0000)]
Add removal of PG_VERSION to optional old cluster deletion script.
Backpatch to 9.0.X.

14 years agoAdd mention of using tools/fsync to test fsync methods. Restructure
Bruce Momjian [Tue, 19 Oct 2010 14:56:33 +0000 (14:56 +0000)]
Add mention of using tools/fsync to test fsync methods. Restructure
recent wal_sync_method doc paragraph to be clearer.

14 years agoAdd levenshtein_less_equal, optimized version for small distances.
Robert Haas [Tue, 19 Oct 2010 13:51:06 +0000 (09:51 -0400)]
Add levenshtein_less_equal, optimized version for small distances.

Alexander Korotkov, heavily revised by me.

14 years agoUnbreak comments on composite type attributes.
Robert Haas [Tue, 19 Oct 2010 11:21:58 +0000 (07:21 -0400)]
Unbreak comments on composite type attributes.

Report and diagnosis by Peter Eisentraut.

14 years agoIn pg_upgrade, rename macro EXEC_EXT to SHELL_EXT for clarity.
Bruce Momjian [Tue, 19 Oct 2010 02:55:54 +0000 (02:55 +0000)]
In pg_upgrade, rename macro EXEC_EXT to SHELL_EXT for clarity.

Backpatch to 9.0.X.

14 years agoSupport key word 'all' in host column of pg_hba.conf
Peter Eisentraut [Mon, 18 Oct 2010 19:14:47 +0000 (22:14 +0300)]
Support key word 'all' in host column of pg_hba.conf

14 years agoDocument the tablespace directory "should" be empty, rather than "must"
Bruce Momjian [Mon, 18 Oct 2010 18:15:58 +0000 (18:15 +0000)]
Document the tablespace directory "should" be empty, rather than "must"
be empty.  Because of binary migration usage, it might not be empty.

14 years agoChange example pg_hba.conf in docs to match altered pg_hba.conf.sample
Robert Haas [Mon, 18 Oct 2010 16:17:06 +0000 (12:17 -0400)]
Change example pg_hba.conf in docs to match altered pg_hba.conf.sample

Peter Eisentraut's recent patch to allow host names in pg_hba.conf
changed the contents of pg_hba.conf.sample

Fujii Masao

14 years agoFix a passel of inappropriately-named global functions in GIN.
Tom Lane [Mon, 18 Oct 2010 01:43:26 +0000 (21:43 -0400)]
Fix a passel of inappropriately-named global functions in GIN.

The GIN code has absolutely no business exporting GIN-specific functions
with names as generic as compareItemPointers() or newScanKey(); that's
just trouble waiting to happen.  I got annoyed about this again just now
and decided to fix it.  This commit ensures that all global symbols
defined in access/gin/ have names including "gin" or "Gin".  There were a
couple of cases, like names involving "PostingItem", where arguably the
names were already sufficiently nongeneric; but I figured as long as I was
risking creating merge problems for unapplied GIN patches I might as well
impose a uniform policy.

I didn't touch any static symbol names.  There might be some places
where it'd be appropriate to rename some static functions to match
siblings that are exported, but I'll leave that for another time.

14 years agoImprove GIN indexscan cost estimation.
Tom Lane [Mon, 18 Oct 2010 00:52:32 +0000 (20:52 -0400)]
Improve GIN indexscan cost estimation.

The better estimate requires more statistics than we previously stored:
in particular, counts of "entry" versus "data" pages within the index,
as well as knowledge of the number of distinct key values.  We collect
this information during initial index build and update it during VACUUM,
storing the info in new fields on the index metapage.  No initdb is
required because these fields will read as zeroes in a pre-existing
index, and the new gincostestimate code is coded to behave (reasonably)
sanely if they are zeroes.

Teodor Sigaev, reviewed by Jan Urbanski, Tom Lane, and Itagaki Takahiro.

14 years agoFix msvc build for localized versions of Visual C++
Magnus Hagander [Sun, 17 Oct 2010 14:36:54 +0000 (16:36 +0200)]
Fix msvc build for localized versions of Visual C++

Look only at the non-localized part of the output from "vcbuild /?",
which is used to determine the version of Visual Studio in use. Different
languages seem to localize different amounts of the string, but we assume
the part "Microsoft Visual C++" won't be modified.

14 years agoFix recent changes to not break non-IPV6-aware systems.
Tom Lane [Sat, 16 Oct 2010 14:12:16 +0000 (10:12 -0400)]
Fix recent changes to not break non-IPV6-aware systems.

14 years agoDocument that translate() removes characters in "from" that don't have
Alvaro Herrera [Sat, 16 Oct 2010 04:11:38 +0000 (01:11 -0300)]
Document that translate() removes characters in "from" that don't have
a corresponding "to" character.

Author: Josh Kupershmidt

14 years agoAllow WITH clauses to be attached to INSERT, UPDATE, DELETE statements.
Tom Lane [Fri, 15 Oct 2010 23:53:59 +0000 (19:53 -0400)]
Allow WITH clauses to be attached to INSERT, UPDATE, DELETE statements.

This is not the hoped-for facility of using INSERT/UPDATE/DELETE inside
a WITH, but rather the other way around.  It seems useful in its own
right anyway.

Note: catversion bumped because, although the contents of stored rules
might look compatible, there's actually a subtle semantic change.
A single Query containing a WITH and INSERT...VALUES now represents
writing the WITH before the INSERT, not before the VALUES.  While it's
not clear that that matters to anyone, it seems like a good idea to
have it cited in the git history for catversion.h.

Original patch by Marko Tiikkaja, with updating and cleanup by
Hitoshi Harada.

14 years agoSupport host names in pg_hba.conf
Peter Eisentraut [Fri, 15 Oct 2010 19:53:39 +0000 (22:53 +0300)]
Support host names in pg_hba.conf

Peter Eisentraut, reviewed by KaiGai Kohei and Tom Lane

14 years agoChange references to SQL/XML:2003 to :2008 and renumber sections accordingly
Peter Eisentraut [Fri, 15 Oct 2010 18:18:47 +0000 (21:18 +0300)]
Change references to SQL/XML:2003 to :2008 and renumber sections accordingly

14 years agoDocument the DISTINCT noise word in the UNION/INTERSECT/EXCEPT constructs.
Tom Lane [Fri, 15 Oct 2010 19:48:45 +0000 (15:48 -0400)]
Document the DISTINCT noise word in the UNION/INTERSECT/EXCEPT constructs.

I also rearranged the order of the sections to match the logical order
of processing steps: the distinct-elimination implied by SELECT DISTINCT
happens before, not after, any UNION/INTERSECT/EXCEPT combination.

Per a suggestion from Hitoshi Harada.

14 years agoAllow pg_ctl to register the service in either AUTO or DEMAND start type
Alvaro Herrera [Fri, 15 Oct 2010 17:30:03 +0000 (14:30 -0300)]
Allow pg_ctl to register the service in either AUTO or DEMAND start type

Author: Quan Zongliang
Documentation updates by David Fetter

14 years agoFix low-risk potential denial of service against RADIUS login.
Magnus Hagander [Fri, 15 Oct 2010 14:59:10 +0000 (16:59 +0200)]
Fix low-risk potential denial of service against RADIUS login.

Corrupt RADIUS responses were treated as errors and not ignored
(which the RFC2865 states they should be). This meant that a
user with unfiltered access to the network of the PostgreSQL
or RADIUS server could send a spoofed RADIUS response
to the PostgreSQL server causing it to reject a valid login,
provided the attacker could also guess (or brute-force) the
correct port number.

Fix is to simply retry the receive in a loop until the timeout
has expired or a valid (signed by the correct RADIUS server)
packet arrives.

Reported by Alan DeKok in bug #5687.

14 years agoCorrect WAL space calculation formula in docs.
Simon Riggs [Fri, 15 Oct 2010 09:17:12 +0000 (10:17 +0100)]
Correct WAL space calculation formula in docs.

Error pointed out by Fujii Masao, though not his patch.

14 years agoImprove comment about ignoring 128 error code on Windows:
Bruce Momjian [Fri, 15 Oct 2010 01:57:41 +0000 (01:57 +0000)]
Improve comment about ignoring 128 error code on Windows:

     * Microsoft reports it is related to mutex failure:
     *   http://archives.postgresql.org/pgsql-hackers/2010-09/msg00790.php

14 years agoAdd pg_user_mappings to the table of system views.
Robert Haas [Thu, 14 Oct 2010 23:12:24 +0000 (19:12 -0400)]
Add pg_user_mappings to the table of system views.

14 years agoSupport MergeAppend plans, to allow sorted output from append relations.
Tom Lane [Thu, 14 Oct 2010 20:56:39 +0000 (16:56 -0400)]
Support MergeAppend plans, to allow sorted output from append relations.

This patch eliminates the former need to sort the output of an Append scan
when an ordered scan of an inheritance tree is wanted.  This should be
particularly useful for fast-start cases such as queries with LIMIT.

Original patch by Greg Stark, with further hacking by Hans-Jurgen Schonig,
Robert Haas, and Tom Lane.

14 years agoFix makefile logic to not break the build when xgettext is missing
Peter Eisentraut [Thu, 14 Oct 2010 19:15:46 +0000 (22:15 +0300)]
Fix makefile logic to not break the build when xgettext is missing

xgettext is only required when make init-po is run manually; it is not
required for a build.  The intent to handle that was already there, but
the ifdef's were in the wrong place.

14 years agoRemove reference.ced
Peter Eisentraut [Thu, 14 Oct 2010 18:32:45 +0000 (21:32 +0300)]
Remove reference.ced

This is a parsed DocBook DTD for the PSGML Emacs mode, but it hasn't
been updated since we switched to DocBook 4.2 about seven years ago.
Also, PSGML has deprecated this method of DTD parsing.

14 years agoComplete the documentation of the USAGE privilege for foreign servers
Peter Eisentraut [Thu, 14 Oct 2010 17:36:42 +0000 (20:36 +0300)]
Complete the documentation of the USAGE privilege for foreign servers

The GRANT reference page failed to mention that the USAGE privilege
allows modifying associated user mappings, although this was already
documented on the CREATE/ALTER/DROP USER MAPPING pages.

14 years agoDocument (compositeval).* field selection syntax
Peter Eisentraut [Thu, 14 Oct 2010 17:07:13 +0000 (20:07 +0300)]
Document (compositeval).* field selection syntax

14 years agoImprovements to docs about pg_archive_cleanup and use of archives
Simon Riggs [Thu, 14 Oct 2010 18:30:15 +0000 (19:30 +0100)]
Improvements to docs about pg_archive_cleanup and use of archives

Brendan Jurd

14 years agoMake startup process respond to signals to cancel waiting on latch.
Simon Riggs [Thu, 14 Oct 2010 18:15:26 +0000 (19:15 +0100)]
Make startup process respond to signals to cancel waiting on latch.
A tidy up for recently committed changes to startup latch.

Fujii Masao

14 years agoFix bug in comment of timeline history file.
Simon Riggs [Thu, 14 Oct 2010 18:06:06 +0000 (19:06 +0100)]
Fix bug in comment of timeline history file.

Fujii Masao

14 years agoApplied patch by Itagaki Takahiro to fix incorrect status calculation in
Michael Meskes [Thu, 14 Oct 2010 15:55:07 +0000 (17:55 +0200)]
Applied patch by Itagaki Takahiro to fix incorrect status calculation in
ecpglib. Instead of parsing the statement just as ask the database server. This
patch removes the whole client side track keeping of the current transaction
status.

14 years agoRemove executable permission from files where it doesn't belong
Peter Eisentraut [Wed, 13 Oct 2010 19:30:25 +0000 (22:30 +0300)]
Remove executable permission from files where it doesn't belong

14 years agoAdd index entries for pg_stat* views
Peter Eisentraut [Wed, 13 Oct 2010 18:29:10 +0000 (21:29 +0300)]
Add index entries for pg_stat* views

14 years agoMention the default pg_ctl wait time in the -t option documentation
Peter Eisentraut [Wed, 13 Oct 2010 17:24:51 +0000 (20:24 +0300)]
Mention the default pg_ctl wait time in the -t option documentation

14 years agoMake title capitalization consistent with surroundings
Peter Eisentraut [Wed, 13 Oct 2010 17:05:16 +0000 (20:05 +0300)]
Make title capitalization consistent with surroundings

14 years agoPut per-letter quicklinks at the top of the HTML bookindex page
Peter Eisentraut [Wed, 13 Oct 2010 16:46:05 +0000 (19:46 +0300)]
Put per-letter quicklinks at the top of the HTML bookindex page

14 years agoAccept 'public' as a pseudo-role name in has_table_privilege() and friends
Itagaki Takahiro [Wed, 13 Oct 2010 05:37:23 +0000 (14:37 +0900)]
Accept 'public' as a pseudo-role name in has_table_privilege() and friends
to see if a particular privilege has been granted to PUBLIC.

The issue was reported by Jim Nasby.
Patch by Alvaro Herrera, and reviewed by KaiGai Kohei.

14 years agoRemove some unnecessary tests of pgstat_track_counts.
Tom Lane [Tue, 12 Oct 2010 18:44:25 +0000 (14:44 -0400)]
Remove some unnecessary tests of pgstat_track_counts.

We may as well make pgstat_count_heap_scan() and related macros just count
whenever rel->pgstat_info isn't null.  Testing pgstat_track_counts buys
nothing at all in the normal case where that flag is ON; and when it's OFF,
the pgstat_info link will be null, so it's still a useless test.

This change is unlikely to buy any noticeable performance improvement,
but a cycle shaved is a cycle earned; and my investigations earlier today
convinced me that we're down to the point where individual instructions in
the inner execution loops are starting to matter.

14 years agoFix typo
Alvaro Herrera [Tue, 12 Oct 2010 16:24:10 +0000 (13:24 -0300)]
Fix typo

14 years agoFix plpython so that it again honors typmod while assigning to tuple fields.
Tom Lane [Tue, 12 Oct 2010 02:16:40 +0000 (22:16 -0400)]
Fix plpython so that it again honors typmod while assigning to tuple fields.

This was broken in 9.0 while improving plpython's conversion behavior for
bytea and boolean.  Per bug report from maizi.

14 years agoFix assorted bugs in GIN's WAL replay logic.
Tom Lane [Mon, 11 Oct 2010 23:04:37 +0000 (19:04 -0400)]
Fix assorted bugs in GIN's WAL replay logic.

The original coding was quite sloppy about handling the case where
XLogReadBuffer fails (because the page has since been deleted).  This
would result in either "bad buffer id: 0" or an Assert failure during
replay, if indeed the page were no longer there.  In a couple of places
it also neglected to check whether the change had already been applied,
which would probably result in corrupted index contents.  I believe that
bug #5703 is an instance of the first problem.  These issues could show up
without replication, but only if you were unfortunate enough to crash
between modification of a GIN index and the next checkpoint.

Back-patch to 8.2, which is as far back as GIN has WAL support.

14 years agoImprove the planner's simplification of NOT constructs.
Tom Lane [Mon, 11 Oct 2010 03:19:50 +0000 (23:19 -0400)]
Improve the planner's simplification of NOT constructs.

This patch merges the responsibility for NOT-flattening into
eval_const_expressions' processing.  It wasn't done that way originally
because prepqual.c is far older than eval_const_expressions.  But putting
this work into eval_const_expressions saves one pass over the qual trees,
and in fact saves even more than that because we can exploit the knowledge
that the subexpressions have already been recursively simplified.  Doing it
this way also lets us do it uniformly over all expressions, whereas
prepqual.c formerly just did it at top level to save cycles.  That should
improve the planner's ability to recognize logically-equivalent constructs.

While at it, also add the ability to fold a NOT into BooleanTest and
NullTest constructs (the latter only for the scalar-datatype case).

Per discussion of bug #5702.

14 years agoTeach psql to do tab completion for names of psql variables.
Tom Lane [Sun, 10 Oct 2010 22:42:35 +0000 (18:42 -0400)]
Teach psql to do tab completion for names of psql variables.

Completion is supported in the context of \set and when interpolating
a variable value using :foo etc.

In passing, fix some places in tab-complete.c that weren't following
project style for comment formatting.

Pavel Stehule, reviewed by Itagaki Takahiro

14 years agoSupport triggers on views.
Tom Lane [Sun, 10 Oct 2010 17:43:33 +0000 (13:43 -0400)]
Support triggers on views.

This patch adds the SQL-standard concept of an INSTEAD OF trigger, which
is fired instead of performing a physical insert/update/delete.  The
trigger function is passed the entire old and/or new rows of the view,
and must figure out what to do to the underlying tables to implement
the update.  So this feature can be used to implement updatable views
using trigger programming style rather than rule hacking.

In passing, this patch corrects the names of some columns in the
information_schema.triggers view.  It seems the SQL committee renamed
them somewhere between SQL:99 and SQL:2003.

Dean Rasheed, reviewed by Bernd Helmle; some additional hacking by me.

14 years agoExtensive ECPG documentation improvements
Peter Eisentraut [Sun, 10 Oct 2010 10:46:02 +0000 (13:46 +0300)]
Extensive ECPG documentation improvements

Satoshi Nagayasu, reviewed and revised by Peter Eisentraut

Since this introduces new refentries that we probably don't want to publish as
man pages, tweak man page stylesheet to omit man pages without manvolnum
element.

Peter Eisentraut

14 years agoAdjust EXPLAIN documentation, so that it's not unreasonably wide.
Robert Haas [Sat, 9 Oct 2010 02:59:48 +0000 (22:59 -0400)]
Adjust EXPLAIN documentation, so that it's not unreasonably wide.

The new formatting matches what we do for COPY.

Per a complaint from Bruce Momjian.

14 years agoSingle-word clarification in postgresql.conf log_truncate_on_rotation
Bruce Momjian [Fri, 8 Oct 2010 19:10:06 +0000 (19:10 +0000)]
Single-word clarification in postgresql.conf log_truncate_on_rotation
comment.

14 years agoFix sloppy usage of TRIGGER_FIRED_BEFORE/TRIGGER_FIRED_AFTER.
Tom Lane [Fri, 8 Oct 2010 17:27:31 +0000 (13:27 -0400)]
Fix sloppy usage of TRIGGER_FIRED_BEFORE/TRIGGER_FIRED_AFTER.

Various places were testing TRIGGER_FIRED_BEFORE() where what they really
meant was !TRIGGER_FIRED_AFTER(), or vice versa.  This needs to be cleaned
up because there are about to be more than two possible states.

We might want to note this in the 9.1 release notes as something for
trigger authors to double-check.

For consistency's sake I also changed some places that assumed that
TRIGGER_FIRED_FOR_ROW and TRIGGER_FIRED_FOR_STATEMENT are necessarily
mutually exclusive; that's not in immediate danger of breaking, but
it's still sloppier than it should be.

Extracted from Dean Rasheed's patch for triggers on views.  I'm committing
this separately since it's an identifiable separate issue, and is the
only reason for the patch to touch most of these particular files.

14 years agoWarn that views can be safely used to hide columns, but not rows.
Robert Haas [Fri, 8 Oct 2010 13:15:17 +0000 (09:15 -0400)]
Warn that views can be safely used to hide columns, but not rows.

14 years agoImprove logging in VACUUM FULL VERBOSE and CLUSTER VERBOSE.
Tom Lane [Fri, 8 Oct 2010 01:46:46 +0000 (21:46 -0400)]
Improve logging in VACUUM FULL VERBOSE and CLUSTER VERBOSE.

This patch resurrects some of the information that could be logged by the
old, now-dead implementation of VACUUM FULL, in particular counts of live
and dead tuples and the time taken for the table rebuild proper.  There's
still no logging about the ensuing index rebuilds, though.

Itagaki Takahiro

14 years agoEliminate some repetitive coding in tuplesort.c.
Tom Lane [Fri, 8 Oct 2010 00:32:21 +0000 (20:32 -0400)]
Eliminate some repetitive coding in tuplesort.c.

Use a macro LogicalTapeReadExact() to encapsulate the error check when
we want to read an exact number of bytes from a "tape".  Per a suggestion
of Takahiro Itagaki.