Re: [PATCH] two-arg current_setting() with fallback

Started by David Christensenover 10 years ago1 messages
#1David Christensen
david@endpoint.com

Date: Thu, 4 Jun 2015 13:55:52 +0530
From: Jeevan Chalke <jeevan.chalke@enterprisedb.com>
To: david.g.johnston@gmail.com
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Subject: Re: [PATCH] two-arg current_setting() with fallback
Message-ID: <CAM2+6=VsTg=pAEdq6ioKpoLx7yapE9cs3UvFbNV310Ur7UpqhQ@mail.gmail.com>

Hi,

Attached patch which fixes my review comments.

Since code changes were good, just fixed reported cosmetic changes.

David, can you please cross check?

Hi Jeevan,

I’m just on the digest list, and it looks like I didn’t get copied on your response, so missed it until now.

Thanks for the review; I reviewed the changes you made and it looks good.

Regards,

David
--
David Christensen
PostgreSQL Team Manager
End Point Corporation
david@endpoint.com
785-727-1171

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers