virajjasani commented on pull request #2261:
URL: https://github.com/apache/hbase/pull/2261#issuecomment-675434004


   I believe we should have generic Admin API for similar use-cases. For 
existing slowLog use-case, I can make it default implementation pointing to 
generic API and provide deprecation notice (with removal in 4.0.0 release).
   
   1. Addressed concerns other than providing generic Admin API for ring buffer 
use-cases.
   2. Let me take that up now and add as separate commit so that it becomes 
easier to review commit by commit.
   3. I can take up thrift changes as a follow up? (to avoid too many changes 
going in same 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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to