Hi, > I think it's unnecessary to open the table. IIUC, the main goal is to avoid > the > cache lookup error in CacheInvalidateRelcacheByRelid(), and the v1 patch is > sufficient for that since it uses CacheInvalidateRelcacheByTuple(). The same > strategy is also used in RemoveRoleFromObjectPolicy() to handle concurrent > table > drops.
Oh, right, CacheInvalidateRelcacheByTuple() does not require the additional cache search. My code tried to serialize these operations, but it may be overengineering. Best regards, Hayato Kuroda FUJITSU LIMITED
