Skip to content

Enable random CPU migrations around mutex ops#9813

Closed
alexey-milovidov wants to merge 16 commits intomasterfrom
thread-fuzzer-random-migrations-mutex
Closed

Enable random CPU migrations around mutex ops#9813
alexey-milovidov wants to merge 16 commits intomasterfrom
thread-fuzzer-random-migrations-mutex

Conversation

@alexey-milovidov
Copy link
Member

@alexey-milovidov alexey-milovidov commented Mar 22, 2020

Changelog category (leave one):

  • Build/Testing/Packaging Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Not for merge. Only to test how it will work in CI.

@alexey-milovidov alexey-milovidov added the pr-build Pull request with build/testing/packaging improvement label Mar 22, 2020
@qoega
Copy link
Member

qoega commented Mar 23, 2020

https://clickhouse-test-reports.s3.yandex.net/9813/622bca235848bf1a7950d6eeb10241204aa069eb/stress_test_(thread)/stderr.log

ThreadSanitizer: data race

    #6 DB::TablesBlockSource::generate() /build/obj-x86_64-linux-gnu/../dbms/src/Storages/System/StorageSystemTables.cpp:258:41 (clickhouse+0xf6a561d)

and

    #5 DB::DatabaseOnDisk::renameTable(DB::Context const&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, DB::IDatabase&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, DB::TableStructureWriteLockHolder&) /build/obj-x86_64-linux-gnu/../dbms/src/Databases/DatabaseOnDisk.cpp:229:16 (clickhouse+0xfbaa24d)

alexey-milovidov added a commit that referenced this pull request Mar 23, 2020
@alexey-milovidov
Fix flacky test #9813
ca1bb2f
@alexey-milovidov alexey-milovidov mentioned this pull request Jun 15, 2020
Merged
@alexey-milovidov alexey-milovidov mentioned this pull request Dec 21, 2020
Merged
@qoega
Copy link
Member

qoega commented May 14, 2021

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented May 14, 2021

Command update: success

Branch has been successfully updated

@nikitamikhaylov
Copy link
Member

nikitamikhaylov commented May 27, 2021

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented May 27, 2021

Command update: success

Branch has been successfully updated

@qoega
Copy link
Member

qoega commented Jun 12, 2021

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Jun 12, 2021

Command update: success

Branch has been successfully updated

@alexey-milovidov
Copy link
Member Author

alexey-milovidov commented Jun 29, 2021

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Jun 29, 2021

Command update: success

Branch has been successfully updated

@alexey-milovidov
Copy link
Member Author

alexey-milovidov commented Jul 13, 2021

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Jul 13, 2021

Command update: success

Branch has been successfully updated

@alexey-milovidov
Copy link
Member Author

alexey-milovidov commented Aug 8, 2021

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Aug 8, 2021

Command update: success

Branch has been successfully updated

@alexey-milovidov
Copy link
Member Author

alexey-milovidov commented Aug 8, 2021

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Aug 8, 2021

Command update: success

Branch has been successfully updated

@alexey-milovidov alexey-milovidov mentioned this pull request Aug 10, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

no-docs-needed pr-build Pull request with build/testing/packaging improvement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alexey-milovidov @qoega @nikitamikhaylov