Skip to content

Error from allocator on JSON import#20286

Merged
nikitamikhaylov merged 8 commits intoClickHouse:masterfrom
nikitamikhaylov:json-import-bugfix
Feb 15, 2021
Merged

Error from allocator on JSON import#20286
nikitamikhaylov merged 8 commits intoClickHouse:masterfrom
nikitamikhaylov:json-import-bugfix

Conversation

@nikitamikhaylov
Copy link
Copy Markdown
Member

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fixed the behavior when in case of broken JSON we tried to read the whole file into memory which leads to exception from the allocator. Fixes #19719

Nikita Mikhailov added 2 commits February 10, 2021 17:16
@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Feb 10, 2021
@tavplubix tavplubix self-assigned this Feb 11, 2021
@nikitamikhaylov
Copy link
Copy Markdown
Member Author

test_testkeeper_back_to_back - new flaky test, will be fixed soon.

@nikitamikhaylov nikitamikhaylov merged commit a77b740 into ClickHouse:master Feb 15, 2021
nikitamikhaylov added a commit that referenced this pull request Feb 15, 2021
Backport #20286 to 21.1: Error from allocator on JSON import
nikitamikhaylov added a commit that referenced this pull request Feb 15, 2021
Backport #20286 to 21.2: Error from allocator on JSON import
nikitamikhaylov added a commit that referenced this pull request Feb 16, 2021
Backport #20286 to 20.8: Error from allocator on JSON import
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error on JSON import

3 participants