-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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 use quota bug #31656
Fix use quota bug #31656
Conversation
tests/integration/test_quota/test.py
Outdated
@@ -393,30 +393,45 @@ def test_query_inserts(): | |||
|
|||
|
|||
def test_consumption_of_show_tables(): | |||
assert re.match( | |||
"myQuota\\tdefault\\t.*\\t31556952\\t0\\t1000\\t0\\t.*\\t\\\\N.*", | |||
instance.query("SHOW QUOTA")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, SHOW TABLES
should use and check quota correctly no matter if SHOW QUOTA
was called before SHOW TABLES
or not.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me help you
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR along with #32575 must fix the issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tks,the problem has been solved
@Mergifyio update |
✅ Branch has been successfully updated |
Backport #31656 to 21.12: Fix use quota bug
Backport #31656 to 21.11: Fix use quota bug
Backport #31656 to 21.10: Fix use quota bug
Backport #31656 to 21.9: Fix use quota bug
Backport #31656 to 21.3: Fix use quota bug
Backport #31656 to 21.8: Fix use quota bug
Changelog category (leave one):
Bug Fix
Changelog entry:
Quota limit was not reached, but the limit was exceeded. This PR fixes #31174
this PR #31337
this change does not solve the problem,need_reset_counters the first time is still false,I think the first time the limit is exceeded, used should be reset to 0 @vitlibar