snazy commented on PR #1986: URL: https://github.com/apache/polaris/pull/1986#issuecomment-3095673836
I haven't noticed this PR before, probably because it's a draft. Gradle's the build tool for Java and would be, if ever, the build tool for Scala. The Python client is a separate tool, that's not built by Gradle, so I think that everything that's related to the Python client should be in purely in the build of the Python client and not in the Gradle build. The approach is neither cacheable Gradle nor does it have an up-to-date check, which does slow down the build. It also introduces an external process execution, that's unrelated to the Gradle build, requiring Python to be installed locally. -- 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]
