bbeaudreault commented on a change in pull request #4180:
URL: https://github.com/apache/hbase/pull/4180#discussion_r839951489
##########
File path:
hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionConfiguration.java
##########
@@ -44,6 +51,23 @@
public static final int PRIMARY_SCAN_TIMEOUT_MICROSECOND_DEFAULT = 1000000;
// 1s
public static final String LOG_SCANNER_ACTIVITY =
"hbase.client.log.scanner.activity";
+ /**
+ * Parameter name for client pause when server is overloaded, denoted by
+ * {@link HBaseServerException#isServerOverloaded()}
+ */
+ public static final String HBASE_CLIENT_PAUSE_FOR_SERVER_OVERLOADED =
+ "hbase.client.pause.server.overloaded";
+
+ static {
+ // This is added where the configs are referenced. It may be too late to
happen before
+ // any user _sets_ the old cqtbe config onto a Configuration option. So we
still need
+ // to handle checking both properties in parsing below. The benefit of
calling this is
+ // that it should still cause Configuration to log a warning if we do end
up falling
+ // through to the old deprecated config.
+ Configuration.addDeprecation(
Review comment:
This is moot now, as of latest commit
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
