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

CI Fixes, TSAN Fix #2833

Merged
merged 7 commits into from
Dec 23, 2021
Merged

CI Fixes, TSAN Fix #2833

merged 7 commits into from
Dec 23, 2021

Conversation

Mytherin
Copy link
Collaborator

Various CI fixes:

  • Turn on informational for the codecov project CI, which should (hopefully) prevent it from failing
  • Move regression test to a separate workflow and add a slightly increased threshold to guard against false positives
  • Restore OSX Debug workflow, it had been accidentally erased
  • Fix TSAN issue that was triggered with the pending query executor by turning the query profiler into a shared pointer and also moving the ownership of the query profiler into the executor. This fixes a potential issue that might happen with the profiler if a query was cancelled while background threads were processing the query.

@Mytherin Mytherin merged commit 3e55c44 into duckdb:master Dec 23, 2021
@Mytherin Mytherin deleted the tsanfailure branch January 17, 2022 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant