Skip to content

prepare for datafusion 53#108

Closed
friendlymatthew wants to merge 2 commits intodatafusion-contrib:mainfrom
friendlymatthew:53
Closed

prepare for datafusion 53#108
friendlymatthew wants to merge 2 commits intodatafusion-contrib:mainfrom
friendlymatthew:53

Conversation

@friendlymatthew
Copy link
Copy Markdown
Member

@friendlymatthew friendlymatthew commented Mar 20, 2026

This uses pydantic-main which is the latest changes on datafusion + a couple of cherry picked commits used in Pydantic's vendored datafusion

Note: one of those cherry picks fixes apache/datafusion#10180!

…down

This allows the ExtractLeafExpressions optimizer to push JSON accessor
functions (json_as_text, json_get, etc.) into the Parquet scan projection,
where custom PhysicalExprAdapters can rewrite them for shredded/reduced
column fallback.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Comment thread tests/main.rs
#[tokio::test]
async fn test_json_get_equals() {
let e = run_query(r"select name, json_get(json_data, 'foo')='abc' from test")
// union comparison now works thanks to the union coercions upport in datafusion
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// union comparison now works thanks to the union coercions upport in datafusion
// union comparison now works thanks to the union coercion support in datafusion

@adriangb adriangb closed this Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Cannot infer common argument type for comparison operation Union..."

2 participants