PostgreSQL 9.1.1 commit log

Stamp 9.1.1.

commit   : 8da4007a4d78168243d0e2ce67d97073ce2812ce    
  
author   : Tom Lane <[email protected]>    
date     : Thu, 22 Sep 2011 17:57:57 -0400    
  
committer: Tom Lane <[email protected]>    
date     : Thu, 22 Sep 2011 17:57:57 -0400    

Click here for diff

M configure
M configure.in
M doc/bug.template
M src/include/pg_config.h.win32
M src/interfaces/libpq/libpq.rc.in
M src/port/win32ver.rc

Update release notes for 9.1.1, 9.0.5, 8.4.9, 8.3.16, 8.2.22.

commit   : 7f70f35031b4dea40ab4fa20638befc430e8ebaa    
  
author   : Tom Lane <[email protected]>    
date     : Thu, 22 Sep 2011 17:39:19 -0400    
  
committer: Tom Lane <[email protected]>    
date     : Thu, 22 Sep 2011 17:39:19 -0400    

Click here for diff

Man, we fixed a lotta bugs since April.  

M doc/src/sgml/release-8.2.sgml
M doc/src/sgml/release-8.3.sgml
M doc/src/sgml/release-8.4.sgml
M doc/src/sgml/release-9.0.sgml
M doc/src/sgml/release-9.1.sgml

Translation updates

commit   : f992679318ceea10f785e8653932f04b4003eef0    
  
author   : Peter Eisentraut <[email protected]>    
date     : Thu, 22 Sep 2011 23:24:25 +0300    
  
committer: Peter Eisentraut <[email protected]>    
date     : Thu, 22 Sep 2011 23:24:25 +0300    

Click here for diff

M src/backend/nls.mk
M src/backend/po/de.po
M src/backend/po/es.po
M src/backend/po/fr.po
A src/backend/po/pl.po
M src/bin/initdb/po/pl.po
M src/bin/psql/po/pl.po
M src/interfaces/libpq/po/pl.po

Fix another bit of unlogged-table-induced breakage.

commit   : c67bba992533acb1df0fd2f5c5164da2ac9543ae    
  
author   : Robert Haas <[email protected]>    
date     : Wed, 21 Sep 2011 10:47:47 -0400    
  
committer: Robert Haas <[email protected]>    
date     : Wed, 21 Sep 2011 10:47:47 -0400    

Click here for diff

Per bug #6205, reported by Abel Abraham Camarillo Ojeda.  This isn't a  
particularly elegant fix, but I'm trying to minimize the chances of  
causing yet another round of breakage.  
  
Adjust regression tests to exercise this case.  

M src/backend/catalog/namespace.c
M src/test/regress/expected/create_table.out
M src/test/regress/sql/create_table.sql

Suppress "unused function" warning when not HAVE_LOCALE_T.

commit   : c47e62949840f670402cb470ce267e9ea2431992    
  
author   : Tom Lane <[email protected]>    
date     : Tue, 20 Sep 2011 17:47:21 -0400    
  
committer: Tom Lane <[email protected]>    
date     : Tue, 20 Sep 2011 17:47:21 -0400    

Click here for diff

Forgot to consider this case ...  

M src/backend/utils/adt/pg_locale.c

Improve reporting of newlocale() failures in CREATE COLLATION.

commit   : 226bc87c2da98ebd9ac08d1677961cb5e7ea4e5a    
  
author   : Tom Lane <[email protected]>    
date     : Tue, 20 Sep 2011 13:23:40 -0400    
  
committer: Tom Lane <[email protected]>    
date     : Tue, 20 Sep 2011 13:23:40 -0400    

Click here for diff

The standardized errno code for "no such locale" failures is ENOENT, which  
we were just reporting at face value, viz "No such file or directory".  
Per gripe from Thom Brown, this might confuse users, so add an errdetail  
message to clarify what it means.  Also, report newlocale() failures as  
ERRCODE_INVALID_PARAMETER_VALUE rather than using  
errcode_for_file_access(), since newlocale()'s errno values aren't  
necessarily tied directly to file access failures.  

M src/backend/utils/adt/pg_locale.c

Avoid unnecessary page-level SSI lock check in heap_insert().

commit   : e5b86c93b66b5caeb25a65bae26b04fc7a068957    
  
author   : Tom Lane <[email protected]>    
date     : Fri, 16 Sep 2011 14:47:27 -0400    
  
committer: Tom Lane <[email protected]>    
date     : Fri, 16 Sep 2011 14:47:27 -0400    

Click here for diff

As observed by Heikki, we need not conflict on heap page locks during an  
insert; heap page locks are only aggregated tuple locks, they don't imply  
locking "gaps" as index page locks do.  So we can avoid some unnecessary  
conflicts, and also do the SSI check while not holding exclusive lock on  
the target buffer.  
  
Kevin Grittner, reviewed by Jeff Davis.  Back-patch to 9.1.  

M src/backend/access/heap/heapam.c

gistendscan() forgot to free so->giststate.

commit   : 57d3dc2035fe8267e877620eddc3823360c7878f    
  
author   : Tom Lane <[email protected]>    
date     : Fri, 16 Sep 2011 04:27:56 -0400    
  
committer: Tom Lane <[email protected]>    
date     : Fri, 16 Sep 2011 04:27:56 -0400    

Click here for diff

This oversight led to a massive memory leak --- upwards of 10KB per tuple  
--- during creation-time verification of an exclusion constraint based on a  
GIST index.  In most other scenarios it'd just be a leak of 10KB that would  
be recovered at end of query, so not too significant; though perhaps the  
leak would be noticeable in a situation where a GIST index was being used  
in a nestloop inner indexscan.  In any case, it's a real leak of long  
standing, so patch all supported branches.  Per report from Harald Fuchs.  

M src/backend/access/gist/gistscan.c

Teach the makefile used to build stand-alone libpq on Windows that libpq needs win32setlocale.c now. The cygwin and MSVC build scripts were changed earlier, but this was neglected. This should fix bug report #6203 by Steve.

commit   : f6950429da2f68de7f7c28db8441625215b40463    
  
author   : Heikki Linnakangas <[email protected]>    
date     : Wed, 14 Sep 2011 17:57:32 +0300    
  
committer: Heikki Linnakangas <[email protected]>    
date     : Wed, 14 Sep 2011 17:57:32 +0300    

Click here for diff

M src/interfaces/libpq/win32.mak

In the manual section on primary_conninfo, recommend using a role with REPLICATION privileges, not SUPERUSER.

commit   : 73341353107e712df052a1e29fb5b70c47f11c9c    
  
author   : Heikki Linnakangas <[email protected]>    
date     : Wed, 14 Sep 2011 09:30:32 +0300    
  
committer: Heikki Linnakangas <[email protected]>    
date     : Wed, 14 Sep 2011 09:30:32 +0300    

Click here for diff

Fujii Masao  

M doc/src/sgml/recovery-config.sgml

deflist_to_tuplestore dumped core on an option with no value.

commit   : 663bee0d5c1e3ff2380377c7894499ae91eabd87    
  
author   : Tom Lane <[email protected]>    
date     : Tue, 13 Sep 2011 11:36:53 -0400    
  
committer: Tom Lane <[email protected]>    
date     : Tue, 13 Sep 2011 11:36:53 -0400    

Click here for diff

Make it return NULL for the option_value, instead.  
  
Per report from Frank van Vugt.  Back-patch to 8.4 where this code was  
added.  

M src/backend/foreign/foreign.c