anoopsjohn commented on pull request #1721: URL: https://github.com/apache/hbase/pull/1721#issuecomment-629200037
I think we dont even need this flushType state variable. The FlushHandler is anyway checking the type by calling the isAboveXXX() method (Even if waked up ). So we can pass in the flushType as local variable. Did not read so carefully. But at high level seems we can avoid the state. ---------------------------------------------------------------- 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]
