jira-importer

githubbot: Github user akira commented on the issue: https://github.com/apache/curator/pull/198 Ran tests locally: ``` [INFO] ------------------------------------------------------------------------ [INFO] Reactor Summary: [INFO] [INFO] Apache Curator ..................................... SUCCESS [ 3.114 s] [INFO] Curator Testing...

githubbot: Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/curator/pull/198#discussion_r105545812 — Diff: curator-recipes/src/main/java/org/apache/curator/framework/recipes/nodes/PersistentNode.java — @@ -117,6 +120,7 @@ public void processResult(CuratorFramework client, CuratorEvent event) { //Update is...

githubbot: Github user akira commented on a diff in the pull request: https://github.com/apache/curator/pull/198#discussion_r106834514 — Diff: curator-recipes/src/main/java/org/apache/curator/framework/recipes/nodes/PersistentNode.java — @@ -117,6 +120,7 @@ public void processResult(CuratorFramework client, CuratorEvent event) { //Update is...

githubbot: Github user akira commented on the issue: https://github.com/apache/curator/pull/198 @lvfangmin I updated the PR based on your feedback - moved the notifyListeners to the `processBackgroundCallback` callback (I squashed the commit...

githubbot: Github user lvfangmin commented on the issue: https://github.com/apache/curator/pull/198 Thanks @akira, the new patch looks good to me.

githubbot: Github user akira commented on the issue: https://github.com/apache/curator/pull/198 @lvfangmin thanks, when do you think it will be merged?

githubbot: Github user lvfangmin commented on the issue: https://github.com/apache/curator/pull/198 @akira sorry for the lately reply, I was waiting for another LGTM, the tests look good on my Mac, I'm going...

githubbot: Github user asfgit closed the pull request at: https://github.com/apache/curator/pull/198

Resolve #856

Resolve #856