On Thu, Mar 13, 2014 at 8:06 PM, Amit Kapila <amit.kapil...@gmail.com> wrote: > On Thu, Mar 13, 2014 at 7:10 PM, Robert Haas <robertmh...@gmail.com> wrote: >> On Thu, Mar 13, 2014 at 12:45 AM, Amit Kapila <amit.kapil...@gmail.com> >> wrote: >>> _bt_doinsert - "insert index tuple (X,Y)" (here it will refer to index tuple >>> location) >> >> I don't think that giving the index tuple location is going to be very >> helpful; can we get the TID for conflicting heap tuple? > > Yes, each index tuple contains reference to TID of heap tuple.
I have updated the patch to pass TID and operation information in error context and changed some of the comments in code. Let me know if the added operation information is useful, else we can use better generic message in context. Christian, can you please once confirm if the attached patch looks okay to you? According to me all the comments raised till now for this patch are addressed, so I will mark it as Ready For Committer unless some one feels we should do something else for this patch. With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com
Description: Binary data
-- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers