Skip to content

Fix flaky integration tests (test_backup_restore/test_input_format_parallel_parsing_memory_tracking)#29919

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

Fix flaky integration tests (test_backup_restore/test_input_format_parallel_parsing_memory_tracking)#29919
alexey-milovidov merged 2 commits intoClickHouse:masterfrom
azat:integration-tests-fixes-v21.10

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Oct 9, 2021

Checklist:

  • Fixes test_backup_restore after Merging #29376 #29649 (Cc: @CurtizJ)
  • Fixes test_input_format_parallel_parsing_memory_tracking/test.py::test_memory_tracking_total

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Oct 9, 2021
…racking_total

Server has pretty low memory limit 3GB, so let's use local to generate
the data.
@azat azat changed the title Fix test_backup_restore after #29649 Fix flaky integration tests Oct 9, 2021
@azat
Copy link
Copy Markdown
Member Author

azat commented Oct 9, 2021

Integration tests (asan) — fail: 1, passed: 1588, flaky: 1
Integration tests (release) — fail: 1, passed: 1588, flaky: 3
Integration tests (thread) — fail: 2, passed: 1587, flaky: 2

@azat azat changed the title Fix flaky integration tests Fix flaky integration tests (test_backup_restore/test_input_format_parallel_parsing_memory_tracking) Oct 9, 2021
@alexey-milovidov alexey-milovidov self-assigned this Oct 10, 2021
@alexey-milovidov
Copy link
Copy Markdown
Member

alexey-milovidov commented Oct 10, 2021

Ok. I did not understand the details though.

@alexey-milovidov alexey-milovidov merged commit 765f7b6 into ClickHouse:master Oct 10, 2021
@azat azat deleted the integration-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