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 2020/07/06 14:55:16 UTC

[GitHub] [shardingsphere] strongduanmu opened a new pull request #6286: support subquery route and add rewrite test cases for sqlserver top rule

strongduanmu opened a new pull request #6286:
URL: https://github.com/apache/shardingsphere/pull/6286


   Fixes #4867 .
   
   Changes proposed in this pull request:
   - support subquery route
   - add rewrite test cases for sqlserver top rule
   


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

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

[GitHub] [shardingsphere] tristaZero commented on pull request #6286: support subquery route and add rewrite test cases for sqlserver top rule

Posted by GitBox <gi...@apache.org>.
tristaZero commented on pull request #6286:
URL: https://github.com/apache/shardingsphere/pull/6286#issuecomment-654568419


   Hi @strongduanmu 
   Thanks for your successive PRs, `TOP()` can be supported as before.
   You aced 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.

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

[GitHub] [shardingsphere] strongduanmu commented on pull request #6286: support subquery route and add rewrite test cases for sqlserver top rule

Posted by GitBox <gi...@apache.org>.
strongduanmu commented on pull request #6286:
URL: https://github.com/apache/shardingsphere/pull/6286#issuecomment-654571511


   > Hi @strongduanmu
   > Thanks for your successive PRs, `TOP()` can be supported as before.
   > You aced it! 😉
   
   @tristaZero It's my pleasure, and I hope to make more contributions to SS community!


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

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

[GitHub] [shardingsphere] tristaZero merged pull request #6286: support subquery route and add rewrite test cases for sqlserver top rule

Posted by GitBox <gi...@apache.org>.
tristaZero merged pull request #6286:
URL: https://github.com/apache/shardingsphere/pull/6286


   


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

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