jaylmiller opened a new pull request, #5308: URL: https://github.com/apache/arrow-datafusion/pull/5308
# Which issue does this PR close? #5230 (does not close it.) # Rationale for this change The current merge benchmarks take MemoryExec as input. In #5230 we want to modify SortPreservingMergeExec and SortExec so that when row encoding is used within the sort node, it is preserved for usage by the merge node. Which means, we need bench cases on main for SortPreservingMergeExec when it has SortExec as input to see if the changes for #5230 are helping perf. # What changes are included in this PR? - Added cases with SortExec as input to the merge bench. - Fixed minor error I made in the sort bench. # Are these changes tested? # Are there any user-facing changes? -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
