On Thu, Aug 20, 2020 at 12:18 PM Masahiko Sawada <masahiko.saw...@2ndquadrant.com> wrote: > > On Thu, 20 Aug 2020 at 14:01, Amit Kapila <amit.kapil...@gmail.com> wrote: > > > > On Wed, Aug 19, 2020 at 12:54 PM Masahiko Sawada > > <masahiko.saw...@2ndquadrant.com> wrote: > > > > Here, we can notice that for the index, we are getting context > > information but not for the heap. The reason is that in > > vacuum_error_callback, we are not printing additional information for > > phases VACUUM_ERRCB_PHASE_SCAN_HEAP and VACUUM_ERRCB_PHASE_VACUUM_HEAP > > when block number is invalid. If we want to cover the 'info' messages > > then won't it be better if we print a message in those phases even > > block number is invalid (something like 'while scanning relation > > \"%s.%s\"") > > Yeah, there is an inconsistency. I agree to print the message even > when the block number is invalid. >
Okay, I will update this and send this patch and rebased patch to display offsets later today or tomorrow. -- With Regards, Amit Kapila.