]> granicus.if.org Git - postgresql/log
postgresql
10 years agodoc: Disable indentation of XHTML output
Peter Eisentraut [Thu, 6 Feb 2014 04:49:25 +0000 (23:49 -0500)]
doc: Disable indentation of XHTML output

Indenting the XHTML output can lead to incorrect rendering.  This only
affects the build via XSLT.

10 years agoFix whitespace
Peter Eisentraut [Thu, 6 Feb 2014 04:12:51 +0000 (23:12 -0500)]
Fix whitespace

10 years agoRemove unnecessary relcache flushes after changing btree metapages.
Tom Lane [Wed, 5 Feb 2014 18:43:37 +0000 (13:43 -0500)]
Remove unnecessary relcache flushes after changing btree metapages.

These flushes were added in my commit d2896a9ed, which added the btree
logic that keeps a cached copy of the index metapage data in index relcache
entries.  The idea was to ensure that other backends would promptly update
their cached copies after a change.  However, this is not really necessary,
since _bt_getroot() has adequate defenses against believing a stale root
page link, and _bt_getrootheight() doesn't have to be 100% right.
Moreover, if it were necessary, a relcache flush would be an unreliable way
to do it, since the sinval mechanism believes that relcache flush requests
represent transactional updates, and therefore discards them on transaction
rollback.  Therefore, we might as well drop these flush requests and save
the time to rebuild the whole relcache entry after a metapage change.

If we ever try to support in-place truncation of btree indexes, it might
be necessary to revisit this issue so that _bt_getroot() can't get caught
by trying to follow a metapage link to a page that no longer exists.
A possible solution to that is to make use of an smgr, rather than
relcache, inval request to force other backends to discard their cached
metapages.  But for the moment this is not worth pursuing.

10 years agoMinor improvements to replication slot documentation.
Robert Haas [Wed, 5 Feb 2014 18:41:25 +0000 (13:41 -0500)]
Minor improvements to replication slot documentation.

Fix a thinko pointed out by Jeff Davis, and convert a couple of other
references into links.

10 years agoPL/Perl: Fix compiler warning
Peter Eisentraut [Wed, 5 Feb 2014 01:04:35 +0000 (20:04 -0500)]
PL/Perl: Fix compiler warning

The code was assigning a (Datum) 0 to a void pointer.  That creates a
warning from clang 3.4.  It was probably a thinko to begin with.

10 years agoImprove connection-failure error handling in contrib/postgres_fdw.
Tom Lane [Tue, 4 Feb 2014 02:30:02 +0000 (21:30 -0500)]
Improve connection-failure error handling in contrib/postgres_fdw.

postgres_fdw tended to say "unknown error" if it tried to execute a command
on an already-dead connection, because some paths in libpq just return a
null PGresult for such cases.  Out-of-memory might result in that, too.
To fix, pass the PGconn to pgfdw_report_error, and look at its
PQerrorMessage() string if we can't get anything out of the PGresult.

Also, fix the transaction-exit logic to reliably drop a dead connection.
It was attempting to do that already, but it assumed that only connection
cache entries with xact_depth > 0 needed to be examined.  The folly in that
is that if we fail while issuing START TRANSACTION, we'll not have bumped
xact_depth.  (At least for the case I was testing, this fix masks the
other problem; but it still seems like a good idea to have the PGconn
fallback logic.)

Per investigation of bug #9087 from Craig Lucas.  Backpatch to 9.3 where
this code was introduced.

10 years agoFix comparison of an array of characters with zero to compare with '\0' instead.
Fujii Masao [Tue, 4 Feb 2014 01:59:39 +0000 (10:59 +0900)]
Fix comparison of an array of characters with zero to compare with '\0' instead.

Report from Andres Freund.

10 years agoRephrase text to avoid links in regress.sgml
Robert Haas [Tue, 4 Feb 2014 01:10:48 +0000 (20:10 -0500)]
Rephrase text to avoid links in regress.sgml

Otherwise, the standalone regress_README build gets unhappy.

10 years agoFix lexing of U& sequences just before EOF.
Tom Lane [Tue, 4 Feb 2014 00:47:57 +0000 (19:47 -0500)]
Fix lexing of U& sequences just before EOF.

Commit a5ff502fceadc7c203b0d7a11b45c73f1b421f69 was a brick shy of a load
in the backend lexer too, not just psql.  Per further testing of bug #9068.

In passing, improve related comments.

10 years agoFix *-qualification of named parameters in SQL-language functions.
Tom Lane [Mon, 3 Feb 2014 19:46:51 +0000 (14:46 -0500)]
Fix *-qualification of named parameters in SQL-language functions.

Given a composite-type parameter named x, "$1.*" worked fine, but "x.*"
not so much.  This has been broken since named parameter references were
added in commit 9bff0780cf5be2193a5bad0d3df2dbe143085264, so patch back
to 9.2.  Per bug #9085 from Hardy Falk.

10 years agoAdjust pg_sleep_for/pg_sleep_until to use clock_timestamp.
Robert Haas [Mon, 3 Feb 2014 19:33:43 +0000 (14:33 -0500)]
Adjust pg_sleep_for/pg_sleep_until to use clock_timestamp.

Otherwise, pg_sleep_until does the wrong thing in a multi-statement
transaction.

Julien Rouhaud

10 years agoDocument a few more regression test hazards.
Robert Haas [Mon, 3 Feb 2014 15:27:47 +0000 (10:27 -0500)]
Document a few more regression test hazards.

Michael Paquier, reviewed by Christian Kruse

10 years agoIn json code, clean up temp memory contexts after processing.
Andrew Dunstan [Mon, 3 Feb 2014 15:40:12 +0000 (10:40 -0500)]
In json code, clean up temp memory contexts after processing.

Craig Ringer.

10 years agoMake pg_basebackup skip temporary statistics files.
Fujii Masao [Mon, 3 Feb 2014 14:19:49 +0000 (23:19 +0900)]
Make pg_basebackup skip temporary statistics files.

The temporary statistics files don't need to be included in the backup
because they are always reset at the beginning of the archive recovery.
This patch changes pg_basebackup so that it skips all files located in
$PGDATA/pg_stat_tmp or the directory specified by stats_temp_directory
parameter.

10 years agoSwitch in psql_scan() must cover all lexer states (except backslash cases).
Tom Lane [Sun, 2 Feb 2014 23:59:34 +0000 (18:59 -0500)]
Switch in psql_scan() must cover all lexer states (except backslash cases).

Oversight in commit f7559c0101afa33bfb4e104036ca46adac900111, which changed
UESCAPE lexing in psql.  Per bug #9068 from Manuel Gómez.

10 years agoAdd <indexterm> for functions pg_sleep_for and pg_sleep_until.
Fujii Masao [Sun, 2 Feb 2014 18:40:36 +0000 (03:40 +0900)]
Add <indexterm> for functions pg_sleep_for and pg_sleep_until.

10 years agoClean up some sloppy coding in repl_gram.y.
Tom Lane [Sun, 2 Feb 2014 17:51:14 +0000 (12:51 -0500)]
Clean up some sloppy coding in repl_gram.y.

Remove unused copy-and-pasted macro definitions, and improve formatting
of recently-added productions.

I got interested in this because buildfarm member protosciurus has been
crashing in "bison repl_gram.y" since commit 858ec11.  It's a long shot
that this will fix that, though maybe the missing trailing semicolon
has something to do with it?  In any case, there's no need to approve
of dead code, nor of code whose formatting isn't even self-consistent
let alone consistent with what's around it.

10 years agoAdd primary_slotname to recovery.conf.sample.
Fujii Masao [Sun, 2 Feb 2014 15:41:50 +0000 (00:41 +0900)]
Add primary_slotname to recovery.conf.sample.

10 years agoFix typos in docs and comments.
Fujii Masao [Sun, 2 Feb 2014 01:28:18 +0000 (10:28 +0900)]
Fix typos in docs and comments.

Thom Brown

10 years agoFix makefile syntax.
Andrew Dunstan [Sun, 2 Feb 2014 00:52:39 +0000 (19:52 -0500)]
Fix makefile syntax.

10 years agoFix some wide-character bugs in the text-search parser.
Tom Lane [Sat, 1 Feb 2014 23:26:55 +0000 (18:26 -0500)]
Fix some wide-character bugs in the text-search parser.

In p_isdigit and other character class test functions generated by the
p_iswhat macro, the code path for non-C locales with multibyte encodings
contained a bogus pointer cast that would accidentally fail to malfunction
if types wchar_t and wint_t have the same width.  Apparently that is true
on most platforms, but not on recent Cygwin releases.  Remove the cast,
as it seems completely unnecessary (I think it arose from a false analogy
to the need to cast to unsigned char when dealing with the <ctype.h>
functions).  Per bug #8970 from Marco Atzeri.

In the same functions, the code path for C locale with a multibyte encoding
simply ANDed each wide character with 0xFF before passing it to the
corresponding <ctype.h> function.  This could result in false positive
answers for some non-ASCII characters, so use a range test instead.
Noted by me while investigating Marco's complaint.

Also, remove some useless though not actually buggy maskings and casts
in the hand-coded p_isalnum and p_isalpha functions, which evidently
got tested a bit more carefully than the macro-generated functions.

10 years agofix whitespace
Andrew Dunstan [Sat, 1 Feb 2014 21:30:26 +0000 (16:30 -0500)]
fix whitespace

10 years agoFix some more bugs in signal handlers and process shutdown logic.
Tom Lane [Sat, 1 Feb 2014 21:20:56 +0000 (16:20 -0500)]
Fix some more bugs in signal handlers and process shutdown logic.

WalSndKill was doing things exactly backwards: it should first clear
MyWalSnd (to stop signal handlers from touching MyWalSnd->latch),
then disown the latch, and only then mark the WalSnd struct unused by
clearing its pid field.

Also, WalRcvSigUsr1Handler and worker_spi_sighup failed to preserve
errno, which is surely a requirement for any signal handler.

Per discussion of recent buildfarm failures.  Back-patch as far
as the relevant code exists.

10 years agoDon't use deprecated dllwrap on Cygwin.
Andrew Dunstan [Sat, 1 Feb 2014 21:08:33 +0000 (16:08 -0500)]
Don't use deprecated dllwrap on Cygwin.

The preferred method is to use "cc -shared", and this allows binaries
to be rebased if required, unlike dllwrap.

Backpatch to 9.0 where we have buildfarm coverage.

There are still some issues with Cygwin, especially modern Cygwin, but
this helps us get closer to good support.

Marco Atzeri.

10 years agoCopy the libpq DLL to the bin directory on Mingw and Cygwin.
Andrew Dunstan [Sat, 1 Feb 2014 20:11:13 +0000 (15:11 -0500)]
Copy the libpq DLL to the bin directory on Mingw and Cygwin.

This has long been done by the MSVC build system, and has caused
confusion in the past when programs like psql have failed to start
because they can't find the DLL. If it's in the same directory as it now
will be they will find it.

Backpatch to all live branches.

10 years agoarrays: tighten checks for multi-dimensional input
Bruce Momjian [Sat, 1 Feb 2014 15:49:17 +0000 (10:49 -0500)]
arrays:  tighten checks for multi-dimensional input

Previously an input array string that started with a single-element
array dimension would then later accept a multi-dimensional segment.

BACKWARD INCOMPATIBILITY

10 years agoIntroduce replication slots.
Robert Haas [Sat, 1 Feb 2014 03:45:17 +0000 (22:45 -0500)]
Introduce replication slots.

Replication slots are a crash-safe data structure which can be created
on either a master or a standby to prevent premature removal of
write-ahead log segments needed by a standby, as well as (with
hot_standby_feedback=on) pruning of tuples whose removal would cause
replication conflicts.  Slots have some advantages over existing
techniques, as explained in the documentation.

In a few places, we refer to the type of replication slots introduced
by this patch as "physical" slots, because forthcoming patches for
logical decoding will also have slots, but with somewhat different
properties.

Andres Freund and Robert Haas

10 years agodocs: mention 'g' is not in the regex embedded options table
Bruce Momjian [Sat, 1 Feb 2014 03:40:08 +0000 (22:40 -0500)]
docs:  mention 'g' is not in the regex embedded options table

Mentioned in substring() and regexp_replace() sections.

10 years agopg_restore: make help output plural for multi-enabled options
Bruce Momjian [Sat, 1 Feb 2014 03:28:54 +0000 (22:28 -0500)]
pg_restore:  make help output plural for multi-enabled options

per report from Josh Kupershmidt

10 years agoClear MyProc and MyProcSignalState before they become invalid.
Robert Haas [Sat, 1 Feb 2014 02:31:08 +0000 (21:31 -0500)]
Clear MyProc and MyProcSignalState before they become invalid.

Evidence from buildfarm member crake suggests that the new test_shm_mq
module is routinely crashing the server due to the arrival of a SIGUSR1
after the shared memory segment has been unmapped.  Although processes
using the new dynamic background worker facilities are more likely to
receive a SIGUSR1 around this time, the problem is also possible on older
branches, so I'm back-patching the parts of this change that apply to
older branches as far as they apply.

It's already generally the case that code checks whether these pointers
are NULL before deferencing them, so the important thing is mostly to
make sure that they do get set to NULL before they become invalid.  But
in master, there's one case in procsignal_sigusr1_handler that lacks a
NULL guard, so add that.

Patch by me; review by Tom Lane.

10 years agodoc: mention statistics reset during crash recovery
Bruce Momjian [Sat, 1 Feb 2014 02:28:15 +0000 (21:28 -0500)]
doc: mention statistics reset during crash recovery

Takayuki Tsunakawa

10 years agochkpass: check for NULL return value from crypt()
Bruce Momjian [Sat, 1 Feb 2014 01:19:53 +0000 (20:19 -0500)]
chkpass:  check for NULL return value from crypt()

Report from Jozef Mlich using Coverity

10 years agodoc: mention data page checksums in WAL section
Bruce Momjian [Sat, 1 Feb 2014 00:04:37 +0000 (19:04 -0500)]
doc: mention data page checksums in WAL section

Backpatch to 9.3

Adjusted patch from Ian Lawrence Barwick

10 years agoDisallow use of SSL v3 protocol in the server as well as in libpq.
Tom Lane [Fri, 31 Jan 2014 22:51:07 +0000 (17:51 -0500)]
Disallow use of SSL v3 protocol in the server as well as in libpq.

Commit 820f08cabdcbb8998050c3d4873e9619d6d8cba4 claimed to make the server
and libpq handle SSL protocol versions identically, but actually the server
was still accepting SSL v3 protocol while libpq wasn't.  Per discussion,
SSL v3 is obsolete, and there's no good reason to continue to accept it.
So make the code really equivalent on both sides.  The behavior now is
that we use the highest mutually-supported TLS protocol version.

Marko Kreen, some comment-smithing by me

10 years agodocs: mention tgconstrindid supports exclusion constraints
Bruce Momjian [Fri, 31 Jan 2014 22:44:18 +0000 (17:44 -0500)]
docs:  mention tgconstrindid supports exclusion constraints

Dean Rasheed

10 years agoAdd some examples to the postgres_fdw documentation.
Tom Lane [Fri, 31 Jan 2014 22:27:50 +0000 (17:27 -0500)]
Add some examples to the postgres_fdw documentation.

Michael Paquier

10 years agodoc: authentication wording improvements
Bruce Momjian [Fri, 31 Jan 2014 22:08:27 +0000 (17:08 -0500)]
doc: authentication wording improvements

Suggested by David Tonhofer

10 years agodocs: specify FOR UPDATE/SHARE incompatibilities
Bruce Momjian [Fri, 31 Jan 2014 21:37:25 +0000 (16:37 -0500)]
docs:  specify FOR UPDATE/SHARE incompatibilities

Document that FOR UPDATE/SHARE are incompatible with GROUP BY, DISTINCT,
HAVING and window functions.

Michael Paquier

10 years agosystem catalogs: reorder pg_amproc entries into proper sections
Bruce Momjian [Fri, 31 Jan 2014 21:04:18 +0000 (16:04 -0500)]
system catalogs:  reorder pg_amproc entries into proper sections

Report form Antonin Houska

10 years agopgindent: add Perl comment
Bruce Momjian [Fri, 31 Jan 2014 19:46:00 +0000 (14:46 -0500)]
pgindent:  add Perl comment

10 years agopgindent: add --list-of-typedefs option
Bruce Momjian [Fri, 31 Jan 2014 18:35:50 +0000 (13:35 -0500)]
pgindent:  add --list-of-typedefs option

Allows typedefs to be specified on the command line, per request from
Andrew.

10 years agoAdd tab completion for ALTER TABLESPACE MOVE in psql.
Fujii Masao [Fri, 31 Jan 2014 16:45:48 +0000 (01:45 +0900)]
Add tab completion for ALTER TABLESPACE MOVE in psql.

10 years agoentab: add new options
Bruce Momjian [Fri, 31 Jan 2014 16:05:15 +0000 (11:05 -0500)]
entab:  add new options

Add new entab options to process only C comment whitespace after
periods, and to protect leading whitespace.

10 years agoFix documented return type of json_array_elements_text.
Andrew Dunstan [Fri, 31 Jan 2014 14:31:58 +0000 (09:31 -0500)]
Fix documented return type of json_array_elements_text.

Per gripe from hubert depesz lubaczewski.

Also improve examples for this and json_array_elements so they match the
example results.

10 years agoAllow unrecognized encoding names in locales, as long as they're the same.
Tom Lane [Fri, 31 Jan 2014 07:03:30 +0000 (02:03 -0500)]
Allow unrecognized encoding names in locales, as long as they're the same.

The buildfarm says commit 58274728fb8e087049df67c0eee903d9743fdeda doesn't
work so well on Windows.  This is because the encoding part of Windows
locale names can be just a code page number, eg "1252", which we don't
consider to be a valid encoding name.  Add a check to accept encoding
parts that are case-insensitively string equal; this at least ensures
that the new code doesn't reject any cases that the old code allowed.

10 years agopgindent: preserve blank lines around #else/#endif
Bruce Momjian [Fri, 31 Jan 2014 03:40:05 +0000 (22:40 -0500)]
pgindent:  preserve blank lines around #else/#endif

This requires a new version of pg_bsd_indent, version 1.3, to be
downloaded.

10 years agoBe forgiving of variant spellings of locale names in pg_upgrade.
Tom Lane [Fri, 31 Jan 2014 00:07:06 +0000 (19:07 -0500)]
Be forgiving of variant spellings of locale names in pg_upgrade.

Even though the server tries to canonicalize stored locale names, the
platform often doesn't cooperate, so it's entirely possible that one DB
thinks its locale is, say, "en_US.UTF-8" while the other has "en_US.utf8".
Rather than failing, we should try to allow this where it's clearly OK.

There is already pretty robust encoding lookup in encnames.c, so make
use of that to compare the encoding parts of the names.  The locale
identifier parts are just compared case-insensitively, which we were
already doing.  The major problem known to exist in the field is variant
encoding-name spellings, so hopefully this will be Good Enough.  If not,
we can try being even laxer.

Pavel Raiskup, reviewed by Rushabh Lathia

10 years agoFix potential coredump on bad locale value in pg_upgrade.
Tom Lane [Thu, 30 Jan 2014 23:10:01 +0000 (18:10 -0500)]
Fix potential coredump on bad locale value in pg_upgrade.

Thinko in error report (and a typo in the message text, too).  We're
failing anyway, but it would be good to print something useful first.
Noted while reviewing a patch to make pg_upgrade's locale code laxer.

10 years agoAdd convenience functions pg_sleep_for and pg_sleep_until.
Robert Haas [Thu, 30 Jan 2014 20:45:14 +0000 (15:45 -0500)]
Add convenience functions pg_sleep_for and pg_sleep_until.

Vik Fearing, reviewed by Pavel Stehule and myself

10 years agoFix bogus handling of "postponed" lateral quals.
Tom Lane [Thu, 30 Jan 2014 19:51:16 +0000 (14:51 -0500)]
Fix bogus handling of "postponed" lateral quals.

When pulling a "postponed" qual from a LATERAL subquery up into the quals
of an outer join, we must make sure that the postponed qual is included
in those seen by make_outerjoininfo().  Otherwise we might compute a
too-small min_lefthand or min_righthand for the outer join, leading to
"JOIN qualification cannot refer to other relations" failures from
distribute_qual_to_rels.  Subtler errors in the created plan seem possible,
too, if the extra qual would only affect join ordering constraints.

Per bug #9041 from David Leverton.  Back-patch to 9.3.

10 years agodocs: improve xref description for xreflabel and refentry links
Bruce Momjian [Thu, 30 Jan 2014 17:26:18 +0000 (12:26 -0500)]
docs:  improve xref description for xreflabel and refentry links

10 years agodocs: mention SGML refentry also pulls text from the link
Bruce Momjian [Thu, 30 Jan 2014 17:21:48 +0000 (12:21 -0500)]
docs:  mention SGML refentry also pulls text from the link

10 years agodocs: add mention of index swapping
Bruce Momjian [Thu, 30 Jan 2014 17:14:24 +0000 (12:14 -0500)]
docs:  add mention of index swapping

Backpatch to 9.3

Greg Smith

10 years agoAdd checks for interval overflow/underflow
Bruce Momjian [Thu, 30 Jan 2014 14:41:43 +0000 (09:41 -0500)]
Add checks for interval overflow/underflow

New checks include input, month/day/time internal adjustments, addition,
subtraction, multiplication, and negation.  Also adjust docs to
correctly specify interval size in bytes.

Report from Rok Kralj

10 years agoFix unsafe references to errno within error messaging logic.
Tom Lane [Thu, 30 Jan 2014 01:03:57 +0000 (20:03 -0500)]
Fix unsafe references to errno within error messaging logic.

Various places were supposing that errno could be expected to hold still
within an ereport() nest or similar contexts.  This isn't true necessarily,
though in some cases it accidentally failed to fail depending on how the
compiler chanced to order the subexpressions.  This class of thinko
explains recent reports of odd failures on clang-built versions, typically
missing or inappropriate HINT fields in messages.

Problem identified by Christian Kruse, who also submitted the patch this
commit is based on.  (I fixed a few issues in his patch and found a couple
of additional places with the same disease.)

Back-patch as appropriate to all supported branches.

10 years agoSilence compiler warnings about possibly unset variables.
Andrew Dunstan [Wed, 29 Jan 2014 23:54:14 +0000 (18:54 -0500)]
Silence compiler warnings about possibly unset variables.

They are in fact set in every case where they are needed, but the
compiler doesn't know that.

Per gripe from Tom Lane.

10 years agoForgot to bump catalog version for json_array_elements_text.
Andrew Dunstan [Wed, 29 Jan 2014 21:38:31 +0000 (16:38 -0500)]
Forgot to bump catalog version for json_array_elements_text.

10 years agoInclude planning time in EXPLAIN ANALYZE output.
Robert Haas [Wed, 29 Jan 2014 21:04:19 +0000 (16:04 -0500)]
Include planning time in EXPLAIN ANALYZE output.

This doesn't work for prepared queries, but it's not too easy to get
the information in that case and there's some debate as to exactly
what the right thing to measure is, so just do this for now.

Andreas Karlsson, with slight doc changes by me.

10 years agoAdd json_array_elements_text function.
Andrew Dunstan [Wed, 29 Jan 2014 20:39:01 +0000 (15:39 -0500)]
Add json_array_elements_text function.

This was a notable omission from the json functions added in 9.3 and
there have been numerous complaints about its absence.

Laurence Rowe.

10 years agoFix thinko in huge_tlb_pages patch.
Heikki Linnakangas [Wed, 29 Jan 2014 19:33:56 +0000 (21:33 +0200)]
Fix thinko in huge_tlb_pages patch.

We calculated the rounded-up size for the allocation, but then failed to
use the rounded-up value in the mmap() call. Oops.

Also, initialize allocsize, to silence warnings seen with some compilers,
as pointed out by Jeff Janes.

10 years agoFurther optimize GIN multi-key searches.
Heikki Linnakangas [Wed, 29 Jan 2014 19:22:08 +0000 (21:22 +0200)]
Further optimize GIN multi-key searches.

When skipping over some items in a posting tree, re-find the new location
by descending the tree from root, rather than walking the right links.
This can save a lot of I/O.

Heavily modified from Alexander Korotkov's fast scan patch.

10 years agoFix pointer processing in new entab.c function
Bruce Momjian [Wed, 29 Jan 2014 18:31:11 +0000 (13:31 -0500)]
Fix pointer processing in new entab.c function

10 years agoAdd C functions to centralize entab processing
Bruce Momjian [Wed, 29 Jan 2014 17:48:07 +0000 (12:48 -0500)]
Add C functions to centralize entab processing

10 years agoAdd more C comments to entab.c.
Bruce Momjian [Wed, 29 Jan 2014 17:22:12 +0000 (12:22 -0500)]
Add more C comments to entab.c.

10 years agoFurther optimize multi-key GIN searches.
Heikki Linnakangas [Wed, 29 Jan 2014 16:23:17 +0000 (18:23 +0200)]
Further optimize multi-key GIN searches.

If we're skipping past a certain TID, avoid decoding posting list segments
that only contain smaller TIDs.

Extracted from Alexander Korotkov's fast scan patch, heavily modified.

10 years agoAllow skipping some items in a multi-key GIN search.
Heikki Linnakangas [Wed, 29 Jan 2014 15:53:39 +0000 (17:53 +0200)]
Allow skipping some items in a multi-key GIN search.

In a multi-key search, ie. something like "col @> 'foo' AND col @> 'bar'",
as soon as we find the next item that matches the first criteria, we don't
need to check the second criteria for TIDs smaller the first match. That
saves a lot of effort, especially if one of the terms is rare, while the
second occurs very frequently.

Based on ideas from Alexander Korotkov's fast scan patch.

10 years agoRemedy two small omissions in the new json functions docs.
Andrew Dunstan [Wed, 29 Jan 2014 14:51:02 +0000 (09:51 -0500)]
Remedy two small omissions in the new json functions docs.

10 years agoFix docs build.
Heikki Linnakangas [Wed, 29 Jan 2014 14:16:19 +0000 (16:16 +0200)]
Fix docs build.

Broken by the huge_tlb_pages patch.

Vik Fearing.

10 years agoAllow using huge TLB pages on Linux (MAP_HUGETLB)
Heikki Linnakangas [Wed, 29 Jan 2014 11:44:45 +0000 (13:44 +0200)]
Allow using huge TLB pages on Linux (MAP_HUGETLB)

This patch adds an option, huge_tlb_pages, which allows requesting the
shared memory segment to be allocated using huge pages, by using the
MAP_HUGETLB flag in mmap(). This can improve performance.

The default is 'try', which means that we will attempt using huge pages,
and fall back to non-huge pages if it doesn't work. Currently, only Linux
has MAP_HUGETLB. On other platforms, the default 'try' behaves the same as
'off'.

In the passing, don't try to round the mmap() size to a multiple of
pagesize. mmap() doesn't require that, and there's no particular reason for
PostgreSQL to do that either. When using MAP_HUGETLB, however, round the
request size up to nearest 2MB boundary. This is to work around a bug in
some Linux kernel versions, but also to avoid wasting memory, because the
kernel will round the size up anyway.

Many people were involved in writing this patch, including Christian Kruse,
Richard Poole, Abhijit Menon-Sen, reviewed by Peter Geoghegan, Andres Freund
and me.

10 years agoFix compiler warning in EXEC_BACKEND builds.
Robert Haas [Wed, 29 Jan 2014 04:35:50 +0000 (23:35 -0500)]
Fix compiler warning in EXEC_BACKEND builds.

Per a report by Rajeev Rastogi.

10 years agoMinor docs fixes for new json functions.
Andrew Dunstan [Tue, 28 Jan 2014 23:28:16 +0000 (18:28 -0500)]
Minor docs fixes for new json functions.

Thom Brown.

10 years agoAdd new make targets check-tests and installcheck-tests.
Andrew Dunstan [Tue, 28 Jan 2014 23:10:00 +0000 (18:10 -0500)]
Add new make targets check-tests and installcheck-tests.

These do not run any specific schedule of tests, but only those
specified as part of the invocation, e.g.:

    make check-tests TESTS="json jsonb"

10 years agoNew json functions.
Andrew Dunstan [Tue, 28 Jan 2014 22:48:21 +0000 (17:48 -0500)]
New json functions.

json_build_array() and json_build_object allow for the construction of
arbitrarily complex json trees. json_object() turns a one or two
dimensional array, or two separate arrays, into a json_object of
name/value pairs, similarly to the hstore() function.
json_object_agg() aggregates its two arguments into a single json object
as name value pairs.

Catalog version bumped.

Andrew Dunstan, reviewed by Marko Tiikkaja.

10 years agoAdd pg_stat_archiver statistics view.
Fujii Masao [Tue, 28 Jan 2014 17:58:22 +0000 (02:58 +0900)]
Add pg_stat_archiver statistics view.

This view shows the statistics about the WAL archiver process's activity.

Gabriele Bartolini, reviewed by Michael Paquier, refactored a bit by me.

10 years agoUpdate comment.
Tom Lane [Tue, 28 Jan 2014 17:34:29 +0000 (12:34 -0500)]
Update comment.

generate_normalized_query() no longer needs to truncate text, but this
one comment didn't get the memo.  Per Peter Geoghegan.

10 years agoRevert C comment change in slot_attisnull()
Bruce Momjian [Tue, 28 Jan 2014 17:28:14 +0000 (12:28 -0500)]
Revert C comment change in slot_attisnull()

Revert 89774b58b0ea2874765cae10c094bb6aaf707feb

10 years agoRemove orphaned prototype
Bruce Momjian [Tue, 28 Jan 2014 16:29:39 +0000 (11:29 -0500)]
Remove orphaned prototype

Rajeev rastogi

10 years agoRevert dup2() checking in syslogger.c
Stephen Frost [Tue, 28 Jan 2014 13:40:41 +0000 (08:40 -0500)]
Revert dup2() checking in syslogger.c

Per the expanded comment-

As we're just trying to reset these to go to DEVNULL, there's not
much point in checking for failure from the close/dup2 calls here,
if they fail then presumably the file descriptors are closed and
any writes will go into the bitbucket anyway.

Pointed out by Tom.

10 years agoLog a detail message for auth failures due to missing or expired password.
Tom Lane [Tue, 28 Jan 2014 02:04:09 +0000 (21:04 -0500)]
Log a detail message for auth failures due to missing or expired password.

It's worth distinguishing these cases from run-of-the-mill wrong-password
problems, since users have been known to waste lots of time pursuing the
wrong theory about what's failing.  Now, our longstanding policy about how
to report authentication failures is that we don't really want to tell the
*client* such things, since that might be giving information to a bad guy.
But there's nothing wrong with reporting the details to the postmaster log,
and indeed the comments in this area of the code contemplate that
interesting details should be so reported.  We just weren't handling these
particular interesting cases usefully.

To fix, add infrastructure allowing subroutines of ClientAuthentication()
to return a string to be added to the errdetail_log field of the main
authentication-failed error report.  We might later want to use this to
report other subcases of authentication failure the same way, but for the
moment I just dealt with password cases.

Per discussion of a patch from Josh Drake, though this is not what
he proposed.

10 years agoKeep pg_stat_statements' query texts in a file, not in shared memory.
Tom Lane [Mon, 27 Jan 2014 20:37:54 +0000 (15:37 -0500)]
Keep pg_stat_statements' query texts in a file, not in shared memory.

This change allows us to eliminate the previous limit on stored query
length, and it makes the shared-memory hash table very much smaller,
allowing more statements to be tracked.  (The default value of
pg_stat_statements.max is therefore increased from 1000 to 5000.)
In typical scenarios, the hash table can be large enough to hold all the
statements commonly issued by an application, so that there is little
"churn" in the set of tracked statements, and thus little need to do I/O
to the file.

To further reduce the need for I/O to the query-texts file, add a way
to retrieve all the columns of the pg_stat_statements view except for
the query text column.  This is probably not of much interest for human
use but it could be exploited by programs, which will prefer using the
queryid anyway.

Ordinarily, we'd need to bump the extension version number for the latter
change.  But since we already advanced pg_stat_statements' version number
from 1.1 to 1.2 in the 9.4 development cycle, it seems all right to just
redefine what 1.2 means.

Peter Geoghegan, reviewed by Pavel Stehule

10 years agoRelax the requirement that all lwlocks be stored in a single array.
Robert Haas [Mon, 27 Jan 2014 16:07:44 +0000 (11:07 -0500)]
Relax the requirement that all lwlocks be stored in a single array.

This makes it possible to store lwlocks as part of some other data
structure in the main shared memory segment, or in a dynamic shared
memory segment.  There is still a main LWLock array and this patch does
not move anything out of it, but it provides necessary infrastructure
for doing that in the future.

This change is likely to increase the size of LWLockPadded on some
platforms, especially 32-bit platforms where it was previously only
16 bytes.

Patch by me.  Review by Andres Freund and KaiGai Kohei.

10 years agoFix typo in README
Heikki Linnakangas [Mon, 27 Jan 2014 07:32:51 +0000 (09:32 +0200)]
Fix typo in README

Amit Langote

10 years agoCode review for auto-tuned effective_cache_size.
Tom Lane [Mon, 27 Jan 2014 05:05:49 +0000 (00:05 -0500)]
Code review for auto-tuned effective_cache_size.

Fix integer overflow issue noted by Magnus Hagander, as well as a bunch
of other infelicities in commit ee1e5662d8d8330726eaef7d3110cb7add24d058
and its unreasonably large number of followups.

10 years agoChange the suffix of auto conf temporary file from "temp" to "tmp".
Fujii Masao [Mon, 27 Jan 2014 03:39:11 +0000 (12:39 +0900)]
Change the suffix of auto conf temporary file from "temp" to "tmp".

Michael Paquier

10 years agoFix typos in comments for ALTER SYSTEM.
Fujii Masao [Mon, 27 Jan 2014 03:23:20 +0000 (12:23 +0900)]
Fix typos in comments for ALTER SYSTEM.

Michael Paquier

10 years agoFix minor leak in pg_dump
Stephen Frost [Sun, 26 Jan 2014 22:58:48 +0000 (17:58 -0500)]
Fix minor leak in pg_dump

Move allocation to after we check the remote server version, to avoid
a possible, very minor, memory leak.  This makes us more consistent
throughout as most places in pg_dump are done in the same way (due, in
part, to previous fixes like this).

Spotted by the Coverity scanner.

10 years agoProvide for client-only installs with MSVC.
Andrew Dunstan [Sun, 26 Jan 2014 22:03:13 +0000 (17:03 -0500)]
Provide for client-only installs with MSVC.

MauMau.

10 years agoCheck dup2() results in syslogger
Stephen Frost [Sun, 26 Jan 2014 21:26:18 +0000 (16:26 -0500)]
Check dup2() results in syslogger

Consistently check the dup2() call results throughout syslogger.c.
It's pretty unlikely that they'll error out, but if they do,
ereport(FATAL) instead of blissfully continuing on.

Spotted by the Coverity scanner.

10 years agoMove the options column of \db+ before the description
Magnus Hagander [Sun, 26 Jan 2014 20:13:57 +0000 (21:13 +0100)]
Move the options column of \db+ before the description

The convention is to have the description field at the end.

Noted by Tom Lane

10 years agoInclude tablespace options in verbose output of \db
Magnus Hagander [Sun, 26 Jan 2014 17:11:15 +0000 (18:11 +0100)]
Include tablespace options in verbose output of \db

10 years agoEnable building with Visual Studion 2013.
Andrew Dunstan [Sun, 26 Jan 2014 14:49:10 +0000 (09:49 -0500)]
Enable building with Visual Studion 2013.

Backpatch to 9.3.

Brar Piening.

10 years agoUse E, not e, for escaping in example docs
Stephen Frost [Sun, 26 Jan 2014 14:40:34 +0000 (09:40 -0500)]
Use E, not e, for escaping in example docs

From the Department of Nitpicking, be consistent with other escaping
and use 'E' instead of 'e' to escape the string in the example docs
for GET DISAGNOSTICS stack = PG_CONTEXT.

Noticed by Department Chief Magnus Hagander.

10 years agoAdjust C comment in slot_attisnull() regarding nulls.
Bruce Momjian [Sat, 25 Jan 2014 21:43:14 +0000 (16:43 -0500)]
Adjust C comment in slot_attisnull() regarding nulls.

10 years agoCopy-editing docs for recovery_target='immediate', and "ie."
Heikki Linnakangas [Sat, 25 Jan 2014 21:30:17 +0000 (23:30 +0200)]
Copy-editing docs for recovery_target='immediate', and "ie."

Phrase the sentence better. Also, spell "i.e." rather than "ie."

Thom Brown

10 years agodocs: mention CREATE TABLE LIKE linkage using INCLUDING DEFAULTS
Bruce Momjian [Sat, 25 Jan 2014 19:32:48 +0000 (14:32 -0500)]
docs: mention CREATE TABLE LIKE linkage using INCLUDING DEFAULTS

Mention that CREATE TABLE LIKE INCLUDING DEFAULTS creates a link between
the original and new tables if a default function modifies the database,
like nextval().

10 years agoAdd recovery_target='immediate' option.
Heikki Linnakangas [Sat, 25 Jan 2014 15:34:04 +0000 (17:34 +0200)]
Add recovery_target='immediate' option.

This allows ending recovery as a consistent state has been reached. Without
this, there was no easy way to e.g restore an online backup, without
replaying any extra WAL after the backup ended.

MauMau and me.

10 years agolibpq: Support TLS versions beyond TLSv1.
Noah Misch [Sat, 25 Jan 2014 00:29:06 +0000 (19:29 -0500)]
libpq: Support TLS versions beyond TLSv1.

Per report from Jeffrey Walton, libpq has been accepting only TLSv1
exactly.  Along the lines of the backend code, libpq will now support
new versions as OpenSSL adds them.

Marko Kreen, reviewed by Wim Lewis.

10 years agopsql: Mention SSL protocol version in \conninfo.
Noah Misch [Sat, 25 Jan 2014 00:23:56 +0000 (19:23 -0500)]
psql: Mention SSL protocol version in \conninfo.

Marko Kreen, reviewed by Wim Lewis.

10 years agoAvoid minor leak in parallel pg_dump
Stephen Frost [Fri, 24 Jan 2014 20:10:08 +0000 (15:10 -0500)]
Avoid minor leak in parallel pg_dump

During parallel pg_dump, a worker process closing the connection caused
a minor memory leak (particularly minor as we are likely about to exit
anyway).  Instead, free the memory in this case prior to returning NULL
to indicate connection closed.

Spotting by the Coverity scanner.

Back patch to 9.3 where this was introduced.

10 years agoReset unused fields in GIN data leaf page footer.
Heikki Linnakangas [Fri, 24 Jan 2014 17:08:52 +0000 (19:08 +0200)]
Reset unused fields in GIN data leaf page footer.

The maxoff field is not used in the new, compressed page format. Let's
reset it when converting an old-format page to the new format. The code
won't care either way, but this makes it possible to use the field for
something else in the future.