-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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.
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
Fixes Join data was released
in grace_hash
join algorithm
#51382
Conversation
eeb2e4e
to
ba5049c
Compare
The "bug fix" category requires a test. |
This is an automated comment for commit aab86da with description of existing statuses. It's updated for the latest CI running
|
fd2170e
to
a45c53f
Compare
07167ee
to
0280106
Compare
@alexey-milovidov would you trigger this ci ? |
0280106
to
7a2d5b7
Compare
I've asked our engineer why the CI has not triggered automatically. |
Changes in tests/CI prohibit autostart by |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Such a great catch!
336f8e9
to
aab86da
Compare
@vdimir please help trigger the CI |
@vdimir There are 3 failures, seems not relate to this pr |
Right now, we are attempting to fix flaky tests in CI. PRs with any failures are blocked from being merged. We are currently collecting the fixes in the following pull request: #51494. Since this issue has been reproduced in stress tests, I might add the fixes from this PR to the PR addressing the failures, otherwise postpone the merging. |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Issue #50220 reports a core in
grace_hash
join. We finally reproduce the exception on local, and found that the issue is related to the failure of creating temporary file. Somehow this is triggered in #49816 #49483.When the exception happen, all the threads of the query process are not stopped immediately at the same time, there is still a thread trying to use the
hash_join
which has been released in the the thread with exceptions. So the core happens.Documentation entry for user-facing changes