Wali05 commented on issue #91: URL: https://github.com/apache/datafusion-spark/issues/91#issuecomment-6862646902
I'd like to pick this one up if that's okay. I looked through `elt.rs` and see what you mean, it forces everything to Utf8 even when all the values are binary. I'm thinking of following what `concat` already does: keep Binary when every value argument is binary, in both `coerce_types` and the return type, and add slt cases that I'll check against Spark. I'd leave `eltOutputAsString` for the SparkOptions work in #92 so this stays small. Happy to adjust if you had something else in mind! -- 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]
