hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From li...@apache.org
Subject svn commit: r1590273 - /hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/regionserver/ThriftHRegionServer.java
Date Sat, 26 Apr 2014 18:18:35 GMT
Author: liyin
Date: Sat Apr 26 18:18:35 2014
New Revision: 1590273

URL: http://svn.apache.org/r1590273
Log:
[master] Fix recursion in ThriftHRegionServer

Author: elliott

Summary: ThriftHRegionServer had an infinte recursion.

Test Plan:
mvn clean test -Dtest=TestThriftServer
mvn clean test -Dtest=TestHFileHistogramE2E

Reviewers: adela, liyintang

Reviewed By: adela

CC: hbase-eng@, manukranthk

Differential Revision: https://phabricator.fb.com/D1297678

Task ID: 4223253

Modified:
    hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/regionserver/ThriftHRegionServer.java

Modified: hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/regionserver/ThriftHRegionServer.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/regionserver/ThriftHRegionServer.java?rev=1590273&r1=1590272&r2=1590273&view=diff
==============================================================================
--- hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/regionserver/ThriftHRegionServer.java
(original)
+++ hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/regionserver/ThriftHRegionServer.java
Sat Apr 26 18:18:35 2014
@@ -644,7 +644,11 @@ public class ThriftHRegionServer impleme
   @Override
   public List<List<Bucket>> getHistograms(List<byte[]> regionNames)
     throws ThriftHBaseException {
-    return this.getHistograms(regionNames);
+    try {
+      return server.getHistograms(regionNames);
+    } catch (IOException e) {
+      throw new ThriftHBaseException(e);
+    }
   }
 
   @Override



Mime
View raw message