Apache9 commented on PR #7291: URL: https://github.com/apache/hbase/pull/7291#issuecomment-3384411907
The change is pretty large, I suggest we start a feature branch to land it step by step. First we introduce the framework for integrating row cache, add a flag to enable/disable it, but the code when enabling row cache can be empty. Then we implement all the necessary fencing code, introduce a row cache interface and a very simple row cache implementation, to verify the correctness. And last, we introduce a more powerful implementation for row cache, which has a good performance. And then we can start some integration tests, like YCSB to verify performance, and ITBLL to verify correctness, if all things are good, we can merge the feature branch back. WDYT? Thanks. -- 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]
