You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "niyue (via GitHub)" <gi...@apache.org> on 2023/12/02 14:37:04 UTC

[PR] GH-39045: [C++][Acero] union node output batches should be unordered [arrow]

niyue opened a new pull request, #39046:
URL: https://github.com/apache/arrow/pull/39046

   ### Rationale for this change
   Acero's union node produce duplicated batch index if having multiple ordered input nodes, which makes down stream nodes unable to process these batches if ordering is a concern. This PR tries to address this issue.
   
   ### What changes are included in this PR?
   This PR fixes this issue by setting the index to unspecified if the order cannot be guaranteed.
   
   ### Are these changes tested?
   Yes
   
   ### 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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] GH-39045: [C++][Acero] union node output batches should be unordered [arrow]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #39046:
URL: https://github.com/apache/arrow/pull/39046#issuecomment-1837163049

   :warning: GitHub issue #39045 **has been automatically assigned in GitHub** to PR creator.


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] GH-39045: [C++][Acero] union node output batches should be unordered [arrow]

Posted by "conbench-apache-arrow[bot] (via GitHub)" <gi...@apache.org>.
conbench-apache-arrow[bot] commented on PR #39046:
URL: https://github.com/apache/arrow/pull/39046#issuecomment-1843819882

   After merging your PR, Conbench analyzed the 6 benchmarking runs that have been run so far on merge-commit fd44a332290229d1dfff5a0f599d82fdc8e276bb.
   
   There were no benchmark performance regressions. 🎉
   
   The [full Conbench report](https://github.com/apache/arrow/runs/19390215144) has more details.


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] GH-39045: [C++][Acero] union node output batches should be unordered [arrow]

Posted by "westonpace (via GitHub)" <gi...@apache.org>.
westonpace merged PR #39046:
URL: https://github.com/apache/arrow/pull/39046


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org