HBase
  1. HBase
  2. HBASE-6263

Use default mode for HBase Thrift gateway if not specified

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.94.0, 0.95.2
    • Fix Version/s: 0.94.2
    • Component/s: Thrift
    • Labels:
    • Release Note:
      The Thrift gateway no longer requires specification on the command line of an implementation type when starting. If missing, the default "threadpool" implementation will be selected.

      Description

      The Thrift gateway should start with a default mode if one is not selected. Currently, instead we see:

      Exception in thread "main" java.lang.AssertionError: Exactly one option out of [-hsha, -nonblocking, -threadpool, -threadedselector] has to be specified
      	at org.apache.hadoop.hbase.thrift.ThriftServerRunner$ImplType.setServerImpl(ThriftServerRunner.java:201)
      	at org.apache.hadoop.hbase.thrift.ThriftServer.processOptions(ThriftServer.java:169)
      	at org.apache.hadoop.hbase.thrift.ThriftServer.doMain(ThriftServer.java:85)
      	at org.apache.hadoop.hbase.thrift.ThriftServer.main(ThriftServer.java:192)
      

      See also BIGTOP-648.

      1. HBASE-6263-0.94.patch
        0.9 kB
        Andrew Purtell
      2. HBASE-6263.patch
        1.0 kB
        Andrew Purtell

        Issue Links

          Activity

          Hide
          stack added a comment -

          Fix up after bulk move overwrote some 0.94.2 fix versions w/ 0.95.0 (Noticed by Lars Hofhansl)

          Show
          stack added a comment - Fix up after bulk move overwrote some 0.94.2 fix versions w/ 0.95.0 (Noticed by Lars Hofhansl)
          Hide
          Hudson added a comment -

          Integrated in HBase-0.94-security-on-Hadoop-23 #7 (See https://builds.apache.org/job/HBase-0.94-security-on-Hadoop-23/7/)
          HBASE-6263. Use default mode for Thrift gateway if not specified (Revision 1373593)

          Result = FAILURE
          apurtell :
          Files :

          • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
          Show
          Hudson added a comment - Integrated in HBase-0.94-security-on-Hadoop-23 #7 (See https://builds.apache.org/job/HBase-0.94-security-on-Hadoop-23/7/ ) HBASE-6263 . Use default mode for Thrift gateway if not specified (Revision 1373593) Result = FAILURE apurtell : Files : /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
          Hide
          Hudson added a comment -

          Integrated in HBase-0.94-security #48 (See https://builds.apache.org/job/HBase-0.94-security/48/)
          HBASE-6263. Use default mode for Thrift gateway if not specified (Revision 1373593)

          Result = FAILURE
          apurtell :
          Files :

          • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
          Show
          Hudson added a comment - Integrated in HBase-0.94-security #48 (See https://builds.apache.org/job/HBase-0.94-security/48/ ) HBASE-6263 . Use default mode for Thrift gateway if not specified (Revision 1373593) Result = FAILURE apurtell : Files : /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
          Hide
          Hudson added a comment -

          Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #131 (See https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/131/)
          HBASE-6263. Use default mode for Thrift gateway if not specified (Revision 1373591)

          Result = FAILURE
          apurtell :
          Files :

          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
          Show
          Hudson added a comment - Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #131 (See https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/131/ ) HBASE-6263 . Use default mode for Thrift gateway if not specified (Revision 1373591) Result = FAILURE apurtell : Files : /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
          Hide
          Hudson added a comment -

          Integrated in HBase-0.94 #396 (See https://builds.apache.org/job/HBase-0.94/396/)
          HBASE-6263. Use default mode for Thrift gateway if not specified (Revision 1373593)

          Result = SUCCESS
          apurtell :
          Files :

          • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
          Show
          Hudson added a comment - Integrated in HBase-0.94 #396 (See https://builds.apache.org/job/HBase-0.94/396/ ) HBASE-6263 . Use default mode for Thrift gateway if not specified (Revision 1373593) Result = SUCCESS apurtell : Files : /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
          Hide
          Hudson added a comment -

          Integrated in HBase-TRUNK #3220 (See https://builds.apache.org/job/HBase-TRUNK/3220/)
          HBASE-6263. Use default mode for Thrift gateway if not specified (Revision 1373591)

          Result = SUCCESS
          apurtell :
          Files :

          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
          Show
          Hudson added a comment - Integrated in HBase-TRUNK #3220 (See https://builds.apache.org/job/HBase-TRUNK/3220/ ) HBASE-6263 . Use default mode for Thrift gateway if not specified (Revision 1373591) Result = SUCCESS apurtell : Files : /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
          Hide
          Andrew Purtell added a comment -

          Committed to trunk and 0.94 branch.

          Show
          Andrew Purtell added a comment - Committed to trunk and 0.94 branch.
          Hide
          Andrew Purtell added a comment -

          Sorry J-D, forgot about this, committing now.

          Show
          Andrew Purtell added a comment - Sorry J-D, forgot about this, committing now.
          Hide
          Jean-Daniel Cryans added a comment -

          Andrew, do you want to commit this? I stopped counting the number of times we messed up a thrift bounce because of this.

          Show
          Jean-Daniel Cryans added a comment - Andrew, do you want to commit this? I stopped counting the number of times we messed up a thrift bounce because of this.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12533517/HBASE-6263-0.94.patch
          against trunk revision .

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

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          -1 patch. The patch command could not apply the patch.

          Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/2263//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12533517/HBASE-6263-0.94.patch against trunk revision . +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. -1 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/2263//console This message is automatically generated.

            People

            • Assignee:
              Andrew Purtell
              Reporter:
              Andrew Purtell
            • Votes:
              1 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development