NihalJain commented on PR #6258: URL: https://github.com/apache/hbase/pull/6258#issuecomment-2362001487
Commit https://github.com/apache/hbase/pull/6258/commits/d6d6246362d834afebcbbc55647d9cd139dcbd99 details: - Removed hbase-diagnostics test jar dependency from hbase-it - Updated ITs to have local util for all shares data/confs - Cleaned up other classes in similar manner - Fix test case failure due to log4j - Also added WALPerformanceEvaluation tool which was missed in previous change - Renamed DiagnosticToolsCommonUtils to KerberosUtils - Moved MockRegionServerServices to corresponding main code as needed by WALPerformanceEvaluation - Copied method required by WALPerformanceEvaluation to WALPerformanceEvaluationUtil (draft, need to see can we do any better here) - Added a new class out of TestFSHLogProvider and extracted only test dependent on WALPerformanceEvaluation This PR may require multiple round of review, thank you for your patience. Should we also create a new module for chaos? I was thinking of doing it as another task of HBASE-28431, if needed? Or else can do i her itself, just that this PR will keep getting larger and larger -- 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: issues-unsubscr...@hbase.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
