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/03/14 07:27:43 UTC

[GitHub] [shardingsphere] abhijeetjejurkar opened a new pull request #16063: Added unit test for TablesContext#findTableNamesByColumnProjection

abhijeetjejurkar opened a new pull request #16063:
URL: https://github.com/apache/shardingsphere/pull/16063


   Fixes: #15815
   
   Signed-off-by: Abhijeet Jejurkar <ab...@gmail.com>
   
   Fixes #ISSUSE_ID.
   
   Changes proposed in this pull request:
   -
   -
   -
   


-- 
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] abhijeetjejurkar commented on pull request #16063: Added unit test for TablesContext#findTableNamesByColumnProjection

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


   @strongduanmu  I Guess Error was due to extra line space at Line number 176. I just removed It. I guess It should work 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] abhijeetjejurkar commented on pull request #16063: Added unit test for TablesContext#findTableNamesByColumnProjection

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


   Sir It is something, "Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.1.0:check (validate) on project shardingsphere-infra-binder: You have 1 Checkstyle violation", I am new to this I will discuss it on slack.


-- 
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] abhijeetjejurkar commented on pull request #16063: Added unit test for TablesContext#findTableNamesByColumnProjection

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


   > @abhijeetjejurkar Good job, merged.
   
   Thankyou😊


-- 
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] strongduanmu merged pull request #16063: Added unit test for TablesContext#findTableNamesByColumnProjection

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


   


-- 
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] strongduanmu commented on pull request #16063: Added unit test for TablesContext#findTableNamesByColumnProjection

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


   > @strongduanmu I Guess Error was due to extra line space at Line number 176. I just removed It. I guess It should work now
   
   @abhijeetjejurkar Thank you, let's wait ci result.


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