[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15097361#comment-15097361 ] Chris Nauroth commented on ZOOKEEPER-2347: -- +1 for the patch. Nice work, [~rakeshr]! >

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15095073#comment-15095073 ] Ted Yu commented on ZOOKEEPER-2347: --- Assuming there was only test change since I performed

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15093401#comment-15093401 ] Rakesh R commented on ZOOKEEPER-2347: - Thanks [~rgs] for the reviews. bq. One question though,

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15088637#comment-15088637 ] Rakesh R commented on ZOOKEEPER-2347: - Thanks again [~fpj] for the review comments. Attached

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15088110#comment-15088110 ] Flavio Junqueira commented on ZOOKEEPER-2347: - [~rakeshr] looks good, I just have a few

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15084965#comment-15084965 ] Rakesh R commented on ZOOKEEPER-2347: - Thanks [~fpj] for the detailed comments. I've modified the

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15082931#comment-15082931 ] Flavio Junqueira commented on ZOOKEEPER-2347: - [~rakeshr] thanks for the update. It looks

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15081602#comment-15081602 ] Ted Yu commented on ZOOKEEPER-2347: --- [~fpj]: Can you review the patch ? > Deadlock shutting down

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15075556#comment-15075556 ] Ted Yu commented on ZOOKEEPER-2347: --- Rakesh: Thanks for updating the test case. > Deadlock shutting

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15070681#comment-15070681 ] Rakesh R commented on ZOOKEEPER-2347: - yes, agreed. I've attached another patch modifying the unit

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15064972#comment-15064972 ] Flavio Junqueira commented on ZOOKEEPER-2347: - The patch looks good, but I'm not really

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15061765#comment-15061765 ] Flavio Junqueira commented on ZOOKEEPER-2347: - To be consistent, I'm reposting the comment

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15062994#comment-15062994 ] Ted Yu commented on ZOOKEEPER-2347: --- Not sure how I can test this with hbase unit test(s). As far

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15063086#comment-15063086 ] Ted Yu commented on ZOOKEEPER-2347: --- Thanks for the pointer, Chris. I ran TestSplitLogManager after

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15063101#comment-15063101 ] Chris Nauroth commented on ZOOKEEPER-2347: -- [~yuzhih...@gmail.com], thank you for the help

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15063059#comment-15063059 ] Chris Nauroth commented on ZOOKEEPER-2347: -- bq. As far as I know, zookeeper still uses ant to

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15063076#comment-15063076 ] Jason Rosenberg commented on ZOOKEEPER-2347: What are the conditions that trigger this

[jira] [Commented] (ZOOKEEPER-2347) Deadlock shutting down zookeeper

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15061675#comment-15061675 ] Rakesh R commented on ZOOKEEPER-2347: - Thanks [~tedyu] for reporting this issue. Also, thanks