HBase
  1. HBase
  2. HBASE-5613

ThriftServer getTableRegions does not return serverName and port

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.94.0, 0.95.0
    • Component/s: Thrift
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    1. ASF.LICENSE.NOT.GRANTED--HBASE-5613.D2403.1.patch
      7 kB
      Phabricator
    2. ASF.LICENSE.NOT.GRANTED--HBASE-5613.D2403.2.patch
      7 kB
      Phabricator
    3. ASF.LICENSE.NOT.GRANTED--HBASE-5613.D2403.3.patch
      8 kB
      Phabricator
    4. ASF.LICENSE.NOT.GRANTED--HBASE-5613.D2403.4.patch
      8 kB
      Phabricator
    5. ASF.LICENSE.NOT.GRANTED--HBASE-5613.D2403.5.patch
      8 kB
      Phabricator
    6. HBASE-5613.0.94.2.txt
      9 kB
      Scott Chen
    7. HBASE-5613.0.94.txt
      4 kB
      Scott Chen

      Activity

      Hide
      Phabricator added a comment -

      sc requested code review of "HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port".
      Reviewers: dhruba, tedyu, JIRA

      ThriftServer getTableRegions does not return serverName and port

      Task ID: #

      Blame Rev:

      TEST PLAN
      Revert Plan:

      Tags:

      REVISION DETAIL
      https://reviews.facebook.net/D2403

      AFFECTED FILES
      src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
      src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java

      MANAGE HERALD DIFFERENTIAL RULES
      https://reviews.facebook.net/herald/view/differential/

      WHY DID I GET THIS EMAIL?
      https://reviews.facebook.net/herald/transcript/5397/

      Tip: use the X-Herald-Rules header to filter Herald messages in your client.

      Show
      Phabricator added a comment - sc requested code review of " HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port". Reviewers: dhruba, tedyu, JIRA ThriftServer getTableRegions does not return serverName and port Task ID: # Blame Rev: TEST PLAN Revert Plan: Tags: REVISION DETAIL https://reviews.facebook.net/D2403 AFFECTED FILES src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java MANAGE HERALD DIFFERENTIAL RULES https://reviews.facebook.net/herald/view/differential/ WHY DID I GET THIS EMAIL? https://reviews.facebook.net/herald/transcript/5397/ Tip: use the X-Herald-Rules header to filter Herald messages in your client.
      Hide
      Phabricator added a comment -

      tedyu has accepted the revision "HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port".

      If tests pass.

      INLINE COMMENTS
      src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java:445 Is this for debugging ?

      REVISION DETAIL
      https://reviews.facebook.net/D2403

      BRANCH
      5613

      Show
      Phabricator added a comment - tedyu has accepted the revision " HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port". If tests pass. INLINE COMMENTS src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java:445 Is this for debugging ? REVISION DETAIL https://reviews.facebook.net/D2403 BRANCH 5613
      Hide
      Phabricator added a comment -

      sc has commented on the revision "HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port".

      INLINE COMMENTS
      src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java:445 Yes. But it's also good to print something during the test.

      REVISION DETAIL
      https://reviews.facebook.net/D2403

      BRANCH
      5613

      Show
      Phabricator added a comment - sc has commented on the revision " HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port". INLINE COMMENTS src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java:445 Yes. But it's also good to print something during the test. REVISION DETAIL https://reviews.facebook.net/D2403 BRANCH 5613
      Hide
      Hadoop QA added a comment -

      -1 overall. Here are the results of testing the latest attachment
      http://issues.apache.org/jira/secure/attachment/12519334/HBASE-5613.D2403.1.patch
      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 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.mapreduce.TestImportTsv
      org.apache.hadoop.hbase.mapred.TestTableMapReduce
      org.apache.hadoop.hbase.mapreduce.TestHFileOutputFormat
      org.apache.hadoop.hbase.master.TestSplitLogManager
      org.apache.hadoop.hbase.thrift.TestThriftServerCmdLine

      Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/1258//testReport/
      Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/1258//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html
      Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1258//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/12519334/HBASE-5613.D2403.1.patch 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 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.mapreduce.TestImportTsv org.apache.hadoop.hbase.mapred.TestTableMapReduce org.apache.hadoop.hbase.mapreduce.TestHFileOutputFormat org.apache.hadoop.hbase.master.TestSplitLogManager org.apache.hadoop.hbase.thrift.TestThriftServerCmdLine Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/1258//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/1258//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1258//console This message is automatically generated.
      Hide
      Scott Chen added a comment -

      TestThriftServerCmdLine should be related. I will fix it.

      Show
      Scott Chen added a comment - TestThriftServerCmdLine should be related. I will fix it.
      Hide
      Phabricator added a comment -

      sc updated the revision "HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port".
      Reviewers: dhruba, tedyu, JIRA

      Fixed unit test

      REVISION DETAIL
      https://reviews.facebook.net/D2403

      AFFECTED FILES
      src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
      src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java

      Show
      Phabricator added a comment - sc updated the revision " HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port". Reviewers: dhruba, tedyu, JIRA Fixed unit test REVISION DETAIL https://reviews.facebook.net/D2403 AFFECTED FILES src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
      Hide
      Hadoop QA added a comment -

      -1 overall. Here are the results of testing the latest attachment
      http://issues.apache.org/jira/secure/attachment/12519494/HBASE-5613.D2403.2.patch
      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 appears to introduce 1 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.TestSplitTransactionOnCluster

      Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/1260//testReport/
      Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/1260//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html
      Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1260//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/12519494/HBASE-5613.D2403.2.patch 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 appears to introduce 1 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.TestSplitTransactionOnCluster Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/1260//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/1260//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1260//console This message is automatically generated.
      Hide
      Ted Yu added a comment -

      The test failure is unrelated to thrift.

      Will integrate if there is no objection.

      Show
      Ted Yu added a comment - The test failure is unrelated to thrift. Will integrate if there is no objection.
      Hide
      Ted Yu added a comment -

      Scott is preparing a patch for 0.94

      Show
      Ted Yu added a comment - Scott is preparing a patch for 0.94
      Hide
      Phabricator added a comment -

      lhofhansl has commented on the revision "HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port".

      +1

      REVISION DETAIL
      https://reviews.facebook.net/D2403

      BRANCH
      5613

      Show
      Phabricator added a comment - lhofhansl has commented on the revision " HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port". +1 REVISION DETAIL https://reviews.facebook.net/D2403 BRANCH 5613
      Hide
      Scott Chen added a comment -

      Upload the patch for 0.94

      Show
      Scott Chen added a comment - Upload the patch for 0.94
      Hide
      Phabricator added a comment -

      sc updated the revision "HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port".
      Reviewers: dhruba, tedyu, JIRA

      Fix the case of calling getTableRegion() for non-existing table

      REVISION DETAIL
      https://reviews.facebook.net/D2403

      AFFECTED FILES
      src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
      src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java

      Show
      Phabricator added a comment - sc updated the revision " HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port". Reviewers: dhruba, tedyu, JIRA Fix the case of calling getTableRegion() for non-existing table REVISION DETAIL https://reviews.facebook.net/D2403 AFFECTED FILES src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
      Hide
      Phabricator added a comment -

      sc has commented on the revision "HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port".

      Sorry. I just found that the original method supports the case that table doesn't exist (it returns an empty list).
      I am adding that logic here so it can be consistent.

      REVISION DETAIL
      https://reviews.facebook.net/D2403

      BRANCH
      5613

      Show
      Phabricator added a comment - sc has commented on the revision " HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port". Sorry. I just found that the original method supports the case that table doesn't exist (it returns an empty list). I am adding that logic here so it can be consistent. REVISION DETAIL https://reviews.facebook.net/D2403 BRANCH 5613
      Hide
      Phabricator added a comment -

      lhofhansl has commented on the revision "HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port".

      INLINE COMMENTS
      src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java:589 Hmm... Now you're make two roundtrips.
      Does getTable throw an exception? Can we catch that instead?
      If not, not a big deal.

      REVISION DETAIL
      https://reviews.facebook.net/D2403

      BRANCH
      5613

      Show
      Phabricator added a comment - lhofhansl has commented on the revision " HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port". INLINE COMMENTS src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java:589 Hmm... Now you're make two roundtrips. Does getTable throw an exception? Can we catch that instead? If not, not a big deal. REVISION DETAIL https://reviews.facebook.net/D2403 BRANCH 5613
      Hide
      Hadoop QA added a comment -

      -1 overall. Here are the results of testing the latest attachment
      http://issues.apache.org/jira/secure/attachment/12519544/HBASE-5613.D2403.3.patch
      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 patch. The patch command could not apply the patch.

      Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1268//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/12519544/HBASE-5613.D2403.3.patch 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 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1268//console This message is automatically generated.
      Hide
      Phabricator added a comment -

      sc has commented on the revision "HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port".

      INLINE COMMENTS
      src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java:589 Good idea. We can catch TableNotExist and return empty list. I will make the change. Thanks!

      REVISION DETAIL
      https://reviews.facebook.net/D2403

      BRANCH
      5613

      Show
      Phabricator added a comment - sc has commented on the revision " HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port". INLINE COMMENTS src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java:589 Good idea. We can catch TableNotExist and return empty list. I will make the change. Thanks! REVISION DETAIL https://reviews.facebook.net/D2403 BRANCH 5613
      Hide
      Phabricator added a comment -

      sc updated the revision "HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port".
      Reviewers: dhruba, tedyu, JIRA

      Address the review comment from Lars.

      REVISION DETAIL
      https://reviews.facebook.net/D2403

      AFFECTED FILES
      src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
      src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java

      Show
      Phabricator added a comment - sc updated the revision " HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port". Reviewers: dhruba, tedyu, JIRA Address the review comment from Lars. REVISION DETAIL https://reviews.facebook.net/D2403 AFFECTED FILES src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
      Hide
      Ted Yu added a comment -

      @Scott:
      Have you refreshed your workspace ? HBASE-5591 has been integrated which might cause conflict with this patch.

      Show
      Ted Yu added a comment - @Scott: Have you refreshed your workspace ? HBASE-5591 has been integrated which might cause conflict with this patch.
      Hide
      Scott Chen added a comment -

      @Ted: You're right. I haven't rebase this branch. Let me do that and update the patch.

      Show
      Scott Chen added a comment - @Ted: You're right. I haven't rebase this branch. Let me do that and update the patch.
      Hide
      Phabricator added a comment -

      sc updated the revision "HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port".
      Reviewers: dhruba, tedyu, JIRA

      Rebase

      REVISION DETAIL
      https://reviews.facebook.net/D2403

      AFFECTED FILES
      src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
      src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java

      Show
      Phabricator added a comment - sc updated the revision " HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port". Reviewers: dhruba, tedyu, JIRA Rebase REVISION DETAIL https://reviews.facebook.net/D2403 AFFECTED FILES src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
      Hide
      Hadoop QA added a comment -

      -1 overall. Here are the results of testing the latest attachment
      http://issues.apache.org/jira/secure/attachment/12519672/HBASE-5613.D2403.5.patch
      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 appears to introduce 1 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.replication.TestReplication

      Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/1280//testReport/
      Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/1280//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html
      Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1280//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/12519672/HBASE-5613.D2403.5.patch 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 appears to introduce 1 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.replication.TestReplication Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/1280//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/1280//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1280//console This message is automatically generated.
      Hide
      Ted Yu added a comment -

      TestReplication is not related to thrift.

      Will integrate if there is no objection.

      Show
      Ted Yu added a comment - TestReplication is not related to thrift. Will integrate if there is no objection.
      Hide
      Ted Yu added a comment -

      Integrated to TRUNK.

      Thanks for the patch Scott

      Thanks for the review Lars

      HBASE-5613.D2403.5.patch doesn't apply to 0.94 - will do integration later.

      Show
      Ted Yu added a comment - Integrated to TRUNK. Thanks for the patch Scott Thanks for the review Lars HBASE-5613 .D2403.5.patch doesn't apply to 0.94 - will do integration later.
      Hide
      Scott Chen added a comment -

      Upload patch for 0.94.

      Show
      Scott Chen added a comment - Upload patch for 0.94.
      Hide
      Ted Yu added a comment -

      Patch integrated to 0.94 as well.

      Show
      Ted Yu added a comment - Patch integrated to 0.94 as well.
      Hide
      Hadoop QA added a comment -

      -1 overall. Here are the results of testing the latest attachment
      http://issues.apache.org/jira/secure/attachment/12519722/HBASE-5613.0.94.2.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 patch. The patch command could not apply the patch.

      Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1284//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/12519722/HBASE-5613.0.94.2.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 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/1284//console This message is automatically generated.
      Hide
      Hudson added a comment -

      Integrated in HBase-TRUNK #2693 (See https://builds.apache.org/job/HBase-TRUNK/2693/)
      HBASE-5613 ThriftServer getTableRegions does not return serverName and port (Scott Chen) (Revision 1304602)

      Result = SUCCESS
      tedyu :
      Files :

      • /hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
      • /hbase/trunk/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
      Show
      Hudson added a comment - Integrated in HBase-TRUNK #2693 (See https://builds.apache.org/job/HBase-TRUNK/2693/ ) HBASE-5613 ThriftServer getTableRegions does not return serverName and port (Scott Chen) (Revision 1304602) Result = SUCCESS tedyu : Files : /hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java /hbase/trunk/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
      Hide
      Hudson added a comment -

      Integrated in HBase-0.94 #48 (See https://builds.apache.org/job/HBase-0.94/48/)
      HBASE-5613 ThriftServer getTableRegions does not return serverName and port (Scott Chen) (Revision 1304621)

      Result = SUCCESS
      tedyu :
      Files :

      • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java
      • /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
      Show
      Hudson added a comment - Integrated in HBase-0.94 #48 (See https://builds.apache.org/job/HBase-0.94/48/ ) HBASE-5613 ThriftServer getTableRegions does not return serverName and port (Scott Chen) (Revision 1304621) Result = SUCCESS tedyu : Files : /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java /hbase/branches/0.94/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServer.java
      Hide
      Phabricator added a comment -

      sc has closed the revision "HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port".

      REVISION DETAIL
      https://reviews.facebook.net/D2403

      To: dhruba, tedyu, JIRA, sc
      Cc: lhofhansl

      Show
      Phabricator added a comment - sc has closed the revision " HBASE-5613 [jira] ThriftServer getTableRegions does not return serverName and port". REVISION DETAIL https://reviews.facebook.net/D2403 To: dhruba, tedyu, JIRA, sc Cc: lhofhansl

        People

        • Assignee:
          Scott Chen
          Reporter:
          Scott Chen
        • Votes:
          0 Vote for this issue
          Watchers:
          4 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development