Skip to content

Fix --hung-check in clickhouse-test#30218

Merged
alexey-milovidov merged 3 commits intoClickHouse:masterfrom
azat:clickhouse-test-fix-hung-check
Oct 17, 2021
Merged

Fix --hung-check in clickhouse-test#30218
alexey-milovidov merged 3 commits intoClickHouse:masterfrom
azat:clickhouse-test-fix-hung-check

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Oct 15, 2021

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

JSONEachRow cannot be parsed with a simple json.loads(), instead it
should be passed to json.loads() line by line.

CI: https://clickhouse-test-reports.s3.yandex.net/30189/a063097fdffc7ca9e9a4bad859dbddc5fa926220/stress_test_(address)/test_run.txt.out.log

Fixes: #30065

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Oct 15, 2021
@azat azat force-pushed the clickhouse-test-fix-hung-check branch from 79b0375 to 546562e Compare October 15, 2021 07:39
@alesapin alesapin self-assigned this Oct 15, 2021
azat added 3 commits October 16, 2021 17:59
JSONEachRow cannot be parsed with a simple json.loads(), instead it
should be passed to json.loads() line by line.

Fixes: ClickHouse#30065
@azat azat force-pushed the clickhouse-test-fix-hung-check branch from 546562e to 612a21c Compare October 16, 2021 15:01
@alexey-milovidov alexey-milovidov merged commit b253f61 into ClickHouse:master Oct 17, 2021
@azat azat deleted the clickhouse-test-fix-hung-check branch October 17, 2021 06:19
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.

5 participants