namanjain24-sudo commented on PR #25191: URL: https://github.com/apache/datafusion/pull/25191#issuecomment-5647111953
Thanks, trimmed it. What I cut (the local test log, and the longer reasoning for leaving `SwitchExpression` alone) is written up in #25190 if it comes up in review. On the red check, I don't think it's this PR: it's `cargo test datafusion-cli`, which has no dependency on `datafusion-substrait`. MinIO withdrew `minio/minio` from Docker Hub on the 11th, so the storage integration tests panic on startup, and my branch is based on a commit from before #25216 moved that pull to quay.io. Rebasing onto main should clear it, happy to do that whenever, though it will probably need another workflow approval from you. -- 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]
