kbendick commented on PR #4638: URL: https://github.com/apache/iceberg/pull/4638#issuecomment-1112898207
Another thought: This runs as part of `gradlew check`, so we don’t technically need to update CI. However, we might want to add this as its CI step eventually. The logic being that this check is relatively inexpensive, and so it could be placed first alongside the PR labeler. Then, if API compatibility fails, we could not run all of the expensive tests. I wouldn’t add it as a standalone check just yet because it will get run as part of `gradle check`, but something to think about. -- 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]
