Skip to content

tests: fix 00705_drop_create_merge_tree flakiness#42522

Merged
tavplubix merged 2 commits intoClickHouse:masterfrom
azat:tests/00705_drop_create_merge_tree
Oct 20, 2022
Merged

tests: fix 00705_drop_create_merge_tree flakiness#42522
tavplubix merged 2 commits intoClickHouse:masterfrom
azat:tests/00705_drop_create_merge_tree

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Oct 20, 2022

CI caught tiemout for this test 1:

2022.10.19 16:43:46.238264 [ 24048 ] {aea0ff2a-f8de-498a-bd9f-0b8069a49f48} <Error> executeQuery: Code: 60. DB::Exception: Table test_orfkwn0y.table doesn't exist. (UNKNOWN_TABLE) (version 22.10.1.1) (from [::1]:60028) (comment: 00705_drop_create_merge_tree.sh) (in query: DROP TABLE table), Stack trace (when copying this message, always include the lines below):
...
2022.10.19 16:53:34.484777 [ 24042 ] {aec5a80a-4492-429b-87fb-7dbf5ffb5d67} <Debug> executeQuery: (from [::1]:57944) (comment: 00705_drop_create_merge_tree.sh) DROP DATABASE test_orfkwn0y  (stage: Complete)

But as you can see there is huge delay between last query from the test and final DROP DATABASE.

Apparently it is the same issue in bash 1.

Anyway it is easier to simply invoke clickhouse-client only two times, since each invocation is very slow (~1-2 sec) in debug build.

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Cc: @alexey-milovidov
Cc: @tavplubix (he also bumped into this issue)

CI caught tiemout for this test [1]:

    2022.10.19 16:43:46.238264 [ 24048 ] {aea0ff2a-f8de-498a-bd9f-0b8069a49f48} <Error> executeQuery: Code: 60. DB::Exception: Table test_orfkwn0y.table doesn't exist. (UNKNOWN_TABLE) (version 22.10.1.1) (from [::1]:60028) (comment: 00705_drop_create_merge_tree.sh) (in query: DROP TABLE table), Stack trace (when copying this message, always include the lines below):
    ...
    2022.10.19 16:53:34.484777 [ 24042 ] {aec5a80a-4492-429b-87fb-7dbf5ffb5d67} <Debug> executeQuery: (from [::1]:57944) (comment: 00705_drop_create_merge_tree.sh) DROP DATABASE test_orfkwn0y  (stage: Complete)

But as you can see there is huge delay between last query from the test
and final DROP DATABASE.

  [1]: https://s3.amazonaws.com/clickhouse-test-reports/42457/65cd040d1565bb7b2a9ba515041c3a139d31a4f9/stateless_tests__tsan__[1/3]/runlog.log

Apparently it is the same issue in bash [1].

  [1]: https://gist.github.com/azat/affbda3f8c6b5c38648d4ab105777d88

Anyway it is easier to simply invoke clickhouse-client only two times,
since each invocation is very slow (~1-2 sec) in debug build.

Signed-off-by: Azat Khuzhin <[email protected]>
@robot-ch-test-poll2 robot-ch-test-poll2 added the pr-not-for-changelog This PR should not be mentioned in the changelog label Oct 20, 2022
@tavplubix tavplubix self-assigned this Oct 20, 2022
@tavplubix
Copy link
Copy Markdown
Member

Integration tests (release) [2/2] - #39185
Stateless tests flaky check (asan) - Test runs too long - OK
Stress test (asan) - #41218
Stress test (msan) - too slow MergeTreeData::clearOldPartsFromFilesystem on shutdown
Stress test (tsan) - The specified key does not exist in BC check

@tavplubix tavplubix merged commit 46559d6 into ClickHouse:master Oct 20, 2022
@azat azat deleted the tests/00705_drop_create_merge_tree branch October 24, 2022 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants