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.

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

GH-35868: [C++] Occasional TSAN failure on asof-join-node-test #35904

Merged
merged 1 commit into from
Jun 6, 2023

Conversation

rtpsw
Copy link
Contributor

@rtpsw rtpsw commented Jun 4, 2023

Rationale for this change

AsofJoinNode may run into a data race when invalidating the key hasher.

The key hasher queried from one thread but invalidated from another. This might be simplified so that the key hasher would only be used from one thread, but this is out of scope for this PR.

What changes are included in this PR?

The invalidated member of the key hasher is made atomic.

Are these changes tested?

Yes, by existing testing.

Are there any user-facing changes?

No.

@github-actions
Copy link

github-actions bot commented Jun 4, 2023

⚠️ GitHub issue #35868 has been automatically assigned in GitHub to PR creator.

@rtpsw
Copy link
Contributor Author

rtpsw commented Jun 5, 2023

cc @pitrou

@github-actions
Copy link

github-actions bot commented Jun 6, 2023

⚠️ GitHub issue #35868 has been automatically assigned in GitHub to PR creator.

@pitrou
Copy link
Member

pitrou commented Jun 6, 2023

Adjusted your PR description to add the explanation you gave in 380aa94#r1218255714

@pitrou
Copy link
Member

pitrou commented Jun 6, 2023

@github-actions crossbow submit test-ubuntu-20.04-cpp-thread-sanitizer

@github-actions github-actions bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Jun 6, 2023
@github-actions
Copy link

github-actions bot commented Jun 6, 2023

Revision: 0ceff4d

Submitted crossbow builds: ursacomputing/crossbow @ actions-487d9b0cb0

Task Status
test-ubuntu-20.04-cpp-thread-sanitizer Github Actions

@pitrou pitrou merged commit c253625 into apache:main Jun 6, 2023
@rtpsw rtpsw deleted the GH-35868 branch June 6, 2023 09:33
@ursabot
Copy link

ursabot commented Jun 6, 2023

Benchmark runs are scheduled for baseline = e2ae492 and contender = c253625. c253625 is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Failed] ec2-t3-xlarge-us-east-2
[Failed ⬇️0.39% ⬆️0.03%] test-mac-arm
[Failed] ursa-i9-9960x
[Failed ⬇️0.33% ⬆️0.06%] ursa-thinkcentre-m75q
Buildkite builds:
[Failed] c2536253 ec2-t3-xlarge-us-east-2
[Failed] c2536253 test-mac-arm
[Failed] c2536253 ursa-i9-9960x
[Failed] c2536253 ursa-thinkcentre-m75q
[Failed] e2ae492b ec2-t3-xlarge-us-east-2
[Failed] e2ae492b test-mac-arm
[Failed] e2ae492b ursa-i9-9960x
[Failed] e2ae492b ursa-thinkcentre-m75q
Supported benchmarks:
ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
test-mac-arm: Supported benchmark langs: C++, Python, R
ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[C++] Occasional TSAN failure on asof-join-node-test
3 participants