On Thu, Aug 12, 2021 at 12:15 PM Drouvot, Bertrand <bdrou...@amazon.com> wrote: > > On 8/10/21 1:59 PM, Drouvot, Bertrand wrote: > > Hi Amit, > > > > The first version of the patch contained a change in > ReorderBufferToastReplace() (to put the call to > RelationIsValid(toast_rel) and display the error message when it is not > valid before the call to ReorderBufferChangeMemoryUpdate()). > > That way we also avoid the failed assertion described in the first > message of this thread (but would report the error message instead). > > Forgot to mention that I did not add this change in the new patch > version >
I think that is the right decision. -- With Regards, Amit Kapila.
