[
https://issues.apache.org/jira/browse/SOLR-5649?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13898324#comment-13898324
]
ASF subversion and git services commented on SOLR-5649:
-------------------------------------------------------
Commit 1567400 from [[email protected]] in branch 'dev/branches/branch_4x'
[ https://svn.apache.org/r1567400 ]
SOLR-5649: Clean up some minor ConnectionManager issues.
> Small ConnectionManager improvements
> ------------------------------------
>
> Key: SOLR-5649
> URL: https://issues.apache.org/jira/browse/SOLR-5649
> Project: Solr
> Issue Type: Bug
> Components: SolrCloud
> Affects Versions: 4.6.1
> Reporter: Gregory Chanan
> Assignee: Mark Miller
> Priority: Minor
> Fix For: 5.0, 4.7
>
> Attachments: SOLR-5649.patch
>
>
> I was just looking through the ConnectionManager and want to jot these down
> before I forget them. I'm happy to make a patch if someone thinks it's
> valuable as well.
> - "clientConnected" doesn't seem to be read, can be eliminated
> - "state" is a private volatile variable, but only used in one function --
> seems unlikely private volatile is what is wanted
> - A comment explaining why disconnected() is not called in the case of
> Expired would be helpful (Expired means we have already waited the timeout
> period so we want to reject updates right away)
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
