On Tue, Mar 31, 2020 at 8:53 AM Justin Pryzby <pry...@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
