hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From te...@apache.org
Subject svn commit: r1293013 - in /hbase/trunk/src/main/java/org/apache/hadoop/hbase: thrift/HbaseHandlerMetricsProxy.java thrift/ThriftMetrics.java thrift2/ThriftHBaseServiceHandler.java
Date Thu, 23 Feb 2012 22:58:42 GMT
Author: tedyu
Date: Thu Feb 23 22:58:42 2012
New Revision: 1293013

URL: http://svn.apache.org/viewvc?rev=1293013&view=rev
Log:
HBASE-5437 HRegionThriftServer does not start because of a bug in HbaseHandlerMetricsProxy
(Scott Chen)

Modified:
    hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/HbaseHandlerMetricsProxy.java
    hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/ThriftMetrics.java
    hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift2/ThriftHBaseServiceHandler.java

Modified: hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/HbaseHandlerMetricsProxy.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/HbaseHandlerMetricsProxy.java?rev=1293013&r1=1293012&r2=1293013&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/HbaseHandlerMetricsProxy.java
(original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/HbaseHandlerMetricsProxy.java
Thu Feb 23 22:58:42 2012
@@ -46,7 +46,7 @@ public class HbaseHandlerMetricsProxy im
                                         Configuration conf) {
     return (Hbase.Iface) Proxy.newProxyInstance(
         handler.getClass().getClassLoader(),
-        handler.getClass().getInterfaces(),
+        new Class[]{Hbase.Iface.class},
         new HbaseHandlerMetricsProxy(handler, metrics, conf));
   }
 

Modified: hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/ThriftMetrics.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/ThriftMetrics.java?rev=1293013&r1=1293012&r2=1293013&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/ThriftMetrics.java (original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift/ThriftMetrics.java Thu Feb 23
22:58:42 2012
@@ -115,6 +115,7 @@ public class ThriftMetrics implements Up
   }
 
   private void createMetricsForMethods(Class<?> iface) {
+    LOG.debug("Creating metrics for interface " + iface.toString());
     for (Method m : iface.getDeclaredMethods()) {
       if (getMethodTimeMetrics(m.getName()) == null)
         LOG.debug("Creating metrics for method:" + m.getName());

Modified: hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift2/ThriftHBaseServiceHandler.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift2/ThriftHBaseServiceHandler.java?rev=1293013&r1=1293012&r2=1293013&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift2/ThriftHBaseServiceHandler.java
(original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/thrift2/ThriftHBaseServiceHandler.java
Thu Feb 23 22:58:42 2012
@@ -81,7 +81,7 @@ public class ThriftHBaseServiceHandler i
     THBaseService.Iface handler = new ThriftHBaseServiceHandler(conf);
     return (THBaseService.Iface) Proxy.newProxyInstance(
         handler.getClass().getClassLoader(),
-        handler.getClass().getInterfaces(),
+        new Class[]{THBaseService.Iface.class},
         new THBaseServiceMetricsProxy(handler, metrics));
   }
 



Mime
View raw message