brijrajk commented on PR #12151: URL: https://github.com/apache/gluten/pull/12151#issuecomment-4922345911
The ClickHouse CI failure was a compile error in `GlutenBloomFilterFallbackSuite`: it imports `VeloxBloomFilterAggregate` (a Velox-only type), but the file lived in `gluten-ut/test` which is compiled by the ClickHouse CI as well. Fixed by moving the suite to `backends-velox/src/test/scala/` where all Velox-specific tests belong. Also rebased onto the latest main, which included a conflict with `CallerInfo.scala` (commit #12477 optimized `inBloomFilterStatFunctionCall` that our PR removes). Resolved by keeping our removal -- the new rule no longer needs the caller-stack check at all. -- 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]
