Skip to content

feat: query mem limiter #7078

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

feat: query mem limiter #7078

wants to merge 19 commits into from
+1,081 −29

Conversation

fengjiachun
Copy link
Collaborator

I hereby agree to the terms of the GreptimeDB CLA.

Refer to a related PR or issue link (optional)

  1. memory_pool_size for dadafusion query layer
  2. scan_memory_limit for mito-engine table scan

PR Checklist

Please convert it to a draft if some of the following conditions are not met.

  • I have written the necessary rustdoc comments.
  • I have added the necessary unit tests and integration tests.
  • This PR requires documentation updates.
  • API changes are backward compatible.
  • Schema or data changes are backward compatible.

@fengjiachun fengjiachun requested review from WenyXu and removed request for v0y4g3r and zhongzc October 13, 2025 08:53
@fengjiachun fengjiachun marked this pull request as draft October 13, 2025 08:53
@github-actions github-actions bot added docs-not-required This change does not impact docs. size/S labels Oct 13, 2025
@fengjiachun fengjiachun mentioned this pull request Oct 15, 2025
Open
2 tasks
@github-actions github-actions bot added size/M and removed size/S labels Oct 15, 2025
@fengjiachun fengjiachun marked this pull request as ready for review October 15, 2025 07:44
@fengjiachun fengjiachun force-pushed the feat/query_mem_limit branch 6 times, most recently from 45a3378 to 48a89f2 Compare October 16, 2025 11:56
@github-actions github-actions bot added size/L and removed size/M labels Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@waynexia waynexia Awaiting requested review from waynexia waynexia is a code owner

@evenyag evenyag Awaiting requested review from evenyag evenyag is a code owner

@WenyXu WenyXu Awaiting requested review from WenyXu

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

docs-not-required This change does not impact docs. size/L

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@fengjiachun