You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
I am trying to use the DataFusion Expr struct along with the expression simplification feature -- see #3708 and #3741 for examples / description of what this means
#3758 adds an API to call coerce so that simplify can work in more cases.
However, the API requires a DFSchema, some wrapper type over Schema that is non trivial to compute. The Simplify API already takes a trait SimplifyInfo to provide just the information required. It would be nice if the coercion function did too, and was called directly as part of the coercion API.
Describe the solution you'd like
I would like the signature of coerce addd in #3758 to take a SimplifyInfo rather than DFSchema
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
I am trying to use the DataFusion
Expr
struct along with the expression simplification feature -- see #3708 and #3741 for examples / description of what this meansType coercion is involved because of this chain:
#3758 adds an API to call coerce so that simplify can work in more cases.
However, the API requires a
DFSchema
, some wrapper type overSchema
that is non trivial to compute. The Simplify API already takes a traitSimplifyInfo
to provide just the information required. It would be nice if the coercion function did too, and was called directly as part of the coercion API.Describe the solution you'd like
I would like the signature of
coerce
addd in #3758 to take aSimplifyInfo
rather thanDFSchema
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
See https://github.com/influxdata/influxdb_iox/pull/5792 for examples of when IOx uses the simplification (and thus needs to call coerce)
The text was updated successfully, but these errors were encountered: