sunchao commented on PR #6427: URL: https://github.com/apache/datafusion-comet/pull/6427#issuecomment-6281709831
@viirya Applied your suggested `CometProjectExec.stringArgs` change in a54320494: `projectList` now uses the shared `summarizeBloomLiterals` helper for plan strings, formatted EXPLAIN and SQL UI descriptions. Added a real Parquet projection regression alongside the filter case. It compares results with Spark, requires a native `CometProjectExec`, checks bounded display strings and verifies that the executable expression retains the full Bloom payload. Follow-up 7b7715cb5 makes the UI assertions compatible with older Spark versions that omit descriptions from DOT output. One remaining scope limit: existing output/alias names are preserved, so an automatically generated name that already embeds the literal can still be large. The projection test uses an explicit alias, and the PR description documents this limitation and the broader upstream Spark formatting follow-up. [CI passed at 7b7715cb5](https://github.com/apache/datafusion-comet/actions/runs/39348677890), including all six `CometFilterDisplaySuite` tests on Spark 4.1.3, Spark 4.1 SQL suites and Spark 3.5 strict compilation. -- 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]
