Skip to content

Fix stress test after 36639#36660

Merged
kssenii merged 9 commits intoClickHouse:masterfrom
kssenii:fix-stress-test
Apr 29, 2022
Merged

Fix stress test after 36639#36660
kssenii merged 9 commits intoClickHouse:masterfrom
kssenii:fix-stress-test

Conversation

@kssenii
Copy link
Copy Markdown
Member

@kssenii kssenii commented Apr 26, 2022 •
edited
Loading

@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-not-for-changelog This PR should not be mentioned in the changelog label Apr 26, 2022
@kssenii kssenii mentioned this pull request Apr 26, 2022
Merged
@kssenii
Copy link
Copy Markdown
Member Author

kssenii commented Apr 27, 2022

@Mergifyio update

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 27, 2022

update

✅ Branch has been successfully updated

This was referenced Apr 27, 2022
Merged
Merged
Merged
This was referenced Apr 28, 2022
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
@kssenii kssenii force-pushed the fix-stress-test branch 2 times, most recently from f9eb2e2 to af30629 Compare April 28, 2022 11:06
This was referenced Apr 28, 2022
Merged
Merged
Merged
@tavplubix tavplubix self-assigned this Apr 28, 2022
@kssenii
Copy link
Copy Markdown
Member Author

kssenii commented Apr 28, 2022

Stress test (debug, actions) — Fatal message in clickhouse-server.log (see fatal_messages.txt)

😑

@alexey-milovidov alexey-milovidov mentioned this pull request Apr 28, 2022
Merged
@kssenii kssenii mentioned this pull request Apr 28, 2022
Merged
@kssenii
Copy link
Copy Markdown
Member Author

kssenii commented Apr 29, 2022

@Mergifyio update

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 29, 2022

update

✅ Branch has been successfully updated

@alexey-milovidov alexey-milovidov mentioned this pull request Apr 29, 2022
Merged
@azat azat mentioned this pull request Apr 29, 2022
Merged
@alesapin alesapin mentioned this pull request Apr 29, 2022
Merged
@kssenii
Copy link
Copy Markdown
Member Author

kssenii commented Apr 29, 2022

I'll fix hung check in a separate pr, it is a different case.

@kssenii kssenii merged commit ca994e0 into ClickHouse:master Apr 29, 2022
@tavplubix
Copy link
Copy Markdown
Member

tavplubix commented Apr 29, 2022

1cbf672 - Stress test (debug, actions):

/var/log/clickhouse-server/clickhouse-server.err.log:2022.04.29 03:05:45.319954 [ 4931 ] {} <Fatal> BaseDaemon: ########################################
/var/log/clickhouse-server/clickhouse-server.err.log:2022.04.29 03:05:45.437839 [ 4931 ] {} <Fatal> BaseDaemon: (version 22.5.1.1, build id: 4EA65F8EA4C114FE) (from thread 767) (query_id: 0b8eadbb-6941-4298-b6f0-455db58dfd0b::201403_1_7_1) (query: ) Received signal Aborted (6)
/var/log/clickhouse-server/clickhouse-server.err.log:2022.04.29 03:05:45.470491 [ 4931 ] {} <Fatal> BaseDaemon: 
/var/log/clickhouse-server/clickhouse-server.err.log:2022.04.29 03:05:45.502150 [ 4931 ] {} <Fatal> BaseDaemon: Stack trace: 0x7f8be8bb803b 0x7f8be8b97859 0x7f8be8b97729 0x7f8be8ba9006 0x258d1270 0x258d160f 0x26288b47 0x26286d7d 0x2628ecd3 0x17b73114 0x2620394d 0x17b73114 0x2620394d 0x17b73114 0x27ae927d 0x17b73114 0x27c9fb5d 0x27caaa28 0x27caae76 0x27e190f3 0x27e18ff1 0x27b999f1 0x27baf8b8 0x27baf875 0x27baf83d 0x27baf815 0x27baf7dd 0x25cba966 0x25cb2d35 0x27b99bb2 0x27b9961b 0x27f52b67 0x27bc2e0f 0x27bc4882 0x27bcbef8 0x27bcbeb5 0x27bcbe7d 0x27bcbe55 0x27bcbe1d 0x17bcdb06 0x17bca035
/var/log/clickhouse-server/clickhouse-server.err.log:2022.04.29 03:05:45.517957 [ 4931 ] {} <Fatal> BaseDaemon: 4. raise @ 0x7f8be8bb803b in ?
/var/log/clickhouse-server/clickhouse-server.err.log:2022.04.29 03:05:45.520307 [ 4931 ] {} <Fatal> BaseDaemon: 5. abort @ 0x7f8be8b97859 in ?
/var/log/clickhouse-server/clickhouse-server.err.log:2022.04.29 03:05:45.522046 [ 4931 ] {} <Fatal> BaseDaemon: 6. ? @ 0x7f8be8b97729 in ?
/var/log/clickhouse-server/clickhouse-server.err.log:2022.04.29 03:05:45.525047 [ 4931 ] {} <Fatal> BaseDaemon: 7. ? @ 0x7f8be8ba9006 in ?
/var/log/clickhouse-server/clickhouse-server.err.log:2022.04.29 03:05:45.985307 [ 4931 ] {} <Fatal> BaseDaemon: 8. /build/build_docker/../src/Common/FileSegment.cpp:657: DB::FileSegment::assertDetachedStatus(std::__1::lock_guard<std::__1::mutex>&) const @ 0x258d1270 in /usr/bin/clickhouse
/var/log/clickhouse-server/clickhouse-server.err.log:2022.04.29 03:05:46.198311 [ 4931 ] {} <Fatal> BaseDaemon: 9. /build/build_docker/../src/Common/FileSegment.cpp:718: DB::FileSegmentsHolder::~FileSegmentsHolder() @ 0x258d160f in /usr/bin/clickhouse

@tavplubix
Copy link
Copy Markdown
Member

tavplubix commented Apr 29, 2022

41aa85f does not actually contain any changes that might fix the bug, so looks like Stress tests are still broken and passed on 41aa85f by chance

@kssenii
Copy link
Copy Markdown
Member Author

kssenii commented Apr 29, 2022

Not really by chance #36743

@tavplubix tavplubix mentioned this pull request Apr 29, 2022
Closed
@qoega qoega mentioned this pull request Apr 29, 2022
Merged
This was referenced Apr 30, 2022
Merged
Merged
@azat azat mentioned this pull request Apr 30, 2022
Merged
@tavplubix tavplubix mentioned this pull request Apr 30, 2022
Merged
@Avogar Avogar mentioned this pull request May 2, 2022
Merged
@novikd novikd mentioned this pull request May 2, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tavplubix tavplubix tavplubix approved these changes

Assignees

@tavplubix tavplubix

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kssenii @tavplubix @robot-ch-test-poll1