[
https://issues.apache.org/jira/browse/SOLR-7660?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shalin Shekhar Mangar updated SOLR-7660:
----------------------------------------
Attachment: SOLR-7660.patch
# Removed zk exists call in getCollectionLive because it already catches
NoNodeException and returns null
# Added a comment in getIndividualColls as to why the zk exists call is
necessary in that method
> Avoid redundant 'exists' calls made to ZK while fetching cluster state updates
> ------------------------------------------------------------------------------
>
> Key: SOLR-7660
> URL: https://issues.apache.org/jira/browse/SOLR-7660
> Project: Solr
> Issue Type: Improvement
> Components: SolrCloud
> Reporter: Shalin Shekhar Mangar
> Assignee: Shalin Shekhar Mangar
> Priority: Minor
> Labels: difficulty-easy, impact-medium
> Fix For: 5.3, Trunk
>
> Attachments: SOLR-7660.patch
>
>
> As noted in
> https://issues.apache.org/jira/browse/SOLR-7636?focusedCommentId=14578332&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14578332
> {quote}
> we have CollectionsHandler -> ZK getChildren (/live_nodes) + ZK read
> (/clusterstate.json) + ZK getChildren + N * ZK exists (for
> getIndividualColls) + W * ZK exists + W * getData (for watched collections) +
> (N-W) * ZK exists + (N-W) * ZK getData (for non-watched collections).
> {quote}
> Some of the ZK exists calls can be avoided without changing the outcome.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
