Skip to content

Fix CLICKHOUSE_TMP in tests (fixes broken CI)#38733

Merged
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
azat:fix-tests-tmp
Jul 2, 2022
Merged

Fix CLICKHOUSE_TMP in tests (fixes broken CI)#38733
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
azat:fix-tests-tmp

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Jul 2, 2022

Previous it was not updated, and always uses the first CLICKHOUSE_TMP
for all tests (that was run from one thread).

Thanks to #38728, that shows the problem.

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Cc: @alexey-milovidov

Previous it was not updated, and always uses the first CLICKHOUSE_TMP
for all tests (that was run from one thread).

Signed-off-by: Azat Khuzhin <[email protected]>
@robot-ch-test-poll robot-ch-test-poll added the pr-not-for-changelog This PR should not be mentioned in the changelog label Jul 2, 2022
@azat azat changed the title Fix CLICKHOUSE_TMP in tests Fix CLICKHOUSE_TMP in tests (fixes CI) Jul 2, 2022
@azat azat changed the title Fix CLICKHOUSE_TMP in tests (fixes CI) Fix CLICKHOUSE_TMP in tests (fixes broken CI) Jul 2, 2022
@alexey-milovidov alexey-milovidov self-assigned this Jul 2, 2022
@alexey-milovidov alexey-milovidov merged commit 14eed49 into ClickHouse:master Jul 2, 2022
@azat azat deleted the fix-tests-tmp branch July 2, 2022 19:05
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