You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by GitBox <gi...@apache.org> on 2022/10/10 06:32:20 UTC

[GitHub] [shardingsphere] zhaojinchao95 opened a new issue, #20867: Add some integration test case for DistSQL RAL

zhaojinchao95 opened a new issue, #20867:
URL: https://github.com/apache/shardingsphere/issues/20867

   Hi community,
   
   ## Aim : 
   
   Add some integration test case for [RAL](https://shardingsphere.apache.org/document/current/cn/user-manual/shardingsphere-proxy/distsql/syntax/ral/)
   
   ## How to do
   
   1. You need to know about the integration test module [shardingsphere-integration-test-suite](https://github.com/apache/shardingsphere/tree/master/shardingsphere-test/shardingsphere-integration-test/shardingsphere-integration-test-suite)
   2. Add test case in `ral-integration-test-cases.xml`
   3. Add [expected data file ](https://github.com/apache/shardingsphere/tree/master/shardingsphere-test/shardingsphere-integration-test/shardingsphere-integration-test-suite/src/test/resources/cases/ral/dataset/empty_rules)
   
   ## How to run 
   
   1. You need have `docker` environment in your local.
   2. Build IT image `./mvnw -B clean install -am -pl shardingsphere-test/shardingsphere-integration-test/shardingsphere-integration-test-suite -Pit.env.docker -DskipTests -Dspotless.apply.skip=true`.
   3. Fill in `it.run.modes `with [it-env.properties](https://github.com/apache/shardingsphere/blob/master/shardingsphere-test/shardingsphere-integration-test/shardingsphere-integration-test-suite/src/test/resources/env/it-env.properties).
   4. Run [GeneralRALIT.class](https://github.com/apache/shardingsphere/blob/master/shardingsphere-test/shardingsphere-integration-test/shardingsphere-integration-test-suite/src/test/java/org/apache/shardingsphere/test/integration/engine/ral/GeneralRALIT.java).
   
   ## Task list : 
   - [x] SHOW INSTANCE LIST
   - [ ] SHOW READWRITE_SPLITTING READ RESOURCE
   - [ ] SHOW AUTHORITY RULE
   - [ ] SHOW TRANSACTION RULE
   - [ ] SHOW SQL_PARSER RULE
   - [ ] SHOW INSTANCE INFO @leedongyang 
   - [ ] SHOW MODE INFO @leedongyang 
   - [ ] COUNT DATABASE RULES
   - [ ] SET VARIABLE sql_show = true
   - [ ] SHOW ALL VARIABLES
   - [ ] SHOW VARIABLE sql_show


-- 
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: notifications-unsubscribe@shardingsphere.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] github-actions[bot] closed issue #20867: Add some integration test case for DistSQL RAL

Posted by GitBox <gi...@apache.org>.
github-actions[bot] closed issue #20867: Add some integration test case for DistSQL RAL
URL: https://github.com/apache/shardingsphere/issues/20867


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] leedongyang commented on issue #20867: Add some integration test case for DistSQL RAL

Posted by GitBox <gi...@apache.org>.
leedongyang commented on issue #20867:
URL: https://github.com/apache/shardingsphere/issues/20867#issuecomment-1240400187

   I want have a try，such as SHOW INSTANCE INFO，SHOW MODE INFO


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] github-actions[bot] commented on issue #20867: Add some integration test case for DistSQL RAL

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on issue #20867:
URL: https://github.com/apache/shardingsphere/issues/20867#issuecomment-1272348568

   Hello , this issue has not received a reply for several days.
   This issue is supposed to be closed.


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] Qianyi951015 commented on issue #20867: Add some integration test case for DistSQL RAL

Posted by GitBox <gi...@apache.org>.
Qianyi951015 commented on issue #20867:
URL: https://github.com/apache/shardingsphere/issues/20867#issuecomment-1346641000

   Hi @zhaojinchao95, I have done the second, third, fourth, fifth, tenth, and eleventh tasks, would you mind having a review when you are free?


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] Qianyi951015 commented on issue #20867: Add some integration test case for DistSQL RAL

Posted by GitBox <gi...@apache.org>.
Qianyi951015 commented on issue #20867:
URL: https://github.com/apache/shardingsphere/issues/20867#issuecomment-1352468178

   Hi, @zhaojinchao95 I have done the eighth and ninth tasks. Would you mind having a review?
   
   BTW: the `COUNT DATABASE RULES` was split into `COUNT SINGLE_TABLE RULE`, `COUNT SHARDING RULE`, `COUNT READWRITE_SPLITTING RULE`, `COUNT DATABASE_DISCOVERY RULE`, `COUNT ENCRYPT RULE`, `COUNT SHADOW RULE` (Ref: #18583).    I have added these cases to RQL.


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] zhaojinchao95 commented on issue #20867: Add some integration test case for DistSQL RAL

Posted by GitBox <gi...@apache.org>.
zhaojinchao95 commented on issue #20867:
URL: https://github.com/apache/shardingsphere/issues/20867#issuecomment-1368212651

   This task are finished at `master` branch, close it


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] TeslaCN commented on issue #20867: Add some integration test case for DistSQL RAL

Posted by GitBox <gi...@apache.org>.
TeslaCN commented on issue #20867:
URL: https://github.com/apache/shardingsphere/issues/20867#issuecomment-1333310367

   Test is not included in release. I'm removing milestone for now.


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] zhaojinchao95 closed issue #20867: Add some integration test case for DistSQL RAL

Posted by GitBox <gi...@apache.org>.
zhaojinchao95 closed issue #20867: Add some integration test case for DistSQL RAL
URL: https://github.com/apache/shardingsphere/issues/20867


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] zhaojinchao95 commented on issue #20867: Add some integration test case for DistSQL RAL

Posted by GitBox <gi...@apache.org>.
zhaojinchao95 commented on issue #20867:
URL: https://github.com/apache/shardingsphere/issues/20867#issuecomment-1240402762

   > I want have a try, such as SHOW INSTANCE INFO, SHOW MODE INFO.
   
   Welcome


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [shardingsphere] zhaojinchao95 closed issue #20867: Add some integration test case for DistSQL RAL

Posted by GitBox <gi...@apache.org>.
zhaojinchao95 closed issue #20867: Add some integration test case for DistSQL RAL
URL: https://github.com/apache/shardingsphere/issues/20867


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org