Skip to content

Conversation

@BipulLamsal
Copy link

Which issue does this PR close?

What changes are included in this PR?

Deprecated 'ExprSchemable::metadata' method
Use to_field and convert the result with FieldMetadata::from

@github-actions github-actions bot added the logical-expr Logical plan and expressions label Nov 28, 2025
Copy link
Member

@martin-g martin-g left a comment

Choose a reason for hiding this comment

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

I like the old API better.
But I am just a user! The DataFusion team has to decide.

Co-authored-by: Martin Grigorov <martin-g@users.noreply.github.com>
@alamb
Copy link
Contributor

alamb commented Nov 29, 2025

I like the old API better. But I am just a user! The DataFusion team has to decide.

I agree this change makes the code harder to use

I wonder if there is some way to make FieldMetadata::from(expr) work (`impl From for &Field)?

I think the fact that to_field actually returns a (relation, field) makes using it akward

@alamb
Copy link
Contributor

alamb commented Nov 29, 2025

Thank you @BipulLamsal and @martin-g

@BipulLamsal
Copy link
Author

BipulLamsal commented Nov 30, 2025

I wonder if there is some way to make FieldMetadata::from(expr) work (`impl From for &Field)?

let me look this up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

logical-expr Logical plan and expressions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deprecate ExprSchemable::metadata

3 participants