You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "yongjinhou (via GitHub)" <gi...@apache.org> on 2023/06/29 06:55:48 UTC
[GitHub] [doris] yongjinhou opened a new issue, #21322: [Enhancement](tvf) Add authentication for workload group tvf
yongjinhou opened a new issue, #21322:
URL: https://github.com/apache/doris/issues/21322
### Search before asking
- [X] I had searched in the [issues](https://github.com/apache/doris/issues?q=is%3Aissue) and found no similar issues.
### Description
Currently, displaying workload group information by tvf is not authenticated, and all workload group information is displayed.
We create workload group g1 by root, and perform select operation as a non-root user.
MySQL [(none)]> select * from workload_groups();
+-------+--------+--------------------------+------------+
| Id | Name | Item | Value |
+-------+--------+--------------------------+------------+
| 10002 | normal | enable_memory_overcommit | true |
| 10002 | normal | max_concurrency | 2147483647|
| 10002 | normal | queue_timeout | 0 |
| 10002 | normal | max_queue_size | 0 |
| 10002 | normal | memory_limit | 30% |
| 10002 | normal | cpu_share | 10 |
| 10082 | g1 | enable_memory_overcommit | true |
| 10082 | g1 | max_concurrency | 2147483647|
| 10082 | g1 | queue_timeout | 0 |
| 10082 | g1 | max_queue_size | 0 |
| 10082 | g1 | memory_limit | 30% |
| 10082 | g1 | cpu_share | 10 |
+-------+--------+--------------------------+------------+
We expect to show only information of workload groups to which the user has permission.
MySQL [(none)]> select * from workload_groups();
+-------+--------+--------------------------+------------+
| Id | Name | Item | Value |
+-------+--------+--------------------------+------------+
| 10002 | normal | enable_memory_overcommit | true |
| 10002 | normal | max_concurrency | 2147483647|
| 10002 | normal | queue_timeout | 0 |
| 10002 | normal | max_queue_size | 0 |
| 10002 | normal | memory_limit | 30% |
| 10002 | normal | cpu_share | 10 |
+-------+--------+--------------------------+------------+
### Solution
Add authentication for workload group tvf.
### Are you willing to submit PR?
- [X] Yes I am willing to submit a PR!
### Code of Conduct
- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] luozenglin closed issue #21322: [Enhancement](tvf) Add authentication for workload group tvf
Posted by "luozenglin (via GitHub)" <gi...@apache.org>.
luozenglin closed issue #21322: [Enhancement](tvf) Add authentication for workload group tvf
URL: https://github.com/apache/doris/issues/21322
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org