You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2023/01/12 14:29:22 UTC
[GitHub] [doris] SaintBacchus opened a new pull request, #15882: [fix](mtmv) Fix show mtmv statement wrong exception
SaintBacchus opened a new pull request, #15882:
URL: https://github.com/apache/doris/pull/15882
# Proposed changes
Issue Number: close #15881
## Problem summary
Wrong exception raise in the right show mtmv statement.
Describe your changes.
Add a null check in the analyze
## Checklist(Required)
1. Does it affect the original behavior:
- [ ] Yes
- [ ] No
- [ ] I don't know
2. Has unit tests been added:
- [ ] Yes
- [ ] No
- [ ] No Need
3. Has document been added or modified:
- [ ] Yes
- [ ] No
- [ ] No Need
4. Does it need to update dependencies:
- [ ] Yes
- [ ] No
5. Are there any changes that cannot be rolled back:
- [ ] Yes (If Yes, please explain WHY)
- [ ] No
## Further comments
If this is a relatively large or complex change, kick off the discussion at [dev@doris.apache.org](mailto:dev@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc...
--
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
[GitHub] [doris] hello-stephen commented on pull request #15882: [fix](mtmv) Fix show mtmv statement wrong exception
Posted by GitBox <gi...@apache.org>.
hello-stephen commented on PR #15882:
URL: https://github.com/apache/doris/pull/15882#issuecomment-1380866888
TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.84 seconds
load time: 491 seconds
storage size: 17122680832 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230112185421_clickbench_pr_79145.html
--
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
[GitHub] [doris] github-actions[bot] commented on pull request #15882: [fix](mtmv) Fix show mtmv statement rise wrong exception for right statement
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #15882:
URL: https://github.com/apache/doris/pull/15882#issuecomment-1386379767
PR approved by at least one committer and no changes requested.
--
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
[GitHub] [doris] SaintBacchus commented on pull request #15882: [fix](mtmv) Fix show mtmv statement wrong exception
Posted by GitBox <gi...@apache.org>.
SaintBacchus commented on PR #15882:
URL: https://github.com/apache/doris/pull/15882#issuecomment-1381227940
@stalary `show mtmv task from db1 on t1` is a right statement.
But now it rise exceptions since `mvName.getDb()` is null.
It should rise exception in this statement `show mtmv task from db1 on db2.t1`
--
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
[GitHub] [doris] stalary commented on pull request #15882: [fix](mtmv) Fix show mtmv statement wrong exception
Posted by GitBox <gi...@apache.org>.
stalary commented on PR #15882:
URL: https://github.com/apache/doris/pull/15882#issuecomment-1381180564
I don't understand. What problem is this supposed to solve?
--
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
[GitHub] [doris] github-actions[bot] commented on pull request #15882: [fix](mtmv) Fix show mtmv statement rise wrong exception for right statement
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #15882:
URL: https://github.com/apache/doris/pull/15882#issuecomment-1386379801
PR approved by anyone and no changes requested.
--
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
[GitHub] [doris] adonis0147 merged pull request #15882: [fix](mtmv) Fix show mtmv statement rise wrong exception for right statement
Posted by GitBox <gi...@apache.org>.
adonis0147 merged PR #15882:
URL: https://github.com/apache/doris/pull/15882
--
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