[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-11 Thread Hadoop QA (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13251377#comment-13251377 ] Hadoop QA commented on HBASE-5666: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-11 Thread Hadoop QA (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13251462#comment-13251462 ] Hadoop QA commented on HBASE-5666: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-10 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250868#comment-13250868 ] Matteo Bertozzi commented on HBASE-5666: The SOCKET_RETRY_WAIT_MS is 200ms but

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-10 Thread stack (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250889#comment-13250889 ] stack commented on HBASE-5666: -- bq. If the client comes up during this time I think that

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-10 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250907#comment-13250907 ] Matteo Bertozzi commented on HBASE-5666: {quote} Does this need to be public? +

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-10 Thread stack (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250915#comment-13250915 ] stack commented on HBASE-5666: -- @Matteo Mind trying what client does when no hbase root node?

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-10 Thread Hadoop QA (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250927#comment-13250927 ] Hadoop QA commented on HBASE-5666: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-10 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250933#comment-13250933 ] Matteo Bertozzi commented on HBASE-5666: @Stack The client crash with this message

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-10 Thread stack (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250941#comment-13250941 ] stack commented on HBASE-5666: -- Should we admend the message to include 'Check the master is

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-10 Thread Hadoop QA (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13251056#comment-13251056 ] Hadoop QA commented on HBASE-5666: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-10 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13251100#comment-13251100 ] Matteo Bertozzi commented on HBASE-5666: @stack do you mean, change the log

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-10 Thread stack (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13251351#comment-13251351 ] stack commented on HBASE-5666: -- Yes. That looks good. Let me retry against hadoopqa. A

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-10 Thread stack (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13251352#comment-13251352 ] stack commented on HBASE-5666: -- I see that you don't have the above change in v7. You want

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-09 Thread stack (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13250269#comment-13250269 ] stack commented on HBASE-5666: -- Patch looks good. Logs '{ +

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-06 Thread Hadoop QA (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13248646#comment-13248646 ] Hadoop QA commented on HBASE-5666: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-06 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13248666#comment-13248666 ] Matteo Bertozzi commented on HBASE-5666: Since the logical flow is: * HMaster

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-06 Thread Uma Maheswara Rao G (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13249141#comment-13249141 ] Uma Maheswara Rao G commented on HBASE-5666: Wrong JIRA...please ignore my

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-04 Thread Zhihong Yu (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13246764#comment-13246764 ] Zhihong Yu commented on HBASE-5666: --- For patch v5, please add javadoc for: {code} +

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-04 Thread Hadoop QA (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13246804#comment-13246804 ] Hadoop QA commented on HBASE-5666: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-04 Thread Hadoop QA (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13246835#comment-13246835 ] Hadoop QA commented on HBASE-5666: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-03 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13245720#comment-13245720 ] Matteo Bertozzi commented on HBASE-5666: Still looking at the 0.90 code... The new

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-03 Thread Zhihong Yu (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13245734#comment-13245734 ] Zhihong Yu commented on HBASE-5666: --- Patch v4 looks good. Minor comments below: Since

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-03 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13245758#comment-13245758 ] Matteo Bertozzi commented on HBASE-5666: The problem that I see with the patch v4

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-03 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13245759#comment-13245759 ] Matteo Bertozzi commented on HBASE-5666: The problem that I see with the patch v4

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-03 Thread Zhihong Yu (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13245795#comment-13245795 ] Zhihong Yu commented on HBASE-5666: --- Interesting idea above. +1 on removing special case

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-02 Thread Hadoop QA (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13244120#comment-13244120 ] Hadoop QA commented on HBASE-5666: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-02 Thread stack (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13244471#comment-13244471 ] stack commented on HBASE-5666: -- On creation of ZooKeeperWatcher, we do following. Why is it

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-02 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13244488#comment-13244488 ] Matteo Bertozzi commented on HBASE-5666: The problem here is that there's no

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-02 Thread stack (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13244492#comment-13244492 ] stack commented on HBASE-5666: -- This is the code that is supposed to create the base node

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-02 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13244537#comment-13244537 ] Matteo Bertozzi commented on HBASE-5666: m... maybe i've lost something but, in

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-02 Thread stack (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13244560#comment-13244560 ] stack commented on HBASE-5666: -- Thanks for digging. Seems like the RecoverableZK is failing

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-02 Thread Hadoop QA (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13244564#comment-13244564 ] Hadoop QA commented on HBASE-5666: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-01 Thread Ted Yu (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13243845#comment-13243845 ] Ted Yu commented on HBASE-5666: --- {code} +if (keeperEx != null) + throw keeperEx;

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-01 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13243855#comment-13243855 ] Matteo Bertozzi commented on HBASE-5666: @Ted woo good catch I've just translated

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-01 Thread Ted Yu (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13243863#comment-13243863 ] Ted Yu commented on HBASE-5666: --- Patch v2 looks good. RegionServer doesn't

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-01 Thread Hadoop QA (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13243871#comment-13243871 ] Hadoop QA commented on HBASE-5666: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-04-01 Thread Hadoop QA (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13243902#comment-13243902 ] Hadoop QA commented on HBASE-5666: -- -1 overall. Here are the results of testing the

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-31 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13243236#comment-13243236 ] Matteo Bertozzi commented on HBASE-5666: diving into the code, I've also noticed

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-31 Thread Zhihong Yu (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13243239#comment-13243239 ] Zhihong Yu commented on HBASE-5666: --- Refactoring ZKUtil.waitForBaseZNode() so that it

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-31 Thread Zhihong Yu (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13243296#comment-13243296 ] Zhihong Yu commented on HBASE-5666: --- Patch makes sense. Can you integrate it into

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-31 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13243302#comment-13243302 ] Matteo Bertozzi commented on HBASE-5666: I was thinking to patch

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-31 Thread Zhihong Yu (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13243310#comment-13243310 ] Zhihong Yu commented on HBASE-5666: ---

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-30 Thread stack (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13242684#comment-13242684 ] stack commented on HBASE-5666: -- We used to retry our very first zk operation a few times

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-30 Thread Zhihong Yu (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13242687#comment-13242687 ] Zhihong Yu commented on HBASE-5666: --- I think we can introduce something similar to the

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-30 Thread stack (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13242696#comment-13242696 ] stack commented on HBASE-5666: -- Please be careful here. What version of hbase are we talking

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-30 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13242734#comment-13242734 ] Matteo Bertozzi commented on HBASE-5666: The exception seems to be present in 0.92

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-30 Thread nkeywal (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13242796#comment-13242796 ] nkeywal commented on HBASE-5666: I confirm I didn't modify this part in trunk. But who

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-30 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13242801#comment-13242801 ] Matteo Bertozzi commented on HBASE-5666: yep, just checked and the code seems

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-28 Thread Zhihong Yu (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13240751#comment-13240751 ] Zhihong Yu commented on HBASE-5666: --- @Matteo: You're suggesting addition of a loop to

[jira] [Commented] (HBASE-5666) RegionServer doesn't retry to check if base node is available

2012-03-28 Thread Matteo Bertozzi (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13240757#comment-13240757 ] Matteo Bertozzi commented on HBASE-5666: @Zhihong Yu: Probably a retry loop is the