wForget commented on PR #5815: URL: https://github.com/apache/kyuubi/pull/5815#issuecomment-1840668810
> Still have to see more evidence for repeated tests. Please consider merge #5809 to introduce benchmark with warmups and repeating rounds, for the reference of further investigation. I think this is an obvious improvement. The benchmark unit test is controversial and not required, we can remove it first and continue reviewing the PR. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
