On Wed, Jul 27, 2022 at 11:39 PM Robert Haas <robertmhaas@gmail.com> wrote:
>
> On Wed, Jul 27, 2022 at 12:37 PM Dilip Kumar <dilipbalaut@gmail.com> wrote:
> > Just realised that this should have been BufferTagsEqual instead of BufferTagEqual
> >
> > I will modify this and send an updated patch tomorrow.
>
> I changed it and committed.
>
> What was formerly 0002 will need minor rebasing.
Thanks, I have rebased other patches, actually, there is a new 0001
patch now. It seems during renaming relnode related Oid to
RelFileNumber, some of the references were missed and in the last
patch set I kept it as part of main patch 0003, but I think it's
better to keep it separate. So took out those changes and created
0001, but you think this can be committed as part of 0003 only then
also it's fine with me.
I have done some cleanup in 0002 as well, basically, earlier we were
storing the result of the BufTagGetRelFileLocator() in a separate
variable which is not required everywhere. So wherever possible I
have avoided using the intermediate variable.
--
Regards,
Dilip Kumar
EnterpriseDB: http://www.enterprisedb.com