-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[fix](auth)fix show routine load db is null #38365
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
Conversation
|
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
|
run buildall |
TPC-H: Total hot run time: 39554 ms |
TPC-DS: Total hot run time: 171941 ms |
dataroaring
left a comment
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.
Please add a regression test.
ClickBench: Total hot run time: 30.55 s |
|
run buildall |
TPC-H: Total hot run time: 39403 ms |
TPC-DS: Total hot run time: 173036 ms |
ClickBench: Total hot run time: 31.09 s |
dataroaring
left a comment
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.
LGTM
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
|
run buildall |
TPC-H: Total hot run time: 39351 ms |
TPC-DS: Total hot run time: 171732 ms |
ClickBench: Total hot run time: 30.27 s |
fix - when label contains dbName, will loss intro by apache#27861 - show routine load for xxx.yyy can export authentication error intro by apache#33347 Note: Cases will be added uniformly in other PRs
fix
when label contains dbName, will loss
cause by: [refactor](cluster)(step-4) remove cluster related to Database #27861
show routine load for xxx.yyycan export authentication errorcause [fix](auth)fix missing authentication #33347
Note: Cases will be added uniformly in other PRs