Skip to content

Conversation

@KochetovNicolai
Copy link
Member

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Replace MemoryTrackerBlockerInThread to LockMemoryExceptionInThread in some places. Reduced MemoryTrackerBlockerInThread level to User.

…n some places. Reduced MemoryTrackerBlockerInThread level to User.
@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 26, 2022
@tavplubix tavplubix self-assigned this Jul 26, 2022
Copy link
Member

@tavplubix tavplubix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but consider making ctor that takes VariableContext private, and pass VariableContext::Global from MemoryTracker only

@tavplubix tavplubix merged commit 8fc075a into master Jul 27, 2022
@tavplubix tavplubix deleted the try-do-enable-global-allocations-for-attach branch July 27, 2022 10:13
@tavplubix tavplubix added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Jul 27, 2022
robot-clickhouse pushed a commit that referenced this pull request Jul 27, 2022
robot-clickhouse pushed a commit that referenced this pull request Jul 27, 2022
robot-clickhouse pushed a commit that referenced this pull request Jul 27, 2022
robot-clickhouse pushed a commit that referenced this pull request Jul 27, 2022
@robot-clickhouse robot-clickhouse added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jul 27, 2022
tavplubix added a commit that referenced this pull request Jul 28, 2022
Backport #39619 to 22.7: Replace MemoryTrackerBlockerInThread to LockMemoryExceptionInThread
KochetovNicolai added a commit that referenced this pull request Aug 1, 2022
Backport #39619 to 22.3: Replace MemoryTrackerBlockerInThread to LockMemoryExceptionInThread
KochetovNicolai added a commit that referenced this pull request Aug 1, 2022
Backport #39619 to 22.5: Replace MemoryTrackerBlockerInThread to LockMemoryExceptionInThread
KochetovNicolai added a commit that referenced this pull request Aug 1, 2022
Backport #39619 to 22.6: Replace MemoryTrackerBlockerInThread to LockMemoryExceptionInThread
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-must-backport Pull request should be backported intentionally. Use this label with great care! 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