Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix race condition and arena tracking bug #1468

Merged
merged 16 commits into from Nov 9, 2023

Conversation

antoniofilipovic
Copy link
Collaborator

Fixes issue on query not being tracked due to issue with jemalloc mallctl call to thread.arena which can give different arena in different times of calling.

[master < Task] PR

  • Check, and update documentation if necessary
  • Provide the full content or a guide for the final git message

To keep docs changelog up to date, one more thing to do:

  • Write a release note here, including added/changed clauses
  • Tag someone from docs team in the comments

@antoniofilipovic antoniofilipovic changed the base branch from master to use-extent-hooks-for-memory-proc-limit November 7, 2023 13:57
@antoniofilipovic
Copy link
Collaborator Author

Fixes issue: #1469

Base automatically changed from use-extent-hooks-for-memory-proc-limit to master November 7, 2023 15:04
@antoniofilipovic antoniofilipovic merged commit 1791557 into master Nov 9, 2023
6 checks passed
@antoniofilipovic antoniofilipovic deleted the fix-race-condition-arena-tracking-bug branch November 9, 2023 17:56
@gitbuda gitbuda added Docs - changelog only Docs - changelog only and removed Docs unnecessary Docs unnecessary labels Nov 12, 2023
@gitbuda gitbuda modified the milestones: mg-v2.13.0, mg-v2.12.1 Nov 14, 2023
@antoniofilipovic
Copy link
Collaborator Author

@vpavicic RN: fix race condition on MEMORY QUERY LIMIT queries, where query memory limit was not sometimes respected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer customer Docs - changelog only Docs - changelog only memory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants