Hi Mikhail

Any thoughts on moving the 'idxRel->rd_index->indisvalid' check into 
'IsIndexUsableForReplicaIdentityFull()' itself?
As I understand it, an index MUST be valid to be usable for REPLICA IDENTITY 
FULL lookups, so it seems more natural to have that check live alongside the 
other usability checks in `IsIndexUsableForReplicaIdentityFull()`, rather than 
in the caller.

Kind regards,
Miłosz Bieniek  



Reply via email to