Skip to content

Stateless flaky tests fixes#29947

Merged
alexey-milovidov merged 2 commits intoClickHouse:masterfrom
azat:stateless-tests-fixes-v21.10
Oct 10, 2021
Merged

Stateless flaky tests fixes#29947
alexey-milovidov merged 2 commits intoClickHouse:masterfrom
azat:stateless-tests-fixes-v21.10

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Oct 10, 2021

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

  • Make 00984_parser_stack_overflow less flaky (increased timeout, but verified in trace_log, unwind really takes too much time)
  • Make 01085_max_distributed_connections less flaky

azat added 2 commits October 10, 2021 09:57
According to query_log/trace_log the problem is real:

    2021.10.10 07:52:37.702364 [ 3736 ] {c6a219f2-3df8-4474-8324-bb307ee7a7a9} <Debug> executeQuery: (from [::1]:38096) (comment: /usr/share/clickhouse-test/queries/0_stateless/00984_parser_stack_overflow.sh) SELECT [[[...
    2021.10.10 07:53:11.407949 [ 3736 ] {c6a219f2-3df8-4474-8324-bb307ee7a7a9} <Error> executeQuery: Code: 306. DB::Exception: Maximum parse depth (1000) exceeded. Consider rising max_parser_depth ...

From trace_log:

    2021-10-10 07:52:37.958938 DB::Dwarf::findDebugInfoOffset
    2021-10-10 07:53:12.583248 DB::Dwarf::findDebugInfoOffset

CI: https://clickhouse-test-reports.s3.yandex.net/29928/2bab572caf91d2fc8962f409fea34149ecec8c95/functional_stateless_tests_(debug).html#fail1
@azat azat force-pushed the stateless-tests-fixes-v21.10 branch from 9f7ec4d to 0c5d26b Compare October 10, 2021 06:57
@alexey-milovidov alexey-milovidov self-assigned this Oct 10, 2021
@alexey-milovidov alexey-milovidov merged commit c64ca7b into ClickHouse:master Oct 10, 2021
@azat azat deleted the stateless-tests-fixes-v21.10 branch October 10, 2021 22:16
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