Skip to content

Add memory accounting in parallel formatting#22425

Merged
alexey-milovidov merged 3 commits intomasterfrom
parallel-formatting-memory
Apr 1, 2021
Merged

Add memory accounting in parallel formatting#22425
alexey-milovidov merged 3 commits intomasterfrom
parallel-formatting-memory

Conversation

@alexey-milovidov
Copy link
Copy Markdown
Member

@alexey-milovidov alexey-milovidov commented Apr 1, 2021

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Add (missing) memory accounting in parallel parsing routines. In previous versions OOM was possible when the resultset contains very large blocks of data. This closes #22008.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Apr 1, 2021
@nikitamikhaylov nikitamikhaylov self-assigned this Apr 1, 2021
@alexey-milovidov alexey-milovidov merged commit e3aa47c into master Apr 1, 2021
@alexey-milovidov alexey-milovidov deleted the parallel-formatting-memory branch April 1, 2021 13:23
alexey-milovidov added a commit that referenced this pull request Apr 2, 2021
Backport #22425 to 21.3: Add memory accounting in parallel formatting
alexey-milovidov added a commit that referenced this pull request Apr 2, 2021
Backport #22425 to 21.2: Add memory accounting in parallel formatting
alexey-milovidov added a commit that referenced this pull request Apr 2, 2021
Backport #22425 to 21.1: Add memory accounting in parallel formatting
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.

Memory limit sometimes doesn't work with TSan

3 participants