ariel-miculas commented on issue #22548: URL: https://github.com/apache/datafusion/issues/22548#issuecomment-4548761792
Another idea: add an Option<additional_capacity> to split_vec_min_alloc: * when it's None, call shrink_to_fit * when it's Some(extra), shrink to `length + extra` instead of shrink_to_fit Of course we need to check each call site for subsequent push operations to make sure we're not causing unintended reallocations as a result of the shrink -- 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]
