virajjasani commented on a change in pull request #754: HBASE-22978 : Online slow response log URL: https://github.com/apache/hbase/pull/754#discussion_r379973226
########## File path: hbase-common/src/main/resources/hbase-default.xml ########## @@ -1956,4 +1956,27 @@ possible configurations would overwhelm and obscure the important. enable this feature. </description> </property> + <property> + <name>hbase.regionserver.slowlog.ringbuffer.size</name> + <value>256</value> + <description> + Default size of ringbuffer to be maintained by each RegionServer in order + to store online slowlog responses. This is an in-memory ring buffer of + requests that were judged to be too slow in addition to the responseTooSlow + logging. The in-memory representation would be complete. + For more details, please look into Doc Section: + Get Slow Response Log from shell + </description> + </property> + <property> + <name>hbase.regionserver.slowlog.buffer.enabled</name> Review comment: The reason to maintain 2 configs is for user to just enable the feature without having to worry about what would be the reasonable size of ring buffer they would have to provide. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services