virajjasani commented on code in PR #5188:
URL: https://github.com/apache/hbase/pull/5188#discussion_r1171885295


##########
hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java:
##########
@@ -341,6 +355,14 @@ private static int getWarnResponseSize(Configuration conf) 
{
     return conf.getInt(WARN_RESPONSE_SIZE, DEFAULT_WARN_RESPONSE_SIZE);
   }
 
+  private static int getWarnResponseTimeScan(Configuration conf) {

Review Comment:
   nit: getWarnResponseTimeScan -> getWarnScanResponseTime



##########
hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java:
##########
@@ -341,6 +355,14 @@ private static int getWarnResponseSize(Configuration conf) 
{
     return conf.getInt(WARN_RESPONSE_SIZE, DEFAULT_WARN_RESPONSE_SIZE);
   }
 
+  private static int getWarnResponseTimeScan(Configuration conf) {
+    return conf.getInt(WARN_RESPONSE_TIME_SCAN, getWarnResponseTime(conf));
+  }
+
+  private static int getWarnResponseSizeScan(Configuration conf) {

Review Comment:
   nit: getWarnResponseSizeScan -> getWarnScanResponseSize



##########
hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java:
##########
@@ -198,6 +200,8 @@ public abstract class RpcServer implements 
RpcServerInterface, ConfigurationObse
   protected final int maxRequestSize;
   protected volatile int warnResponseTime;
   protected volatile int warnResponseSize;
+  protected volatile int warnResponseTimeScan;
+  protected volatile int warnResponseSizeScan;

Review Comment:
   nit: warnScanResponseTime and warnScanResponseSize



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to