You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@helix.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2014/11/21 05:37:33 UTC
[jira] [Commented] (HELIX-555) ClusterStateVerifier leaks
ZkClients.
[ https://issues.apache.org/jira/browse/HELIX-555?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14220509#comment-14220509 ]
ASF GitHub Bot commented on HELIX-555:
--------------------------------------
GitHub user atcurtis opened a pull request:
https://github.com/apache/helix/pull/12
[HELIX-555] Fix deficiency in ClusterStateVerifier api
https://issues.apache.org/jira/browse/HELIX-555
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/atcurtis/helix HELIX-555
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/helix/pull/12.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #12
----
commit b2794e744e945da966c7ac6ae6408636951281d3
Author: Antony T Curtis <ac...@linkedin.com>
Date: 2014-11-21T04:36:01Z
[HELIX-555] Fix deficiency in ClusterStateVerifier api
----
> ClusterStateVerifier leaks ZkClients.
> -------------------------------------
>
> Key: HELIX-555
> URL: https://issues.apache.org/jira/browse/HELIX-555
> Project: Apache Helix
> Issue Type: Bug
> Affects Versions: 0.6.3
> Reporter: Antony T Curtis
> Priority: Blocker
>
> The classes in ClusterStateVerifier tend to leak ZkClients because there is no way to provide an already constructed client to the class.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)