Re: Proof of concept: auto updatable views [Review of Patch]

From: Thom Brown <thom(at)linux(dot)com>
To: Amit Kapila <amit(dot)kapila(at)huawei(dot)com>
Cc: Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>, robertmhaas(at)gmail(dot)com, tgl(at)sss(dot)pgh(dot)pa(dot)us, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Proof of concept: auto updatable views [Review of Patch]
Date: 2012-10-10 16:08:47
Message-ID: CAA-aLv6Aq1-oQTpO6gwfa4JK761JH3-b=VcQuEhRZSPajGsyVQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 24 September 2012 12:10, Amit Kapila <amit(dot)kapila(at)huawei(dot)com> wrote:
> On Sunday, September 23, 2012 12:33 AM Dean Rasheed wrote:
>> On 18 September 2012 14:23, Amit kapila <amit(dot)kapila(at)huawei(dot)com> wrote:
>> > Please find the review of the patch.
>> >
>>
>> Thanks for the review. Attached is an updated patch, and I've include
>> some responses to specific review comments below.
>
> I have verified your updated patch. It works fine and according to me it is
> ready for committer to check this patch.
> I have updated in CF page as Ready for Committer.

This patch will need adjusting to account for the fact that OID 3172
is now assigned to lo_truncate64 as of 3 days ago.

--
Thom

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Kohei KaiGai 2012-10-10 16:09:56 Re: [bugfix] sepgsql didn't follow the latest core API changes
Previous Message Bruce Momjian 2012-10-10 15:56:58 Re: pg_upgrade not detecting version properly