notfilippo commented on PR #11978:
URL: https://github.com/apache/datafusion/pull/11978#issuecomment-2293500975

   > Given that we have ScalarValue::Utf8, and we have StringView in schema. We 
can then get the corresponding StrginViewArray.
   
   I feel like this is a little bit more complex because casting a scalar value 
to a logically equal type (`arrow_cast('test', 'Dictionary(Int32, Utf8)')`) is 
a no-op for all cases but one the cast is at the top level, which indicates 
that you actually want an __array__ of that type. So we need a way to 
distinguish those two scenarios, which is why I've added this:
   
   
https://github.com/apache/datafusion/blob/78dc034d4b7d644e6c34745286e82a6332dd7606/datafusion/physical-plan/src/projection.rs#L312
   
   But I'm not really sure about it...


-- 
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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to