release note item

Started by Andrew Dunstanover 13 years ago3 messages
#1Andrew Dunstan
andrew@dunslane.net

Regarding the item:

* Properly handle empty arrays returned by PL/Perl functions (Andrew
Dunstan) DETAILS?

This was a bug fix, not a feature, and in any case is due to Alex
Hunsaker, not me.

cheers

andrew

#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Andrew Dunstan (#1)
Re: release note item

Andrew Dunstan <andrew@dunslane.net> writes:

Regarding the item:
* Properly handle empty arrays returned by PL/Perl functions (Andrew
Dunstan) DETAILS?

This was a bug fix, not a feature, and in any case is due to Alex
Hunsaker, not me.

I assume this is in reference to these commits:

Author: Andrew Dunstan <andrew@dunslane.net>
Branch: REL9_1_STABLE Release: REL9_1_0 [22a55b3af] 2011-08-17 12:03:26 -0400

Properly handle empty arrays returned from plperl functions.

Bug reported by David Wheeler, fix by Alex Hunsaker.
# Please enter the commit message for your changes. Lines starting
# with '#' will be ignored, and an empty message aborts the commit.
# On branch master
# Changes to be committed:
# (use "git reset HEAD <file>..." to unstage)
#
# modified: src/pl/plperl/plperl.c
#
# Untracked files:
# (use "git add <file>..." to include in what will be committed)
#
# autom4te.cache/
# configure.in~
# doc/src/sgml/ref/grant.sgml~
# src/backend/port/win32_latch.c~
# src/bin/psql/command.c~
# src/include/pg_config.h.win32~
# src/pl/plpython/plpython.c~
# src/tools/msvc/pgbison.bat~
# src/tools/msvc/pgbison.pl.bak
# src/tools/msvc/pgflex.bat~
# src/tools/msvc/pgflex.pl.bak
# src/tools/pgindent/README~
# src/tools/pgindent/pgindent.pl
# src/tools/pgindent/pgindent.pl~
# xxxxx
# yyyyyy

Author: Andrew Dunstan <andrew@dunslane.net>
Branch: master [68c903a66] 2011-08-17 11:59:18 -0400

Properly handle empty arrays returned from plperl functions.

Bug reported by David Wheeler, fix by Alex Hunsaker.

git_changelog failed to merge them because of the different commit
messages, which is probably why Bruce thought it was a HEAD-only
commit. AFAICS we should just remove this item, since it was (or
should've been) documented already in 9.1 release notes.

regards, tom lane

#3Bruce Momjian
bruce@momjian.us
In reply to: Tom Lane (#2)
Re: release note item

On Mon, May 21, 2012 at 12:19:27PM -0400, Tom Lane wrote:

Andrew Dunstan <andrew@dunslane.net> writes:

Regarding the item:
* Properly handle empty arrays returned by PL/Perl functions (Andrew
Dunstan) DETAILS?

This was a bug fix, not a feature, and in any case is due to Alex
Hunsaker, not me.

I assume this is in reference to these commits:

Thanks, removed from 9.2 release notes.

---------------------------------------------------------------------------

Author: Andrew Dunstan <andrew@dunslane.net>
Branch: REL9_1_STABLE Release: REL9_1_0 [22a55b3af] 2011-08-17 12:03:26 -0400

Properly handle empty arrays returned from plperl functions.

Bug reported by David Wheeler, fix by Alex Hunsaker.
# Please enter the commit message for your changes. Lines starting
# with '#' will be ignored, and an empty message aborts the commit.
# On branch master
# Changes to be committed:
# (use "git reset HEAD <file>..." to unstage)
#
# modified: src/pl/plperl/plperl.c
#
# Untracked files:
# (use "git add <file>..." to include in what will be committed)
#
# autom4te.cache/
# configure.in~
# doc/src/sgml/ref/grant.sgml~
# src/backend/port/win32_latch.c~
# src/bin/psql/command.c~
# src/include/pg_config.h.win32~
# src/pl/plpython/plpython.c~
# src/tools/msvc/pgbison.bat~
# src/tools/msvc/pgbison.pl.bak
# src/tools/msvc/pgflex.bat~
# src/tools/msvc/pgflex.pl.bak
# src/tools/pgindent/README~
# src/tools/pgindent/pgindent.pl
# src/tools/pgindent/pgindent.pl~
# xxxxx
# yyyyyy

Author: Andrew Dunstan <andrew@dunslane.net>
Branch: master [68c903a66] 2011-08-17 11:59:18 -0400

Properly handle empty arrays returned from plperl functions.

Bug reported by David Wheeler, fix by Alex Hunsaker.

git_changelog failed to merge them because of the different commit
messages, which is probably why Bruce thought it was a HEAD-only
commit. AFAICS we should just remove this item, since it was (or
should've been) documented already in 9.1 release notes.

regards, tom lane

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ It's impossible for everything to be true. +