yihua commented on PR #6151: URL: https://github.com/apache/hudi/pull/6151#issuecomment-1193065573
As we discussed, there is a risk of landing this if there are any changes on the bundles at this point. Before landing the PR: (1) we should try to avoid any dependency change for production code and bundling. Adjusting dependency for tests is ok and should be limited to tests only. We shouldn't change the compile pom for merely fixing tests. (2) for any disabled tests in Azure CI, try to find a way to run them in Github CI to maintain the coverage. (3) make sure root pom changes for switching profiles do not change any behavior for building all bundles. -- 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]
