== PostgreSQL Weekly News - November 30 2008 ==
JPUG's Let's PostgreSQL site is now open.
http://lets.postgresql.jp/
Drafts of a Russian translation of the Introduction and Tutorial
sections of the PostgreSQL manual are at:
http://postgresql.ru.net/manual/index.html
== PostgreSQL Product News ==
Archiveopteryx 2.13 and 3.0.4 released.
http://www.archiveopteryx.org/2.13
http://www.archiveopteryx.org/3.0.4
PostgreSQL has new JDBC drivers.
http://jdbc.postgresql.org/changes.html
Slony 2.0.0 released.
http://slony.info/
== PostgreSQL Jobs for November ==
http://archives.postgresql.org/pgsql-jobs/2008-11/threads.php
== PostgreSQL Local ==
David Fetter will be speaking at OSDC Sydney December 3-5.
http://www.osdc.com.au/2008/index.html
FOSDEM 2009 will take place February 7-8 2009 in Brussels, Belgium.
PostgreSQL will share a booth with the BSD groups and have a developer
room. Submit your 20-minute, 45 minute or lightning talks to
fosdem (at) postgresql (dot) eu.
PGCon Brazil will be take place October 23-24 2009 at Unicamp in
Campinas, Sao Paulo state.
== PostgreSQL in the News ==
Planet PostgreSQL: http://planet.postgresql.org/
PostgreSQL Weekly News is brought to you this week by David Fetter and
Devrim GUNDUZ.
Submit news and announcements by Sunday at 3:00pm Pacific time.
Please send English language ones to david@fetter.org, German language
to pwn@pgug.de, Italian language to pwn@itpug.org.
== Applied Patches ==
Magnus Hagander committed:
- Add support for matching wildcard server certificates to the new SSL
code. This uses the function fnmatch() which is not available on
all platforms (notably Windows), so import the implementation from
NetBSD into src/port.
- In pgsql/src/interfaces/libpq/fe-secure.c, disable FNM_CASEFOLD.
Need a proper solution later, but just comment it out for now so the
buildfarm recovers.
- Support regular expressions in pg_ident.conf.
Peter Eisentraut committed:
- CLUSTER VERBOSE and corresponding clusterdb --verbose option. Jim
Cox and Peter Eisentraut
- Move FAQ_AIX information to installation instructions. The
information on why the shared libraries are built the way they are
was not relevant to end users and has been made a mailing list
archive link in Makefile.shlib.
- In pgsql/src/test/regress/pg_regress.c, if pg_ctl stop fails, print
an error message and exit (instead of going out of our way to ignore
the exit status). If this ever really happens, worst case you will
not see the "score" at the bottom of the test run.
- Add %expect 0 to all parser input files to prevent conflicts
slipping by.
- In pgsql/src/backend/catalog/sql_features.txt, feature F442 "Mixed
column references in set functions" is supported.
- configure check for docbook2man program, used in the new XML-based
man page build target. This covers from-source, Debian, and Fedora
installation variants.
- In pgsql/src/backend/catalog/sql_features.txt, mark features related
to WITH/SELECT as supported.
- In pgsql/src/backend/catalog/sql_features.txt, drop CLI related
features from the list, since we don't track the ODBC business in
core.
- In pgsql/src/backend/catalog/sql_features.txt, add that PL/Java is
an implementation of SQL/JRT, and drop claim that PL/pgSQL has much
to do with SQL/PSM.
- Reduce risk of accidentally running temp-install regression tests
against a mismatching installation. Pick a default port number
calculated from the version number, and try a few times with other
numbers if that one doesn't work. Check if we can connect to the
port before starting our own postmaster, to detect some other server
already running there. To simplify the code, drop --temp-port
option and use --port for both temp-install and pre-installed case.
Tom Lane committed:
- In pgsql/src/port/Makefile, omit src/port/pipe.c on non-Windows
platforms. It's useless and draws complaints about empty object
files on some platforms, eg Darwin.
- Remove PGINTERVALSTYLE from the set of special environment variables
for libpq. As noted by Peter, adding this variable created a risk
of unexpected connection failures when talking to older server
versions, and since it doesn't do anything you can't do with
PGOPTIONS, it doesn't seem really necessary. Removing it does
occasion a few extra lines in pg_regress.c, but saving a getenv()
call per libpq connection attempt is perhaps worth that anyway.
- In pgsql/doc/src/sgml/information_schema.sgml,
information_schema.key_column_usage.position_in_unique_constraint
was misdocumented as not being implemented. In reality it has
worked since the release of 8.2.
- In psql, adjust the behavior of the PQExpBuffer code to make it have
well-defined results (ie, an empty "broken" buffer) if memory
overrun occurs anywhere along the way to filling the buffer. The
previous coding would just silently discard portions of the intended
buffer contents, as exhibited in trouble report from Sam Mason.
Also, tweak psql's main loop to correctly detect and report such
overruns. There's probably much more that should be done in this
line, but this is a start.
- Make DISCARD ALL discard advisory locks, too. Marko Kreen
- In pgsql/src/test/regress/pg_regress.c, let's see if using -c
instead of -- style fixes the PGOPTIONS issue.
- In pgsql/src/interfaces/libpq/pqexpbuffer.h, tweak new
PQExpBufferBroken macro to suppress warnings from pickier versions
of gcc.
- In pgsql/src/backend/executor/functions.c, fix minor memory leak
introduced in recent SQL-functions hacking: the DestReceiver created
during postquel_start needs to be destroyed during postquel_end. In
a moment of brain fade I had assumed this would be taken care of by
FreeQueryDesc, but it's not (and shouldn't be).
- In pgsql/doc/src/sgml/ref/discard.sgml, tweak wording of DISCARD ALL
description to avoid giving the impression that the presented list
of equivalent operations is meant to be the primary definition of
what it does. Per comment from Guillaume Smet.
- Back-patch change to make DISCARD ALL release advisory locks. Per
discussion.
- Cosmetic cleanups in contrib/isn (doesn't fix the passbyval problem)
- In pgsql/src/backend/optimizer/path/joinrels.c, my recent fix for
semijoin planning didn't actually work for a semijoin with a RHS
that can't be unique-ified --- join_is_legal has to check that
before deciding to build a join, else we'll have an unimplementable
joinrel. Per report from Greg Stark.
- Fix an oversight in uninstall_isn.sql: must drop operator families
explicitly.
- Partial fix for fallout from temp-port changes. ecpg still needs
more work, but I think this is enough to turn the buildfarm green
again.
- Make sure we give an appropriate user-facing error when attempting
to drop a table that is referenced by an open cursor. Fix unstable
ecpg regression test result that was produced by this oversight.
- Fix recently-added array_agg tests to ensure they produce stable
results regardless of plan changes. Per intermittent buildfarm
failures on "pigeon" and others.
- In pgsql/src/backend/utils/fmgr/funcapi.c, remove inappropriate
memory context switch in shutdown_MultiFuncCall(). This was a
thinko introduced in a patch from last February; it results in
memory leakage if an SRF is shut down before the actual end of
query, because subsequent code will be running in a longer-lived
context than it's expecting to be.
- Add a "LIKE = typename" clause to CREATE TYPE for base types. This
allows the basic representational details (typlen, typalign,
typbyval, typstorage) to be copied from an existing type rather than
listed explicitly in the CREATE TYPE command. The immediate reason
for this is to provide a simple solution for add-on modules that
want to define types represented as int8, float4, or float8: as of
8.4 the appropriate PASSEDBYVALUE setting is platform-specific and
so it's hard for a SQL script to know what to do. This patch fixes
the contrib/isn breakage reported by Rushabh Lathia.
- Clean up the API for DestReceiver objects by eliminating the
assumption that a Portal is a useful and sufficient additional
argument for CreateDestReceiver --- it just isn't, in most cases.
Instead formalize the approach of passing any needed parameters to
the receiver separately. One unexpected benefit of this change is
that we can declare typedef Portal in a less surprising location.
This patch is just code rearrangement and doesn't change any
functionality. I'll tackle the HOLD-cursor-vs-toast problem in a
follow-on patch.
- Fix dblink and tablefunc to not return with the wrong
CurrentMemoryContext. Per buildfarm results.
- In pgsql/contrib/tablefunc/tablefunc.c, refactor crosstab() to build
and return a tuplestore instead of using value-per-call mode. This
should be more efficient in normal usage, but the real problem with
the prior coding was that it returned with a SPI call still active.
That could cause problems if execution was interleaved with anything
else that might use SPI.
Alvaro Herrera committed:
- Use ResourceOwners in the snapshot manager, instead of attempting to
track them by hand. As an added bonus, the new code is smaller and
more understandable, and the ugly loops are gone. This had been
discussed all along but never implemented. It became clear that it
really needed to be fixed after a bug report by Pavan Deolasee.
Michael Meskes committed:
- In ecpg, when creating a varchar struct name, braces must be
discarded.
- In pgsql/src/interfaces/ecpg/preproc/type.c, removed strchrnul.
- In pgsql/src/interfaces/ecpg/preproc/type.c, replaced strchrnul by
strchr.
- Applied patch by Ron Mayer to merge the new interval style into
ecpg.
- In pgsql/src/interfaces/ecpg/pgtypeslib/interval.c, interval.c needs
limits.h.
Heikki Linnakangas committed:
- Rely on relcache invalidation to update the cached size of the FSM.
- In pgsql/src/backend/storage/buffer/localbuf.c, fix #ifdeffed
debugging code to work with relation forks.
- In pgsql/src/backend/storage/freespace/freespace.c, don't force
creation of the FSM on searches. It will still be created as soon
as the first page fills up, and is marked as (almost) full, though.
- In pgsql/src/backend/catalog/heap.c, fix obsolete comment regarding
FSM truncation.
- In pgsql/src/backend/tsearch/ts_typanalyze.c, fix bug in the
tsvector stats collection function, which caused a crash if the
sample contains just a one tsvector, containing only one lexeme.
== Rejected Patches (for now) ==
No one was disappointed this week :-)
== Pending Patches ==
Hitoshi Harada and Heikki Linnakangas sent in a flock of patches for
windowing functions.
Ramon Lawrence sent in another revision of his patch to improve the
performance of multi-batch hash joins for skewed data sets.
KaiGai Kohei sent in another revision of his SE-PostgreSQL patches.
Peter Eisentraut sent in three revisions of a patch to make the
regression test setup less brittle.
Magnus Hagander sent in another patch to fix LC_TIME on Windows.
Peter Eisentraut sent in and withdrew another revision of the distinct
types patch including work from Jeff Davis.
Alvaro Herrera sent in a patch to silence a snapshot warning.
Pavel Stehule sent in another revision of his patch to implement
default function parameters.
Koichi Suzuki sent in another revision of his pg_readahead patch.
Heikki Linnakangas sent in another revision of his visibility map
patch.
Fujii Masao sent in another revision of his synchronous replication
patch.
Magnus Hagander sent in another revision of his SSL wildcard-matching
patch.
