Trym Møller created SOLR-4066:
---------------------------------
Summary: SolrZKClient changed interface
Key: SOLR-4066
URL: https://issues.apache.org/jira/browse/SOLR-4066
Project: Solr
Issue Type: Bug
Components: SolrCloud
Affects Versions: 4.0, 4.0.1, 4.1
Environment: Any
Reporter: Trym Møller
Priority: Minor
The constructor of SolrZKClient has changed, I expect to ensure clean up of
resources. The strategy is as follows:
connManager = new ConnectionManager(...)
try {
...
} catch (Throwable e) {
connManager.close();
throw new RuntimeException();
}
try {
connManager.waitForConnected(clientConnectTimeout);
} catch (Throwable e) {
connManager.close();
throw new RuntimeException();
}
This results in a different exception (RuntimeException) returned from the
constructor as earlier (nice exceptions as UnknownHostException,
TimeoutException).
Can this be changed so we keep the old nice exceptions e.g. as follows
(requiring the constructor to declare these) or at least include them as cause
in the RuntimeException?
boolean closeBecauseOfException = true;
try {
...
connManager.waitForConnected(clientConnectTimeout);
closeBecauseOfException = false
} finally {
if (closeBecauseOfException) {
connManager.close();
}
}
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
