On Thu, Jul 14, 2022 at 5:18 PM Dilip Kumar <dilipbal...@gmail.com> wrote:
> Apart from this I have fixed all the pending issues that includes > > - Change existing macros to inline functions done in 0001. > - Change pg_class index from (tbspc, relfilenode) to relfilenode and > also change RelidByRelfilenumber(). In RelidByRelfilenumber I have > changed the hash to maintain based on just the relfilenumber but we > still need to pass the tablespace to identify whether it is a shared > relation or not. If we want we can make it bool but I don't think > that is really needed here. > - Changed logic of GetNewRelFileNumber() based on what Robert > described, and instead of tracking the pending logged relnumbercount > now I am tracking last loggedRelNumber, which help little bit in > SetNextRelFileNumber in making code cleaner, but otherwise it doesn't > make much difference. > - Some new asserts in buf_internal inline function to validate value > of computed/input relfilenumber. I was doing some more testing by setting the FirstNormalRelFileNumber to a high value(more than 32 bits) I have noticed a couple of problems there e.g. relpath is still using OIDCHARS macro which says max relfilenumber file name can be only 10 character long which is no longer true. So there we need to change this value to 20 and also need to carefully rename the macros and other variable names used for this purpose. Similarly there was some issue in macro in buf_internal.h while fetching the relfilenumber. So I will relook into all those issues and repost the patch soon. -- Regards, Dilip Kumar EnterpriseDB: http://www.enterprisedb.com
