FrankChen021 commented on PR #19911: URL: https://github.com/apache/druid/pull/19911#issuecomment-5210258018
> imo a 12k line PR is kind of hard to review, not to mention disruptive to any open PR that touches a test in the entire module (this is part of why i have broken up druid-processing into several smaller migrations) You are right. I didn't expect such a large change. I already asked Codex to analyze and split changes into multiple PRs based on modules. Codex is still monitoring the CI status of these PRs and will apply fixes automatically until CI is green. After that let's split the changes into multiple PRs again. -- 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]
