pepijnve opened a new pull request, #8583: URL: https://github.com/apache/arrow-rs/pull/8583
# Which issue does this PR close? Not logged yet - Closes #NNN. # Rationale for this change RecordBatch project currently uses the validating factory function. Since project starts from a valid RecordBatch these checks are redundant. A small amount of work can be saved by using `new_unchecked` instead. A change I'm working on for DataFusion uses `RecordBatch#project` in the inner expression evaluation loop to reduce the amount of redundant array filtering `case` expressions need to do. While a micro optimisation, avoiding redundant work in inner loops seems worthwhile. # What changes are included in this PR? - Use `new_unchecked` instead of `try_new_with_options` in `RecordBatch#project` # Are these changes tested? No additional tests added. # Are there any user-facing changes? No -- 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]
