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/08/02 10:48:11 UTC

[GitHub] [shardingsphere] TeslaCN opened a new issue, #19810: PostgreSQL/openGauss RESET statement cannot be parsed caused by #15309

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

   ## Bug Report
   
   Related to #18738.
   
   ### Which version of ShardingSphere did you use?
   
   master - 73674d5ef5de2adce8a4a881e79baf4c401c352c
   
   ### Which project did you use? ShardingSphere-JDBC or ShardingSphere-Proxy?
   
   SQL parser.
   
   ### Expected behavior
   
   PostgreSQL / openGauss RESET statement could be parsed correctly.
   
   ### Actual behavior
   
   ```
   bmsql_sharding=> reset all;
   ERROR:  You have an error in your SQL syntax
   ```
   
   ### Reason analyze (If you can)
   
   #15309 add an `EOF` which broke the rule of RESET statement.
   


-- 
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] RaigorJiang commented on issue #19810: PostgreSQL/openGauss RESET statement cannot be parsed caused by #15309

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

   > @RaigorJiang If we removed the EOF, does Scaling DistSQL `RESET` still work in PostgreSQL?
   
   No, removing EOF will cause `RESET SCALING jobId` not to work


-- 
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 closed issue #19810: PostgreSQL/openGauss RESET statement cannot be parsed caused by #15309

Posted by GitBox <gi...@apache.org>.
strongduanmu closed issue #19810: PostgreSQL/openGauss RESET statement cannot be parsed caused by #15309
URL: https://github.com/apache/shardingsphere/issues/19810


-- 
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] runqi-zhao commented on issue #19810: PostgreSQL/openGauss RESET statement cannot be parsed caused by #15309

Posted by GitBox <gi...@apache.org>.
runqi-zhao commented on issue #19810:
URL: https://github.com/apache/shardingsphere/issues/19810#issuecomment-1202588101

   please assign it to me, I will have a try


-- 
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 #19810: PostgreSQL/openGauss RESET statement cannot be parsed caused by #15309

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

   @RaigorJiang If we removed the EOF, does Scaling DistSQL `RESET` still work in PostgreSQL?


-- 
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 #19810: PostgreSQL/openGauss RESET statement cannot be parsed caused by #15309

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

   ### UPDATE
   I found that `EOF` has been added to `PostgreSQLStatementParser.g4` (by #16818), so that one in `resetParameter` can now be removed.


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