Hi, Osumi-san, Thank you for your comment.
On 4/11/22 15:40, Osumi, Takamichi/大墨 昂道 wrote:
> (1) Name of isRelNodeCollision
changed.
> (2) Remove curly brackets for one sentence
changed.
> (3) variables of isRelNodeCollision
changed.
> (4) Having Assert in isRelNodeCollision
added.
> (5) argument name of isRelNodeCollision
changed.
BTW, I also change the parameter from null to SnapshotAny
scandesc = systable_beginscan(pg_class,
ClassTblspcRelfilenodeIndexId,
- true, NULL, 2, skey);
+ true, SnapshotAny, 2, skey);
I'm not sure, but I think SnapshotAny is more suitable after reading the
source(comment) in function GetNewOidWithIndex().
Please see the attachment
Regards,
Shenhao Wang