git.postgresql.org Git - postgresql.git/log
Tom Lane [Wed, 1 Apr 2009 03:32:29 +0000 (03:32 +0000)]
Index some array functions, per Mario Splivalo.
Tom Lane [Wed, 1 Apr 2009 03:23:50 +0000 (03:23 +0000)]
Remove last references to the crypt auth method, per Andreas Scherbaum.
Tom Lane [2009年3月31日 22:54:31 +0000 (22:54 +0000)]
Fix contrib/pgstattuple and contrib/pageinspect to prevent attempts to read
temporary tables of other sessions; that is unsafe because of the way our
buffer management works. Per report from Stuart Bishop.
This is redundant with the bufmgr.c checks in HEAD, but not at all redundant
in the back branches.
Remove some "Other" sections in the release notes by putting the items
at the top of their sections.
Tom Lane [2009年3月31日 22:12:48 +0000 (22:12 +0000)]
Modify the relcache to record the temp status of both local and nonlocal
temp relations; this is no more expensive than before, now that we have
pg_class.relistemp. Insert tests into bufmgr.c to prevent attempting
to fetch pages from nonlocal temp relations. This provides a low-level
defense against bugs-of-omission allowing temp pages to be loaded into shared
buffers, as in the contrib/pgstattuple problem reported by Stuart Bishop.
While at it, tweak a bunch of places to use new relcache tests (instead of
expensive probes into pg_namespace) to detect local or nonlocal temp tables.
Don't crash initdb when we fail to get the current username.
Give an error message and exit instead, like we do elsewhere...
Per report from Wez Furlong and Robert Treat.
Tom Lane [2009年3月31日 17:59:56 +0000 (17:59 +0000)]
Add a "relistemp" boolean column to pg_class, which is true for temporary
relations (including a temp table's indexes and toast table/index), and
false for normal relations. For ease of checking, this commit just adds
the column and fills it correctly --- revising the relation access machinery
to use it will come separately.
Fix a rare race condition when commit_siblings > 0 and a transaction commits
at the same instant as a new backend is spawned. Since CountActiveBackends()
doesn't hold ProcArrayLock, it needs to be prepared for the case that a
pointer at the end of the proc array is still NULL even though numProcs says
it should be valid, since it doesn't hold ProcArrayLock. Backpatch to 8.1.
8.0 and earlier had this right, but it was broken in the split of PGPROC and
sinval shared memory arrays.
Per report and proposal by Marko Kreen.
Update release note introductory description.
Tom Lane [2009年3月31日 01:41:27 +0000 (01:41 +0000)]
Add PQinitOpenSSL() function to support applications that use libcrypto
but not OpenSSL (or perhaps vice versa, if that's possible).
Andrew Chernow, with minor editorialization by me.
More new subsections in release notes.
More release note changes, including a lower level of subsections.
More release note adjustments, reordering.
More release note wording improvements; section order adjustments.
Reorder release note sections.
Tom Lane [2009年3月30日 17:30:44 +0000 (17:30 +0000)]
Fix window function plan generation to cope with volatile sort expressions.
(Not clear how useful these really are, but failing is no good...)
Per report from David Fetter and Robert Treat.
Update URL to Python bug tracker. Backpatch to 8.3; doesn't seem worthy of
further backpatch.
Tom Lane [2009年3月30日 04:08:43 +0000 (04:08 +0000)]
Fix an oversight in the support for storing/retrieving "minimal tuples" in
TupleTableSlots. We have functions for retrieving a minimal tuple from a slot
after storing a regular tuple in it, or vice versa; but these were implemented
by converting the internal storage from one format to the other. The problem
with that is it invalidates any pass-by-reference Datums that were already
fetched from the slot, since they'll be pointing into the just-freed version
of the tuple. The known problem cases involve fetching both a whole-row
variable and a pass-by-reference value from a slot that is fed from a
tuplestore or tuplesort object. The added regression tests illustrate some
simple cases, but there may be other failure scenarios traceable to the same
bug. Note that the added tests probably only fail on unpatched code if it's
built with --enable-cassert; otherwise the bug leads to fetching from freed
memory, which will not have been overwritten without additional conditions.
Fix by allowing a slot to contain both formats simultaneously; which turns out
not to complicate the logic much at all, if anything it seems less contorted
than before.
Back-patch to 8.2, where minimal tuples were introduced.
More release note markup.
More release note markup.
Add SGML markup for commands/literal/application/etc in release notes;
still more work to do.
Consistent 8.4 release note naming for Itagaki Takahiro
Clarify variable naming: pq_initssllib -> pq_init_ssl_lib
Update release notes to say citext is multi-byte aware, per suggestion
from patch author:
Add /contrib/citext as a case-insensitive, multibyte-capable
text data type (David Wheeler)
Better document that SET ROLE does not uset ALTER ROLE SET settings;
suggested wording from Josh Berkus.
Better document PQinitSSL(0) behavior in regards to libcrypto.
Tom Lane [2009年3月28日 00:10:23 +0000 (00:10 +0000)]
Add documentation of the fact that dtrace probes evaluate their parameters
even when not active. Explain how to prevent that with an ENABLED() check.
Document in release notes that NOT IN is only for NOT EXIST clauses.
Andrew Gierth
Tom Lane [2009年3月27日 19:58:11 +0000 (19:58 +0000)]
On Solaris, we should only force use of our own getopt(); it's okay to use
the system's getopt_long(). The previous coding was the result of a sloppy
discussion that failed to draw this distinction. The result was that PG
programs don't handle options as users of that platform expect. Per
gripe from Chuck McDevitt.
Although this is a pre-existing bug, I'm not backpatching since I think we
could do with a bit of beta testing before concluding this is really OK.
Fix markup, per Devrim
Tom Lane [2009年3月27日 18:56:57 +0000 (18:56 +0000)]
Add an errdetail explaining why we reject infinite dates and timestamps
while converting to XML. Bernd Helmle
Tom Lane [2009年3月27日 18:30:21 +0000 (18:30 +0000)]
Fix possible failures when a tuplestore switches from in-memory to on-disk
mode while callers hold pointers to in-memory tuples. I reported this for
the case of nodeWindowAgg's primary scan tuple, but inspection of the code
shows that all of the calls in nodeWindowAgg and nodeCtescan are at risk.
For the moment, fix it with a rather brute-force approach of copying
whenever one of the at-risk callers requests a tuple. Later we might
think of some sort of reference-count approach to reduce tuple copying.
Tom Lane [2009年3月27日 15:57:11 +0000 (15:57 +0000)]
Teach reindex_index() to clear pg_index.indcheckxmin when possible.
Greg Stark, slightly modified by me.
Fix tab completion of ANALYZE VERBOSE <tab>. It was previously confused
with EXPLAIN ANALYZE VERBOSE.
Greg Sabino Mullane, reformatted by myself. Backpatch to 8.1, where the
bug was introduced.
Clearify new SSL certificate verification in libpq
Fix release notes about pg_hba changes
Updated release wording, per Greg Stark:
Previously EXPLAIN VERBOSE had output an internal
representation of the
Second batch of release note fixes by Guillaume Smet
Mark Greg as the instigator of the statistics target increase:
Increase the default value of default_statistics_target from 10 to 100
(Greg Sabino Mullane, Tom)
First round of release note changes from Guillaume Smet.
Fix release note attribution:
Support the IS0 8601 time interval syntax (Tom, Kevin
Grittner)
per Ron Mayer
Improved release notes interval wording:
Made interval seconds rounding more consistent across output
formats (Ron Mayer)
Ron Mayer
Remove duplicate interval (fractional seconds) items; Ron Mayer
Document that Datestyle no longer controls interval output, per
suggestion from Ron Mayer
Tom Lane [2009年3月26日 22:29:13 +0000 (22:29 +0000)]
Make pg_standby's maxretries option do what one would expect. Fujii Masao
Gettext plural support
In the backend, I changed only a handful of exemplary or important-looking
instances to make use of the plural support; there is probably more work
there. For the rest of the source, this should cover all relevant cases.
Tom Lane [2009年3月26日 20:55:49 +0000 (20:55 +0000)]
Fix a couple of places that still claimed LC_COLLATE and LC_CTYPE can't
be changed after initdb.
Tom Lane [2009年3月26日 19:24:54 +0000 (19:24 +0000)]
Adjust phrasing of complaints about multi-byte COPY delimiter strings.
Per pgsql-hackers discussion of 2009年02月17日.
Tom Lane [2009年3月26日 17:15:35 +0000 (17:15 +0000)]
If we expect a hash join to be performed in multiple batches, suppress
"physical tlist" optimization on the outer relation (ie, force a projection
step to occur in its scan). This avoids storing useless column values when
the outer relation's tuples are written to temporary batch files.
Modified version of a patch by Michael Henderson and Ramon Lawrence.
Correction: ansi-join ->anti-join.
Tom Lane [2009年3月26日 03:46:14 +0000 (03:46 +0000)]
Fix markup so that 'make HISTORY' works. A couple very minor
editorial improvements.
Reorder 8.4 release note sections.
Re-add release notes for release 8.3.7.
Adjust AS OF release notes markup.
Mention release note items current as of 2009年03月16日.
First version of 8.4 release notes; markup/cleanup/reorganization still
required.
Tom Lane [2009年3月25日 23:20:01 +0000 (23:20 +0000)]
Add btree_gin contrib module.
Teodor Sigaev and Oleg Bartunov
Tom Lane [2009年3月25日 22:19:02 +0000 (22:19 +0000)]
Adjust the APIs for GIN opclass support functions to allow the extractQuery()
method to pass extra data to the consistent() and comparePartial() methods.
This is the core infrastructure needed to support the soon-to-appear
contrib/btree_gin module. The APIs are still upward compatible with the
definitions used in 8.3 and before, although *not* with the previous 8.4devel
function definitions.
catversion bump for changes in pg_proc entries (although these are just
cosmetic, since GIN doesn't actually look at the function signature before
calling it...)
Teodor Sigaev and Oleg Bartunov
Tom Lane [2009年3月25日 15:03:13 +0000 (15:03 +0000)]
Fix old thinko in pgp.h: the idea is to declare some named enum types,
not global variables of anonymous enum types. This didn't actually hurt
much because most linkers will just merge the duplicated definitions ...
but some will complain. Per bug #4731 from Ceriel Jacobs.
Backpatch to 8.1 --- the declarations don't exist before that.
Use double quotes instead of single quotes.
Complete list of valid fork names, and use double quotes.
Update line count for \? output (obtained from wc -l)
In the \? output, align the columns consistently.
Polish SQL/MED terminology
No period if it's not a sentence
Tom Lane [2009年3月24日 22:06:03 +0000 (22:06 +0000)]
Install a search tree depth limit in GIN bulk-insert operations, to prevent
them from degrading badly when the input is sorted or nearly so. In this
scenario the tree is unbalanced to the point of becoming a mere linked list,
so insertions become O(N^2). The easiest and most safely back-patchable
solution is to stop growing the tree sooner, ie limit the growth of N. We
might later consider a rebalancing tree algorithm, but it's not clear that
the benefit would be worth the cost and complexity. Per report from Sergey
Burladyan and an earlier complaint from Heikki.
Back-patch to 8.2; older versions didn't have GIN indexes.
Tom Lane [2009年3月24日 21:12:56 +0000 (21:12 +0000)]
Fix stupid parenthesization mistake. Per bug #4728 from Bruce Toll.
Tom Lane [2009年3月24日 20:17:18 +0000 (20:17 +0000)]
Implement "fastupdate" support for GIN indexes, in which we try to accumulate
multiple index entries in a holding area before adding them to the main index
structure. This helps because bulk insert is (usually) significantly faster
than retail insert for GIN.
This patch also removes GIN support for amgettuple-style index scans. The
API defined for amgettuple is difficult to support with fastupdate, and
the previously committed partial-match feature didn't really work with
it either. We might eventually figure a way to put back amgettuple
support, but it won't happen for 8.4.
catversion bumped because of change in GIN's pg_am entry, and because
the format of GIN indexes changed on-disk (there's a metapage now,
and possibly a pending list).
Teodor Sigaev
Fix my typo
Remove munging of xml and xpath params to xpath(). The XML must now be a well formed XML document.
Tom Lane [2009年3月23日 16:36:27 +0000 (16:36 +0000)]
Const-ify the parse table passed to fillRelOptions. The previous coding
meant it had to be built on-the-fly at each entry to default_reloptions.
Update constraint_exclusion docs because parameter is now 3-valued, not
just enable/disable.
Add expected file for locale-enabled installations.
Tom Lane [2009年3月23日 01:52:38 +0000 (01:52 +0000)]
More fixes for 8.4 DTrace probes. Remove useless BUFFER_HIT/BUFFER_MISS
probes --- the BUFFER_READ_DONE probe provides the same information and more
besides. Expand the LOCK_WAIT_START/DONE probe arguments so that there's
actually some chance of telling what is being waited for. Update and
clean up the documentation.
Clarify libpq 'sslverify' documentation wording.
Tom Lane [2009年3月22日 22:39:05 +0000 (22:39 +0000)]
Add isExtend to the parameters of the buffer_read_start and buffer_read_done
DTrace probes, so that ordinary reads can be distinguished from relation
extension operations. Move buffer_read_start probe to before the
smgrnblocks() call that's needed in the isExtend case, since really that step
should be charged as part of the time needed for the extension operation.
(This makes it slightly harder to match the read_start with the associated
read_done, since now you can't match them on blockNumber, but it should still
be possible since isExtend operations on the same relation can never be
interleaved.) Per recent discussion.
In passing, add the page identity (forkNum/blockNum) to the parameters of the
buffer_flush_start/buffer_flush_done probes, which were unaccountably lacking
the info.
Clarify 'cert' authentication documention.
Tom Lane [2009年3月22日 18:06:35 +0000 (18:06 +0000)]
Clean up pg_SSPI_error() coding a little bit: make the messages more
consistent, translate where intended, const-ify declarations.
Resolves a gripe from Alvaro as well as some stuff I didn't like.
Tom Lane [2009年3月22日 16:44:26 +0000 (16:44 +0000)]
Remove the -d and -D options of pg_dump and pg_dumpall. The functionality
is still available, but you must now write the long equivalent --inserts
or --column-inserts. This change is made to eliminate confusion with the
use of -d to specify a database name in most other Postgres client programs.
Original patch by Greg Mullane, modified per subsequent discussion.
Tom Lane [2009年3月22日 01:12:32 +0000 (01:12 +0000)]
Remove the datetime keywords ABSTIME and RELTIME, which we'd been treating as
noise words for the last twelve years, for compatibility with Berkeley-era
output formatting of the special INVALID values for those datatypes.
Considering that the datatypes themselves have been deprecated for awhile,
this is taking backwards compatibility a little far. Per gripe from Josh
Berkus.
Tom Lane [2009年3月21日 00:04:40 +0000 (00:04 +0000)]
Optimize multi-batch hash joins when the outer relation has a nonuniform
distribution, by creating a special fast path for the (first few) most common
values of the outer relation. Tuples having hashvalues matching the MCVs
are effectively forced to be in the first batch, so that we never write
them out to the batch temp files.
Bryce Cutt and Ramon Lawrence, with some editorialization by me.
Rename pg_restore -m to -j, and add documentation about what good numbers
are. per discussion on hackers
Don't set the signal handler for SIGQUIT on Windows. Buildfarm shows that
reinstalling the default signal handler doesn't work as it is on Windows.
Presumably core dumps on SIGQUIT are not a problem on Windows, so rather
than figure out what header files or other changes are required to make it
work, just don't bother.
Tom Lane [2009年3月18日 20:18:18 +0000 (20:18 +0000)]
Be more clear about when to use gist__int_ops vs. gist__intbig_ops.
Per suggestion from Ron Mayer.
Don't intercept SIGQUIT as a signal to trigger failover; that's what
postmaster uses for immediate shutdown. Trap SIGUSR1 as the preferred
signal for that.
Per report by Fujii Masao and subsequent discussion on -hackers.
Fix case of the just resurrected UCS_to_BIG5.pl script, and update
Makefile to use it.
Add seven kanji characters defined in the Windows 950 codepage to our
big5/win950 <-> UTF8 conversion tables.
Per report by Roger Chang.
Fix Windows-specific race condition in syslogger. This could've been
the cause of the "could not write to log file: Bad file descriptor"
errors reported at
http://archives.postgresql.org//pgsql-general/2008-06/msg00193.php
Backpatch to 8.3, the race condition was introduced by the CSV logging
patch.
Analysis and patch by Gurjeet Singh.
Tom Lane [2009年3月17日 21:36:21 +0000 (21:36 +0000)]
Add just-assigned CVE number to release notes for recent security issue.
Improve zero-year comments.
Document that datetime year '0' is considered in a recent century, not
just '00'.
Add -w option to --help output. It was apparently forgotten when -w was
implemented.
Tom Lane [2009年3月15日 22:05:17 +0000 (22:05 +0000)]
Fix contrib/hstore to throw an error for keys or values that don't fit in its
data structure, rather than silently truncating them. Andrew Gierth
Tom Lane [2009年3月15日 20:31:19 +0000 (20:31 +0000)]
Clean up the code for to_timestamp's conversion of year plus ISO day number
to date, as per bug #4702 and subsequent discussion. In particular, make it
work for years specified using AD/BC or CC fields, and fix the test for "no
year specified" so that it doesn't trigger inappropriately for 1 BC (which it
was doing even in code paths that had nothing to do with to_timestamp). I
also did some minor code beautification in the non-ISO-day-number code path.
This area has been busted all along, but because the code has been rewritten
repeatedly, it would be considerable trouble to back-patch. It's such a
corner case that it doesn't seem worth the effort.
Tom Lane [2009年3月13日 22:50:44 +0000 (22:50 +0000)]
Fix identify_locking_dependencies to reflect the fact that fix_dependencies
previously repointed TABLE dependencies to TABLE DATA. Mea culpa.
Tom Lane [2009年3月13日 17:46:21 +0000 (17:46 +0000)]
Restore previous ordering of BUFFER_FLUSH_START probe. I had wanted to
make it include the time for the possible smgropen() call, but that
results in a null pointer dereference :-(.
An alternative solution would be to fetch the buffer tag instead of
looking at *reln, but I'll just put it back as it was for the moment.
BTW, this indicates that DTrace probes evaluate their arguments even
when nominally inactive. What was that about "zero cost", again?
Tom Lane [2009年3月12日 22:35:48 +0000 (22:35 +0000)]
Update back-branch release notes.
Tom Lane [2009年3月12日 00:53:25 +0000 (00:53 +0000)]
Fix core dump due to null-pointer dereference in to_char() when datetime
format codes are misapplied to a numeric argument. (The code still produces
a pretty bogus error message in such cases, but I'll settle for stopping the
crash for now.) Per bug #4700 from Sergey Burladyan.
Problem exists in all supported branches, so patch all the way back.
In HEAD, also clean up some ugly coding in the nearby cache management
code.
Tom Lane [2009年3月11日 23:19:25 +0000 (23:19 +0000)]
Code review for dtrace probes added (so far) to 8.4. Adjust placement of
some bufmgr probes, take out redundant and memory-leak-inducing path arguments
to smgr__md__read__done and smgr__md__write__done, fix bogus attempt to
recalculate space used in sort__done, clean up formatting in places where
I'm not sure pgindent will do a nice job by itself.
Some languages have symbols with zero display's width or/and vowels/signs which
are not an alphabetic character although they are not word-breakers too.
So, treat them as part of word.
Per off-list discussion with Dibyendra Hyoju <dibyendra@gmail.com> and
and Bal Krishna Bal <balkrishna7bal@gmail.com> about Nepali language and
Devanagari alphabet.
This is the main PostgreSQL git repository.
RSS
Atom