Liangda-w commented on pull request #10052:
URL: https://github.com/apache/shardingsphere/pull/10052#issuecomment-822469483


   Hi, I think since we can't let `physicalAttributesClause` contains an empty 
String, we can indicate the possibility of empty String at a higher level (in 
the clauses conatining `physicalAttributesClause`). So for example I changed 
`oidIndexClause` to
   
   `OIDINDEX indexName? LP_ ((physicalAttributesClause | TABLESPACE 
tablespaceName)+ | ) RP_`
   
   What do think of this way to solve this issue? @wgy8283335 @tristaZero 


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


Reply via email to