adriangb commented on PR #22706: URL: https://github.com/apache/datafusion/pull/22706#issuecomment-4602618225
Would you be open to creating an EPIC for the other types as well? Interval, Duration, Float16, Decimal256, Dictionary and FixedSizeBinary in particular seem like pretty common primitives that we should be able to handle. You bring up a good point with memory regression tests: would it make sense to also merge some [dhat](https://docs.rs/dhat/latest/dhat/) tests in one of the setup PRs so we can measure before / after memory usage? E.g. you write a test and set the limit to 8GB, then when you add support for a type lower that to 2GB. -- 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]
