alamb opened a new issue, #11881:
URL: https://github.com/apache/datafusion/issues/11881

   ### Is your feature request related to a problem or challenge?
   
   In https://github.com/apache/datafusion/pull/11796 @dharanad  added a rule 
for binary operators such that if `Utf8View` is in any side, we coerce to 
`Utf8`.
   
   I think it would be better to coerce to Utf8View as that coercsion will 
often be faster (it is faster to cast Utf8 -> Utf8View than the other way 
around)
   
   @XiangpengHao  notes: 
https://github.com/apache/datafusion/pull/11796#discussion_r1705034145
   
   > Agree, similar to this policy: 
   
   
https://github.com/apache/datafusion/blob/b50887f00a1e6009d5ee01c9194ac5ea3269ed8b/datafusion/expr/src/type_coercion/binary.rs#L935-L947
   
   
   
   ### Describe the solution you'd like
   
   Cast to Utf8View rather than Utf8 in the aforementioned code
   
   ### Describe alternatives you've considered
   
   _No response_
   
   ### Additional context
   
   _No response_


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to