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

feat: support adjust query's priority #15352

Merged
merged 16 commits into from May 10, 2024

Conversation

dqhl76
Copy link
Collaborator

@dqhl76 dqhl76 commented Apr 28, 2024

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

Tracking on: #15462

feat: support adjust query's priority

support dynamic adjust query's priority

How to use

SET PRIORITY (HIGH | MEDIUM | LOW) <session_id>

can get session_id from show processlist;

Test

send two same query: select avg(number) from numbers(10000000000);

root@localhost:8000/default> set priority high '39c01f8f-6c14-40bc-a1f8-1d24e9cd4a75';
root@localhost:8000/default> set priority low '7575b113-ce11-474f-b97a-292f0bb57c42';

39c01f8f-6c14-40bc-a1f8-1d24e9cd4a75

root@localhost:8000/default> select avg(number) from numbers(10000000000);
1 row read in 102.907 sec. Processed 10 billion row, 74.51 GiB (97.18 million row/s, 741.39 MiB/s)

7575b113-ce11-474f-b97a-292f0bb57c42

root@localhost:8000/default> select avg(number) from numbers(10000000000);
1 row read in 135.393 sec. Processed 10 billion row, 74.51 GiB (73.86 million row/s, 563.50 MiB/s)

Tests

  • Unit Test
  • Logic Test
  • Benchmark Test
  • No Test - Explain why

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • New Feature (non-breaking change which adds functionality)
  • Breaking Change (fix or feature that could cause existing functionality not to work as expected)
  • Documentation Update
  • Refactoring
  • Performance Improvement
  • Other (please describe):

This change is Reviewable

Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
@github-actions github-actions bot added the pr-feature this PR introduces a new feature to the codebase label Apr 28, 2024
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>

fix: license header

Signed-off-by: Liuqing Yue <dqhl76@gmail.com>

fix: license header

Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>

fixup stateful test

Signed-off-by: Liuqing Yue <dqhl76@gmail.com>

fix: ci test suits permission dennie

Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>

save

Signed-off-by: Liuqing Yue <dqhl76@gmail.com>

fixup

Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
@dqhl76 dqhl76 marked this pull request as ready for review May 7, 2024 10:01
@dqhl76 dqhl76 requested a review from zhang2014 May 7, 2024 10:01
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
Signed-off-by: Liuqing Yue <dqhl76@gmail.com>
@dqhl76 dqhl76 enabled auto-merge May 10, 2024 03:37
@dqhl76 dqhl76 added this pull request to the merge queue May 10, 2024
Merged via the queue into datafuselabs:main with commit 0001c82 May 10, 2024
75 checks passed
@dqhl76 dqhl76 deleted the priority-scheudling branch May 10, 2024 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-feature this PR introduces a new feature to the codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants