bert-beyondloops commented on issue #15205: URL: https://github.com/apache/datafusion/issues/15205#issuecomment-4507914367
I personally would go for the fix behind the compact() functionality on ScalarValue See https://github.com/apache/datafusion/issues/21928 In this way, the developer still has the option to just use the `ScalarValue::try_from_array` _as is_ without any side effects of recopying memory and possible performance impact. If reducing memory footprint is really needed, a compact() call can be made explicitly. -- 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]
