findepi commented on issue #12622: URL: https://github.com/apache/datafusion/issues/12622#issuecomment-2668739537
> The type coercion in LP only take care about NativeType::String and we end up with `DataType::Utf8` and `DataType::Utf8View`. We consider these resolved types in LP. If LP only cares about "native types" (aka logical types), then it doesn't know & doesn't care whether later it will be Utf8 or Utf8View. W can think of "logical types" as just an equivalence relation over physical types, but that's not how the code & abstractions should look like. So, for this issue to be complete, we need the LP to be "decoupled" from physical types. I.e not use arrow DataType at all. -- 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]
