On Mon, Jan 21, 2019 at 10:39 PM Nishant, Fnu <nisha...@amazon.com> wrote:
>
> Thanks Amit for your review.
>
> On 1/20/19, 6:55 AM, "Amit Kapila" <amit.kapil...@gmail.com> wrote:
>     > I think you need to change below code as well....
>        Assert(buffer2 == InvalidBuffer || buffer1 <= buffer2);
>
> Done. Updated the patch.
>

Attached is an updated patch.  I have edited comments and commit
message in the patch.  I would like to backpatch this till 9.4 unless
anyone thinks otherwise.

BTW, do you have a reproducible test case for this fix?  How did you
catch this, browsing code?

How do we pronounce your name, is Nishant Fnu okay?  I would like to
mention you as the author of the patch, so need to write your name.

-- 
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com

Attachment: lock-order.1.patch
Description: Binary data

Reply via email to