Hi Vignesh C,
> > Hmm, can you please elaborate more on this? The declaration > > and assignment are already on different lines. > > > > ps: pgindent changed this line a bit. Does that look better? > > I thought of changing it to something like below: > bool isUsableIndex; > Oid idxoid = lfirst_oid(lc); > Relation indexRelation = index_open(idxoid, AccessShareLock); > IndexInfo *indexInfo = BuildIndexInfo(indexRelation); > > isUsableIndex = IsIndexUsableForReplicaIdentityFull(indexInfo); > > Alright, this looks slightly better. I did a small change to your suggestion, basically kept *lfirst_oid * as the first statement in the loop. I'll attach the changes on v38 in the next e-mail. Thanks, Onder KALACI
