Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

colexec: close the ordered sync used by the external sorter #75771

Merged
merged 2 commits into from
Feb 2, 2022
Merged

colexec: close the ordered sync used by the external sorter #75771

merged 2 commits into from
Feb 2, 2022

Commits on Feb 1, 2022

  1. colexec: close the ordered sync used by the external sorter 

    Previously, we forgot to close the ordered synchronizer that is used by
the external sorter to merge already sorted partitions. This could
result in some tracing spans never being finished and is now fixed.

Release note: None
    @yuzefovich
    yuzefovich committed Feb 1, 2022
    Configuration menu
    Copy the full SHA
    b424913 View commit details
    Browse the repository at this point in the history

  2. colexec: return an error rather than logging it on Close in some cases 

    This error eventually will be logged anyway, but we should try to
propagate it up the stack as much as possible.

Release note: None
    @yuzefovich
    yuzefovich committed Feb 1, 2022
    Configuration menu
    Copy the full SHA
    91abc48 View commit details
    Browse the repository at this point in the history