[GitHub] [shardingsphere] TeslaCN opened a new issue, #25138: MySQL user/system variable was not parsed into VariableSegment

Wed, 12 Apr 2023 00:12:17 -0700 


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

   ## Bug Report
   
   ### Which version of ShardingSphere did you use?
   
   master - 1533b52e8380f2efb612fa55c087da54786f151e
   
   ### Expected behavior
   
   User/System variables could be parsed into `VariableSegment`.
   
   ### Actual behavior
   
   
![image](https://user-images.githubusercontent.com/20503072/231378205-b99c4e85-7280-4933-841f-631ae51b0f1c.png)
   
   
![image](https://user-images.githubusercontent.com/20503072/231378253-62d88a07-514e-4e2b-9fc2-a3a8c573238a.png)
   
   
   ### Steps to reproduce the behavior, such as: SQL to execute, sharding rule 
configuration, when exception occur etc.
   
   ```sql
   select @@max_allowed_packet AS max_allowed_packet, @@character_set_client;
   select @@version_comment limit 1;
   ```
   
   ### Analyze
   
   `MySQLDALStatementSQLVisitor` override `visitVariable`, `visitUserVariable`, 
`visitSystemVariable`. But `visitSimpleExpr` doesn't handle `ctx.variable()`. 
So the following are dead codes.
   
   
https://github.com/apache/shardingsphere/blob/1533b52e8380f2efb612fa55c087da54786f151e/sql-parser/dialect/mysql/src/main/java/org/apache/shardingsphere/sql/parser/mysql/visitor/statement/impl/MySQLDALStatementSQLVisitor.java#L986-L1008


-- 
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: 
[email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to