[
https://issues.apache.org/jira/browse/SOLR-8250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15119921#comment-15119921
]
ASF subversion and git services commented on SOLR-8250:
-------------------------------------------------------
Commit eb6bf9bc548b3469ccbd34a5dd1f02d3bb11f3a3 in lucene-solr's branch
refs/heads/master from jbernste
[ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=eb6bf9b ]
SOLR-8250: Implement ResultSetMetaDataImpl getColumnLabel(int column) and
getColumnDisplaySize(int column)
> remove member/variable shadowing in ElectionContext
> ---------------------------------------------------
>
> Key: SOLR-8250
> URL: https://issues.apache.org/jira/browse/SOLR-8250
> Project: Solr
> Issue Type: Wish
> Affects Versions: Trunk
> Reporter: Christine Poerschke
> Attachments: SOLR-8250.patch
>
>
> Was looking to change some logging in this file and stumbled across this.
> Attaching patch to illustrate and describing in words here also.
> #1 {{OverseerElectionContext.runLeaderProcess}} has a local variable {{id}}
> that shadows the {{id}} member in the {{ElectionContext}} base class.
> #2 So i went and renamed {{id}} to {{coreNodeName}} (the {{ElectionContext}}
> constructor uses that as its argument name already and thus {{this.id =
> coreNodeName;}} becomes {{this.coreNodeName = coreNodeName;}}, so far, so
> good.
> #3 Next then i noticed that the
> {{ShardLeaderElectionContext.startLeaderInitiatedRecoveryOnReplicas}} local
> variable {{coreNodeName}} is now shadowing {{ElectionContext}}'s coreNodeName
> member and that on the preceding line local variable shardId already shadows
> the {{shardId}} member in {{ShardLeaderElectionContextBase}} class.
> #4 So then to remove the shadowing i renamed the local variables giving
> {{cloudDescCollection}} (formerly coll), {{cloudDescShardId}} (formerly
> shardId) and {{cloudDescCoreNodeName}} (formerly coreNodeName) as the local
> variables with {{ShardLeaderElectionContextBase}}'s members still being
> called {{collection}} and {{shardId}} plus the {{coreNodeName}} member from
> the {{ElectionContext}} base class.
> #5 This then highlights when the local variables are used and when the (now
> no longer shadowed) class members are used and we have a mix with
> {code}
> - String znodePath =
> zkController.getLeaderInitiatedRecoveryZnodePath(coll, shardId);
> + String znodePath =
> zkController.getLeaderInitiatedRecoveryZnodePath(cloudDescCollection,
> cloudDescShardId);
> {code}
> and
> {code}
> - final Replica.State lirState =
> zkController.getLeaderInitiatedRecoveryState(coll, shardId,
> replicaCoreNodeName);
> + final Replica.State lirState =
> zkController.getLeaderInitiatedRecoveryState(cloudDescCollection,
> cloudDescShardId, replicaCoreNodeName);
> {code}
> using the local variables but
> {code}
> List<ZkCoreNodeProps> replicaProps =
> - zkController.getZkStateReader().getReplicaProps(collection,
> shardId, coreNodeName);
> + zkController.getZkStateReader().getReplicaProps(collection,
> cloudDescShardId, cloudDescCoreNodeName);
> {code}
> and
> {code}
> zkController.ensureReplicaInLeaderInitiatedRecovery(cc,
> - collection, shardId, coreNodeProps,
> core.getCoreDescriptor(),
> + collection, cloudDescShardId, coreNodeProps,
> core.getCoreDescriptor(),
> false /* forcePublishState */);
> {code}
> using the collection member but the shard id and core node name local
> variables.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
