malinjawi commented on PR #11924: URL: https://github.com/apache/gluten/pull/11924#issuecomment-4294918893
> Hi @malinjawi, thanks for fixing the CI coverage. I noticed -P delta was once in the x86 yml but then removed in this commit [ace29ec](https://github.com/apache/gluten/commit/ace29ecaf301aa83899c5d223ac3359d36eda14a) of the PR, was there any specific reason? Hey @zhztheplayer yes that was a mistake form my end I thought that like iceberg we should add the delta tests to the [.github/workflows/velox_backend_enhanced.yml](https://github.com/apache/gluten/pull/11924/changes#diff-bd7e2fd4528331f36fc3940415cb692ab288db9a3ba3ee2b8cb21e446c5b26c2). Let me know if i should change this or revert. -- 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]
