You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Zhihong Yu (Created) (JIRA)" <ji...@apache.org> on 2012/04/19 02:52:40 UTC

[jira] [Created] (HBASE-5826) Improve sync of HLog edits

Improve sync of HLog edits
--------------------------

                 Key: HBASE-5826
                 URL: https://issues.apache.org/jira/browse/HBASE-5826
             Project: HBase
          Issue Type: Improvement
            Reporter: Zhihong Yu


HBASE-5782 solved the correctness issue for the sync of HLog edits.
Todd provided a patch that would achieve higher throughput.

This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Zhihong Yu updated HBASE-5826:
------------------------------

    Attachment: 5826-v3.txt
    
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Todd Lipcon (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13257159#comment-13257159 ] 

Todd Lipcon commented on HBASE-5826:
------------------------------------

I'm happy to work on this, if folks would like. But, I probably won't be able to commit a lot of time - so if people are antsy to get this sooner rather than later, feel free to steal it from me :)
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13278422#comment-13278422 ] 

Hudson commented on HBASE-5826:
-------------------------------

Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #8 (See [https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/8/])
    HBASE-5826 Revert, Todd has review comments pending. (Revision 1339673)
HBASE-5826 Improve sync of HLog edits (Todd) (Revision 1339671)

     Result = FAILURE
tedyu : 
Files : 
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLog.java
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/wal/SequenceFileLogWriter.java
* /hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestHLog.java

tedyu : 
Files : 
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLog.java
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/wal/SequenceFileLogWriter.java
* /hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestHLog.java

                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826-v4.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-5826) Improve sync of HLog edits

Posted by "Ted Yu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ted Yu updated HBASE-5826:
--------------------------

    Attachment:     (was: 5826.txt)
    
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>         Attachments: 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13277911#comment-13277911 ] 

Zhihong Yu commented on HBASE-5826:
-----------------------------------

Integrated to trunk.

Thanks for the patch, Todd.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13277836#comment-13277836 ] 

Zhihong Yu commented on HBASE-5826:
-----------------------------------

Will integrate this morning (PST) if there is no objection.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13277209#comment-13277209 ] 

Zhihong Yu commented on HBASE-5826:
-----------------------------------

TestAdmin#testCloseRegionThatFetchesTheHRIFromMeta failure wasn't related to the patch.

Locally it passed for me.

Are there further review comments about Todd's patch ?
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13277915#comment-13277915 ] 

Zhihong Yu commented on HBASE-5826:
-----------------------------------

Just saw Todd's comment.

Patch reverted.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Todd Lipcon (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13277908#comment-13277908 ] 

Todd Lipcon commented on HBASE-5826:
------------------------------------

I'm not sure the error handling paths are write. What happens if the syncing thread hits an error - the other threads might end up stuck in "wait()" forever.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13277987#comment-13277987 ] 

Hadoop QA commented on HBASE-5826:
----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12527851/5826-v4.txt
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 3 new or modified tests.

    +1 hadoop23.  The patch compiles against the hadoop 0.23.x profile.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    -1 findbugs.  The patch appears to introduce 32 new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

     -1 core tests.  The patch failed these unit tests:
                       org.apache.hadoop.hbase.coprocessor.TestMasterObserver

Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/1911//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/1911//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html
Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1911//console

This message is automatically generated.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826-v4.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Zhihong Yu updated HBASE-5826:
------------------------------

    Attachment: 5826-v4.txt
    
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826-v4.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Zhihong Yu updated HBASE-5826:
------------------------------

    Attachment: 5826-v2.txt

Patch rebased on trunk.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Zhihong Yu updated HBASE-5826:
------------------------------

    Status: Patch Available  (was: Open)
    
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>         Attachments: 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-5826) Improve sync of HLog edits

Posted by "stack (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

stack updated HBASE-5826:
-------------------------

    Priority: Critical  (was: Major)
    
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Ted Yu
>            Assignee: Todd Lipcon
>            Priority: Critical
>             Fix For: 0.96.0
>
>         Attachments: 5826.txt, 5826-v2.txt, 5826-v3.txt, 5826-v4.txt, 5826-v5.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Assigned] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Zhihong Yu reassigned HBASE-5826:
---------------------------------

    Assignee: Todd Lipcon
    
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-5826) Improve sync of HLog edits

Posted by "Ted Yu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ted Yu updated HBASE-5826:
--------------------------

    Attachment: 5826.txt
    
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>         Attachments: 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-5826) Improve sync of HLog edits

Posted by "Ted Yu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ted Yu updated HBASE-5826:
--------------------------

    Attachment: 5826-v5.txt

Rebased against trunk.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Ted Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826.txt, 5826-v2.txt, 5826-v3.txt, 5826-v4.txt, 5826-v5.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13277152#comment-13277152 ] 

Hadoop QA commented on HBASE-5826:
----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12527695/5826-v2.txt
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 3 new or modified tests.

    +1 hadoop23.  The patch compiles against the hadoop 0.23.x profile.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    -1 findbugs.  The patch appears to introduce 31 new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

     -1 core tests.  The patch failed these unit tests:
                       org.apache.hadoop.hbase.regionserver.wal.TestLogRollingNoCluster

Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/1895//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/1895//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html
Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1895//console

This message is automatically generated.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Zhihong Yu updated HBASE-5826:
------------------------------

    Fix Version/s: 0.96.0
    
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13277992#comment-13277992 ] 

Hudson commented on HBASE-5826:
-------------------------------

Integrated in HBase-TRUNK #2891 (See [https://builds.apache.org/job/HBase-TRUNK/2891/])
    HBASE-5826 Revert, Todd has review comments pending. (Revision 1339673)

     Result = SUCCESS
tedyu : 
Files : 
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLog.java
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/wal/SequenceFileLogWriter.java
* /hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestHLog.java

                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826-v4.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13277951#comment-13277951 ] 

Hudson commented on HBASE-5826:
-------------------------------

Integrated in HBase-TRUNK #2890 (See [https://builds.apache.org/job/HBase-TRUNK/2890/])
    HBASE-5826 Improve sync of HLog edits (Todd) (Revision 1339671)

     Result = FAILURE
tedyu : 
Files : 
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLog.java
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/wal/SequenceFileLogWriter.java
* /hbase/trunk/src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestHLog.java

                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826-v4.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13264036#comment-13264036 ] 

Hadoop QA commented on HBASE-5826:
----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12524921/5826.txt
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 3 new or modified tests.

    +1 hadoop23.  The patch compiles against the hadoop 0.23.x profile.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    -1 findbugs.  The patch appears to introduce 2 new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

     -1 core tests.  The patch failed these unit tests:
                       org.apache.hadoop.hbase.regionserver.wal.TestLogRollingNoCluster

Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/1673//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/1673//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html
Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1673//console

This message is automatically generated.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>         Attachments: 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259831#comment-13259831 ] 

Hadoop QA commented on HBASE-5826:
----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12523827/5826.txt
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 3 new or modified tests.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    +1 findbugs.  The patch does not introduce any new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

     -1 core tests.  The patch failed these unit tests:
                       org.apache.hadoop.hbase.TestDrainingServer

Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/1610//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/1610//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html
Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1610//console

This message is automatically generated.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>         Attachments: 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13442570#comment-13442570 ] 

Hadoop QA commented on HBASE-5826:
----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12542625/5826-v5.txt
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 3 new or modified tests.

    +1 hadoop2.0.  The patch compiles against the hadoop 2.0 profile.

    -1 javadoc.  The javadoc tool appears to have generated 94 warning messages.

    -1 javac.  The applied patch generated 5 javac compiler warnings (more than the trunk's current 4 warnings).

    -1 findbugs.  The patch appears to introduce 13 new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

     -1 core tests.  The patch failed these unit tests:
                       org.apache.hadoop.hbase.regionserver.wal.TestWALReplayCompressed
                  org.apache.hadoop.hbase.client.TestFromClientSideWithCoprocessor
                  org.apache.hadoop.hbase.regionserver.wal.TestWALReplay

Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/2702//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2702//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop2-compat.html
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2702//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2702//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2702//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/2702//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/2702//console

This message is automatically generated.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Ted Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826.txt, 5826-v2.txt, 5826-v3.txt, 5826-v4.txt, 5826-v5.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13277937#comment-13277937 ] 

Zhihong Yu commented on HBASE-5826:
-----------------------------------

@Todd:
Were you referring to the following code ?
{code}
+    synchronized void waitForSync(long txid) throws InterruptedException {
+      while (syncedTillHere < txid) {
+        wait();
+      }
+    }
{code}
It turns out that the above method is never referenced.

See patch v4 where it is removed.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826-v4.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13277204#comment-13277204 ] 

Hadoop QA commented on HBASE-5826:
----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12527713/5826-v3.txt
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 3 new or modified tests.

    +1 hadoop23.  The patch compiles against the hadoop 0.23.x profile.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    -1 findbugs.  The patch appears to introduce 31 new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

     -1 core tests.  The patch failed these unit tests:
                       org.apache.hadoop.hbase.client.TestAdmin

Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/1896//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/1896//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html
Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1896//console

This message is automatically generated.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13279049#comment-13279049 ] 

Zhihong Yu commented on HBASE-5826:
-----------------------------------

@Todd:
Do you have further comments ?
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>            Assignee: Todd Lipcon
>             Fix For: 0.96.0
>
>         Attachments: 5826-v2.txt, 5826-v3.txt, 5826-v4.txt, 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-5826) Improve sync of HLog edits

Posted by "Zhihong Yu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-5826?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Zhihong Yu updated HBASE-5826:
------------------------------

    Attachment: 5826.txt

Todd's patch, for trunk.
                
> Improve sync of HLog edits
> --------------------------
>
>                 Key: HBASE-5826
>                 URL: https://issues.apache.org/jira/browse/HBASE-5826
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Zhihong Yu
>         Attachments: 5826.txt
>
>
> HBASE-5782 solved the correctness issue for the sync of HLog edits.
> Todd provided a patch that would achieve higher throughput.
> This JIRA is a continuation of Todd's work submitted there.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira