[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-08-04 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13428723#comment-13428723 ] Hudson commented on HBASE-6337: --- Integrated in HBase-0.94-security-on-Hadoop-23 #6 (See

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

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

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-10 Thread Zhihong Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13410864#comment-13410864 ] Zhihong Ted Yu commented on HBASE-6337: --- I ran TestHLog with patch v4 3 times and

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-10 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13410945#comment-13410945 ] Lars Hofhansl commented on HBASE-6337: -- +1 on patch, +1 for 0.94 (can pull back into

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-10 Thread Zhihong Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13410965#comment-13410965 ] Zhihong Ted Yu commented on HBASE-6337: --- I am running test suite for 0.94 Will

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-10 Thread Zhihong Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13410997#comment-13410997 ] Zhihong Ted Yu commented on HBASE-6337: --- Here is test result for 0.94: {code} Tests

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-10 Thread Zhihong Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13411087#comment-13411087 ] Zhihong Ted Yu commented on HBASE-6337: --- Ran 0.94 test suite again. The tests

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-10 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13411165#comment-13411165 ] Hudson commented on HBASE-6337: --- Integrated in HBase-0.94-security #39 (See

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-09 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13409214#comment-13409214 ] ramkrishna.s.vasudevan commented on HBASE-6337: --- @Chunhui If the region dir

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-09 Thread chunhui shen (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13409221#comment-13409221 ] chunhui shen commented on HBASE-6337: - @ram bq.If the region dir is deleted by CJ then

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-09 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13409557#comment-13409557 ] stack commented on HBASE-6337: -- @Lars I'm not sure why it was done originally. Prakash was

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-09 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13409564#comment-13409564 ] stack commented on HBASE-6337: -- @Chunhui The patch looks good. Thanks for all the cleanup.

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-09 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13409929#comment-13409929 ] Lars Hofhansl commented on HBASE-6337: -- Change itself looks to me. If the writing

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-09 Thread chunhui shen (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13410011#comment-13410011 ] chunhui shen commented on HBASE-6337: - bq.Have you checked out the open region side of

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-08 Thread chunhui shen (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13409149#comment-13409149 ] chunhui shen commented on HBASE-6337: - @ted With the patch, we will split log to

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-08 Thread Zhihong Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13409158#comment-13409158 ] Zhihong Ted Yu commented on HBASE-6337: --- Thanks for the quick response. {code} +

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-08 Thread Zhihong Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13409164#comment-13409164 ] Zhihong Ted Yu commented on HBASE-6337: --- For patch v2, I got a test failure: {code}

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-08 Thread chunhui shen (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13409182#comment-13409182 ] chunhui shen commented on HBASE-6337: - bq.If you can put the 0.94 patch onto a real

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-08 Thread Lars Hofhansl (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13409190#comment-13409190 ] Lars Hofhansl commented on HBASE-6337: -- Why was the rename done in the first place?

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-08 Thread chunhui shen (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13409196#comment-13409196 ] chunhui shen commented on HBASE-6337: - bq.Why was the rename done in the first place?

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

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

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-06 Thread chunhui shen (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13407730#comment-13407730 ] chunhui shen commented on HBASE-6337: - I will upload a patch later as a start.

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-06 Thread Zhihong Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13408046#comment-13408046 ] Zhihong Ted Yu commented on HBASE-6337: --- Where does the following code from

[jira] [Commented] (HBASE-6337) [MTTR] Remove renaming tmp log file in SplitLogManager

2012-07-06 Thread stack (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-6337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13408344#comment-13408344 ] stack commented on HBASE-6337: -- If you are removing tmp dir, don't be half-hearted about it.