On Tue, Mar 31, 2020 at 8:53 AM Justin Pryzby <pryzby@telsasoft.com> wrote:
>
> On Tue, Mar 31, 2020 at 07:50:45AM +0530, Amit Kapila wrote:
> > One thing I have noticed is that there is some saving by using
> > vacrelstats->relnamespace as that avoids sys cache lookup. OTOH,
> > using vacrelstats->relname doesn't save much, but maybe for the sake
> > of consistency, we can use it.
>
> Mostly I wrote that to avoid repeatedly calling functions/macro with long name.
> I consider it a minor cleanup. I think we should put them to use. The
> LVRelStats describes them as not being specifically for the error context.
>
Pushed. I think we are done here. The patch is marked as committed in
CF. Thank you!
--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com