[ https://issues.apache.org/jira/browse/ARROW-10165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17210151#comment-17210151 ]
Andrew Lamb commented on ARROW-10165: ------------------------------------- Per comments on the PR, we have decided on a different approach here. I expect the code will be done under the aegis of https://issues.apache.org/jira/browse/ARROW-10163. Closing this one for now > [Rust] [DataFusion] Allow DataFusion to cast all type combinations supported > by Arrow cast kernel > ------------------------------------------------------------------------------------------------- > > Key: ARROW-10165 > URL: https://issues.apache.org/jira/browse/ARROW-10165 > Project: Apache Arrow > Issue Type: Sub-task > Reporter: Andrew Lamb > Assignee: Andrew Lamb > Priority: Minor > Labels: pull-request-available > Time Spent: 2h > Remaining Estimate: 0h > > When the DataFusion planner inserts casts, today it relies on special logic > to determine the valid coded casts. > The actual arrow cast kernels support a much wider range of data types, and > thus DataFusion is artificially limiting the casts it supports for no > particularly good reason I can see. > This ticket tracks the work to remove the extra casting checking in the > datafusion planner and instead simply rely on runtime check of arrow cast > compute kernel > The potential downside of this approach is that the error may be generated > later in the execution process (rather than the planner), and possibly have a > less specific error message, the upside is there is less code and we get > several conversions immediately (like timestamp predicate casting) > I also plan to add DictionaryArray support to the casting kernels and I would > like to avoid having to replicate some part of that logic in DataFusion -- This message was sent by Atlassian Jira (v8.3.4#803005)