HBase
  1. HBase
  2. HBASE-7072

HBase-5256 breaks 0.92-0.94 compatibility

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.92.1, 0.92.2, 0.94.0
    • Fix Version/s: 0.92.3
    • Component/s: master, shell
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      HBase-5286 changes RegionLoad writable in 94.0, making it incompatible with 92. A fix was made in HBase-5795 where a 94 client can read response from a 92 server, but not vice versa. Currently, if a 92 client tries to do read RegionLoad (HBase shell "status" command, or, 92 master and 94 regionserver), it just hangs.

      1. HBase-7072-92-v2.patch
        5 kB
        Himanshu Vashishtha
      2. HBase-7072-92.patch
        5 kB
        Himanshu Vashishtha

        Activity

        Himanshu Vashishtha created issue -
        Hide
        Himanshu Vashishtha added a comment -

        92 client can have a another read method in which it can understand the 94 server response (similar to 5795)

        Show
        Himanshu Vashishtha added a comment - 92 client can have a another read method in which it can understand the 94 server response (similar to 5795)
        Hide
        Ted Yu added a comment -

        HBASE-5286 doesn't seem related.
        Can you clarify ?

        Show
        Ted Yu added a comment - HBASE-5286 doesn't seem related. Can you clarify ?
        Hide
        Ted Yu added a comment -

        HBASE-5256 was the issue that started all this.

        Show
        Ted Yu added a comment - HBASE-5256 was the issue that started all this.
        Himanshu Vashishtha made changes -
        Field Original Value New Value
        Summary HBase-5286 breaks 92-94 compatibility HBase-5256 breaks 92-94 compatibility
        Hide
        Himanshu Vashishtha added a comment -

        Ted Yu Yes, that's correct. I messed up with the number. Thanks.

        Show
        Himanshu Vashishtha added a comment - Ted Yu Yes, that's correct. I messed up with the number. Thanks.
        Hide
        Himanshu Vashishtha added a comment -

        This patch provides a method to read 94 server response.
        I tested a
        a) 0.94.0 server with a patched 0.92.2 client
        b) patched 0.92.2 server against a 0.94.0 client.
        c) 0.92.1 server with a patched 0.92.2 client

        All of the above tests pass.

        Show
        Himanshu Vashishtha added a comment - This patch provides a method to read 94 server response. I tested a a) 0.94.0 server with a patched 0.92.2 client b) patched 0.92.2 server against a 0.94.0 client. c) 0.92.1 server with a patched 0.92.2 client All of the above tests pass.
        Himanshu Vashishtha made changes -
        Attachment HBase-7072-92.patch [ 12551612 ]
        Hide
        Enis Soztutar added a comment -

        It is just unfortunate that we had HBASE-5256 committed to 0.94. At this point we cannot revert it since it is already released. It seems that the only thing we can do for 92 is to apply this patch.
        +1 on the patch. My only nit is to remove the line.

        +      // super.readFields(in);
        
        Show
        Enis Soztutar added a comment - It is just unfortunate that we had HBASE-5256 committed to 0.94. At this point we cannot revert it since it is already released. It seems that the only thing we can do for 92 is to apply this patch. +1 on the patch. My only nit is to remove the line. + // super .readFields(in);
        Hide
        Ted Yu added a comment -

        Please fix the spelling for method name:

        +    public void readFiels94(DataInput in) throws IOException {
        

        Otherwise patch looks good.

        Show
        Ted Yu added a comment - Please fix the spelling for method name: + public void readFiels94(DataInput in) throws IOException { Otherwise patch looks good.
        Hide
        Himanshu Vashishtha added a comment -

        Comments incorporated. Thanks for review.

        Show
        Himanshu Vashishtha added a comment - Comments incorporated. Thanks for review.
        Himanshu Vashishtha made changes -
        Attachment HBase-7072-92-v2.patch [ 12551640 ]
        Ted Yu made changes -
        Summary HBase-5256 breaks 92-94 compatibility HBase-5256 breaks 0.92-0.94 compatibility
        Hide
        Ted Yu added a comment -

        Integrated to 0.92

        Thanks for the patch, Himanshu.

        Thanks for the review, Enis.

        Show
        Ted Yu added a comment - Integrated to 0.92 Thanks for the patch, Himanshu. Thanks for the review, Enis.
        Ted Yu made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Hadoop Flags Reviewed [ 10343 ]
        Resolution Fixed [ 1 ]
        Hide
        Hudson added a comment -

        Integrated in HBase-0.92 #601 (See https://builds.apache.org/job/HBase-0.92/601/)
        HBASE-7072 HBase-5256 breaks 0.92-0.94 compatibility (Himanshu) (Revision 1404407)

        Result = FAILURE
        tedyu :
        Files :

        • /hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/HServerLoad.java
        Show
        Hudson added a comment - Integrated in HBase-0.92 #601 (See https://builds.apache.org/job/HBase-0.92/601/ ) HBASE-7072 HBase-5256 breaks 0.92-0.94 compatibility (Himanshu) (Revision 1404407) Result = FAILURE tedyu : Files : /hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/HServerLoad.java
        Hide
        Hudson added a comment -

        Integrated in HBase-0.92-security #146 (See https://builds.apache.org/job/HBase-0.92-security/146/)
        HBASE-7072 HBase-5256 breaks 0.92-0.94 compatibility (Himanshu) (Revision 1404407)

        Result = FAILURE
        tedyu :
        Files :

        • /hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/HServerLoad.java
        Show
        Hudson added a comment - Integrated in HBase-0.92-security #146 (See https://builds.apache.org/job/HBase-0.92-security/146/ ) HBASE-7072 HBase-5256 breaks 0.92-0.94 compatibility (Himanshu) (Revision 1404407) Result = FAILURE tedyu : Files : /hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/HServerLoad.java

          People

          • Assignee:
            Himanshu Vashishtha
            Reporter:
            Himanshu Vashishtha
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development