Apache9 commented on PR #8057: URL: https://github.com/apache/hbase/pull/8057#issuecomment-4272414477
Reverting a PR is very common if we have some mistakes in the PR, we can apply it later after fixing the mistakes. At least we should not increase the test timeout limit, and also the load balancer change. And on the tmp directory fix, could you please paste the log sections which shows problems for sharing tmp directory? Or you can paste the AI chat log and the full log file it mentioned so we can review it. > Just to add a little more context, I started with a couple of fixes, but when the fix PR itself started hitting new flapper, I kept diagnosing and adding fixes to the same PR until I got a green build. I'd say this is common in HBase, that's why we have a flaky dashboard to show flaky tests. Starting a mini cluster in test is heavy and easy to have races when running on a slow machine... On the AI tool, Sonnet 4.5 is not powerful enough, Opus 4.6 will be bettter. And for debugging flaky tests, I recommend use Codex 5.4 xhigh since HBase is a project live for more than 15 years, we have lots of strange code in our code base, you need to think more and more before actually doing things... -- 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]
