taklwu commented on a change in pull request #4123:
URL: https://github.com/apache/hbase/pull/4123#discussion_r813158344
##########
File path: hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java
##########
@@ -94,6 +94,8 @@
static final String REST_CSRF_METHODS_TO_IGNORE_DEFAULT =
"GET,OPTIONS,HEAD,TRACE";
public static final String SKIP_LOGIN_KEY = "hbase.rest.skip.login";
static final int DEFAULT_HTTP_MAX_HEADER_SIZE = 64 * 1024; // 64k
+ static final String HTTP_HEADER_CACHE_SIZE =
"hbase.rest.jetty.httpConfig.headerCacheSize";
Review comment:
[nit] maybe use non camel case? although it's very long.
```suggestion
static final String HTTP_HEADER_CACHE_SIZE =
"hbase.rest.http.header.cache.size";
```
--
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]
