Re: Package namespace and Safe init cleanup for plperl UPDATE 3 [PATCH]

On Mon, Feb 1, 2010 at 03:58, Tim Bunce <Tim.Bunce@pobox.com> wrote:
> On Sat, Jan 30, 2010 at 06:38:59PM -0700, Alex Hunsaker wrote:
>
>> plc_safe_ok.pl seems to loose its CVS $PostgreSQL$ keyword.
>Probably a slip-up when I merged the changes from HEAD up through my
>chain of branches.

Can you send an updated patch?  I think Andrew will probably fix it up
anyway but better safe than sorry.

>> That being said *im* ok with it.  Its useful from a debug standpoint.
>
> Yes. And, as I mentioned previously, I expect people like myself, David
> Wheeler, and others to experiment with the undocumented functionality
> and define and document a good API to it for the 9.1 release.

Huh, I missed that.

> I'd much rather get this change in than shoot for a larger change that
> doesn't get committed due to long-running discussions.  (Which seems
> more likely as Andrew's going to be less available for the rest of the
> commitfest.)

Plus its hard to get people to agree on anything GUCy (my new favorite
pun) thats not well thought out and tested.

Anyway yes I agree, but I thought I should at least raise it for
discussion. You'll notice the patch has been marked "Ready for
Commiter" this whole time. =)