andygrove commented on PR #7000: URL: https://github.com/apache/datafusion-comet/pull/7000#issuecomment-6779574683
Thanks @comphead. I merged `main` (1ddb4fe91b). You are right that `CometArrowEvalPythonExec` in `spark-4.1+` still imported Guava's `Objects`, and a new operator that `main` added since used `Objects.hashCode` with three arguments, which `java.util.Objects` does not have. Both use `Objects.hash` now, and the default Spark 4.1 profile compiles. `CometTextNativeScanExec` is not on `main` yet, so whichever of that PR and this one lands second needs the one-line change. The description now says that the dependency check covers the compile and runtime scopes. -- 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]
