ads' corner

== Angewandte Patches ==

Tom Lane committed:

- Add range checks to time_recv() and timetz_recv(), to prevent binary
   input of time values that would not be accepted via textual input.
   Per gripe from Andrew McNamara.  This is potentially a
   back-patchable bug fix, but for the moment it doesn't seem
   sufficiently high impact to justify doing that.

- Remove unused declarations of EncodeTimeOnly and DecodeTimeOnly.

- Allow the second argument of pg_get_expr() to be just zero when
   deparsing an expression that's not supposed to contain variables.
   Per discussion with Gevik Babakhani, this eliminates the need for an
   ugly kluge (namely, specifying some unrelated relation name).
   Remove one such kluge from pg_dump.

- Remove the useless and rather inconsistent return values of
   EncodeDateOnly, EncodeTimeOnly, EncodeDateTime, EncodeInterval.
   These don't have any good reason to fail, and their callers were
   mostly not checking anyway.

- Improve documentation about function volatility: mention the
   snapshot visibility effects in a couple of places where people are
   likely to look for it.  Per discussion of recent question from Karl
   Nack.

- Ignore RECHECK in CREATE OPERATOR CLASS, just throwing a NOTICE,
   instead of throwing an error as 8.4 had been doing.  The error
   interfered with porting old database definitions (particularly for
   pg_migrator) without really buying any safety.  Per bug #4817 and
   subsequent discussion.

- In pgsql/doc/src/sgml/release-8.4.sgml, improve release note
   explanation of the change in libpq's handling of default usernames
   versus Kerberos tickets.  Per confusion about what bug #4824 was
   really about.

- In pgsql/src/backend/catalog/index.c, update obsolete comment in
   index_drop().  When the comment was written, queries frequently took
   no lock at all on individual indexes.  That's not true any more, but
   we still need lock on the parent table to make it safe to use cached
   lists of index OIDs.

Bruce Momjian committed:

- In pgsql/doc/src/sgml/backup.sgml, remove tabs from SGML file.

- In pgsql/doc/src/sgml/libpq.sgml, document that forking while having
   open libpq connections is not advised.

Michael Meskes committed:

- In pgsql/src/interfaces/libpq/fe-exec.c, reverting patch just in
   case a compiler treats this enum as signed.

- Change macros to make gcc quiet when parsing.

Peter Eisentraut committed:

- Fix compiler warnings on Sun Studio of the sort '"tsquery_op.c",
   line 193: warning: syntax error: empty declaration.'  Zdenek Kotala.

- Fix to use the same format specifiers in both branches of a
   ngettext().  Zdenek Kotala.

Magnus Hagander committed:

- In pgsql/src/backend/libpq/auth.c, properly return the usermap
   result when doing gssapi authentication.  Without this, the username
   was in practice never matched against the kerberos principal used to
   log in.

Heikki Linnakangas committed:

- In pgsql/src/backend/access/transam/xlog.c, when archiving is
   enabled, rotate the last WAL segment at shutdown so that all
   transactions are archived.  Original patch by Guillaume Smet.

== Abgelehnte Patches (bis jetzt) ==

No one was disappointed this week :-)

== Eingesandte Patches ==