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 11:14:49 UTC

[GitHub] [shardingsphere] RaigorJiang opened a new issue, #21469: [DistSQL] Add SQL parser test case for `ClearShardingHintStatement`

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

   
   ### Background
   
   `CLEAR SHARDING HINT` is a syntax in [DistSQL RAL](https://shardingsphere.apache.org/document/current/en/user-manual/shardingsphere-proxy/distsql/syntax/ral/), when `ShardingSphereSQLParserEngine` receives this SQL text, it parses it as `ClearShardingHintStatement`. Now we need to add a test case for this parsing process to assert that the parsing is correct.
   
   
   ### Aim
   
   Add SQL parser test case for `ClearShardingHintStatement`.
   
   - Affected files:
   
     - SQLParserTestCases (**modify**)
   
     - test/parser/src/main/resources/case/ral/updatable.xml (**modify**)
   
     - test/parser/src/main/resources/sql/supported/ral/updatable.xml (**modify**)
   
     - test/parser/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/jaxb/cases/domain/statement/distsql/ral/updatable/ClearShardingHintStatementTestCase.java (**create**)
   
     - test/parser/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/asserts/statement/distsql/ral/impl/updatable/ClearShardingHintStatementAssert.java (**create**)
   
     - UpdatableRALStatementAssert (**modify**)
   
   
   - After all, execute `DistSQLParserParameterizedTest#assertDistSQL()` for validation
   
   ### Basic Qualifications
   
   - Java
   - Maven
   
   
   ### Example FYI
   
   - ShowAllVariablesStatement
   - ShowAllVariablesStatementTestCase
   - ShowAllVariablesStatementAssert
   - SQLParserTestCases
   - search for "show-all-variables"


-- 
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] vinayakghatul commented on issue #21469: [DistSQL] Add SQL parser test case for `ClearShardingHintStatement`

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

   Hi team,Can I pick this?


-- 
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] RaigorJiang commented on issue #21469: [DistSQL] Add SQL parser test case for `ClearShardingHintStatement`

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

   Hi @vinayakghatul ,
   Sure, welcome to Apache ShardingSphere, I just assign it to you.


-- 
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 #21469: [DistSQL] Add SQL parser test case for `ClearShardingHintStatement`

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

   
   please assign to me, i will finish asap
   
   


-- 
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] RaigorJiang closed issue #21469: [DistSQL] Add SQL parser test case for `ClearShardingHintStatement`

Posted by GitBox <gi...@apache.org>.
RaigorJiang closed issue #21469: [DistSQL] Add SQL parser test case for `ClearShardingHintStatement`
URL: https://github.com/apache/shardingsphere/issues/21469


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