Apache9 commented on PR #8057: URL: https://github.com/apache/hbase/pull/8057#issuecomment-4264674068
Please revert this PR because it is not qualified enough. Wo do not prevent using AI coding tool but at least you should review the code by yourself and be resposible for the code. For example, increasing the timeout value is not the correct way here, you should make the test smaller instead of increasing the timeout. And for load balancer, changing the stop time calculation changes the behavior, we should discuss first before changing the behavior. And on the directory sharing, the method is only called when creating hadoop.log.dir and hadoop.tmp.dir directories, can you confirm that the failures on CI are caused by sharing these two directories? We need evidence, not only the conclusion from AI. And flaky rerun has been a problem for a long time, I've also seen the reruns all failed because of mini cluster already started. But we should have common solution for all tests, not patching a single test. And no matter with or without AI, you should split this PR to several small ones since they are different problems. -- 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]
