On Wed, Jul 12, 2023 at 7:08 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Wed, Jul 12, 2023 at 12:31 PM Masahiko Sawada <sawada.m...@gmail.com> > wrote: > > > > On Tue, Jul 11, 2023 at 5:31 PM Peter Smith <smithpb2...@gmail.com> wrote: > > > > > > > I don't think we have concluded any action for it. I agree that > > IsIndexOnlyOnExpression() is redundant. We don't need to check *all* > > index fields actually. I've attached a draft patch. It removes > > IsIndexOnlyOnExpression() and merges > > RemoteRelContainsLeftMostColumnOnIdx() to > > FindUsableIndexForReplicaIdentityFull. One concern is that we no > > longer do the assertion check with > > IsIndexUsableForReplicaIdentityFull(). What do you think? > > > > I think this is a valid concern. Can't we move all the checks > (including the remote attrs check) inside > IsIndexUsableForReplicaIdentityFull() and then call it from both > places? Won't we have attrmap information available in the callers of > FindReplTupleInLocalRel() via ApplyExecutionData?
You mean to pass ApplyExecutionData or attrmap down to RelationFindReplTupleByIndex()? I think it would be better to call it from FindReplTupleInLocalRel() instead. Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com