TheNeuralBit commented on pull request #16631: URL: https://github.com/apache/beam/pull/16631#issuecomment-1024805793
I'm hesitant to make this change without understanding what's going wrong. I took a look at the gradle scans you shared in the jira. In the old passing build, vendored calcite is brought in as a dependency of sql extensions as I'd expect:  In the current failing build, sql extensions has _no dependencies_:  So it seems like there's a larger issue here, and just bringing in one transitive dependency is just a band-aid. Unfortunately I'm not sure how to get to the root cause ... it seems likely this is related to the different configurations (provided vs. compileClasspath) due to the gradle 7 upgrade, but I don't know where to go from there. -- 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]
