bharathv commented on a change in pull request #754: HBASE-22978 : Online slow response log URL: https://github.com/apache/hbase/pull/754#discussion_r379692739
########## 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: Why two configs? if "hbase.regionserver.slowlog.ringbuffer.size" > 0, it means it is enabled..(in the interest of reducing the number of knobs) ---------------------------------------------------------------- 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