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 2022/12/14 06:37:23 UTC

[GitHub] [doris] nextdreamblue opened a new pull request, #15074: [fix](multi-catalog) fix get many same name db/table when show where

nextdreamblue opened a new pull request, #15074:
URL: https://github.com/apache/doris/pull/15074

   when show databases/tables/table status where xxx, it will change a selectStmt to select result from information_schema, it need catalog info to scan schema table, or may get many database or table info from multi catalog.
   
   for example
   mysql> show databases where schema_name='test';
   +----------+
   | Database |
   +----------+
   | test     |
   | test     |
   +----------+
   
   MySQL [internal.test]> show tables from test where table_name='test_dc';
   +----------------+
   | Tables_in_test |
   +----------------+
   | test_dc        |
   | test_dc        |
   +----------------+
   
   # Proposed changes
   
   Issue Number: close #xxx
   
   ## Problem summary
   
   Describe your changes.
   
   ## Checklist(Required)
   
   1. Does it affect the original behavior: 
       - [ ] Yes
       - [x] No
       - [ ] I don't know
   2. Has unit tests been added:
       - [ ] Yes
       - [ ] No
       - [x] No Need
   3. Has document been added or modified:
       - [ ] Yes
       - [ ] No
       - [x] No Need
   4. Does it need to update dependencies:
       - [ ] Yes
       - [x] No
   5. Are there any changes that cannot be rolled back:
       - [ ] Yes (If Yes, please explain WHY)
       - [x] 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] github-actions[bot] commented on pull request #15074: [fix](multi-catalog) fix get many same name db/table when show where

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #15074:
URL: https://github.com/apache/doris/pull/15074#issuecomment-1350489708

   clang-tidy review says "All clean, LGTM! :+1:"


-- 
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] nextdreamblue commented on pull request #15074: [fix](multi-catalog) fix get many same name db/table when show where

Posted by GitBox <gi...@apache.org>.
nextdreamblue commented on PR #15074:
URL: https://github.com/apache/doris/pull/15074#issuecomment-1350529480

   > please add regression test for this case
   
   ok, 


-- 
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 #15074: [fix](multi-catalog) fix get many same name db/table when show where

Posted by GitBox <gi...@apache.org>.
hello-stephen commented on PR #15074:
URL: https://github.com/apache/doris/pull/15074#issuecomment-1350539510

   TeamCity pipeline, clickbench performance test result:
    the sum of best hot time: 34.84 seconds
    load time: 470 seconds
    storage size: 17123356913 Bytes
    https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221214072937_clickbench_pr_63165.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] nextdreamblue commented on pull request #15074: [fix](multi-catalog) fix get many same name db/table when show where

Posted by GitBox <gi...@apache.org>.
nextdreamblue commented on PR #15074:
URL: https://github.com/apache/doris/pull/15074#issuecomment-1352749799

   > please add regression test for this case
   
   pls review this pr: https://github.com/apache/doris/pull/15076


-- 
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] nextdreamblue closed pull request #15074: [fix](multi-catalog) fix get many same name db/table when show where

Posted by GitBox <gi...@apache.org>.
nextdreamblue closed pull request #15074: [fix](multi-catalog) fix get many same name db/table when show where
URL: https://github.com/apache/doris/pull/15074


-- 
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