Hi Amit, Sorry, I was not clear. The code is fine, I only meant the comment.
update_exists also passes NULL, but a search did happen before it. So "NULL if no search was done" is not quite right. Fixing get_tuple_desc() later is fine, let's do it separately. 0002 looks good. Maybe add a hint to the new error in lock_conflictlog_relation(), like the two in subscriptioncmds.c. Thanks, Shihao
