Re: [PATCH] two-arg current_setting() with fallback - Mailing list pgsql-hackers

From David Christensen
Subject Re: [PATCH] two-arg current_setting() with fallback
Date
Msg-id 035424DB-2622-4B5B-B4B7-88E2CE02C778@endpoint.com
Whole thread Raw
In response to [PATCH] two-arg current_setting() with fallback  (David Christensen <david@endpoint.com>)
List pgsql-hackers
> 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







pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: [GENERAL] 9.4.1 -> 9.4.2 problem: could not access status of transaction 1
Next
From: Alvaro Herrera
Date:
Subject: Re: [GENERAL] 9.4.1 -> 9.4.2 problem: could not access status of transaction 1