adrian-wang opened a new pull request #28393:
URL: https://github.com/apache/spark/pull/28393


   
   ### What changes were proposed in this pull request?
   `def splitSemiColon` cannot handle unescaped quote mark like "'" or '"' 
correctly. When there are unmatched quotes in a string, `splitSemiColon` will 
not drop off semicolon as expected.
   
   ### Why are the changes needed?
   Some regex expression will use quote mark in string. As config 
"spark.sql.parser.escapedStringLiterals" default value `false` allows us use 
unescaped string, we should process semicolon position correctly.
   
   ### Does this PR introduce any user-facing change?
   No
   
   
   ### How was this patch tested?
   Added Unit test and also manual test.
   


----------------------------------------------------------------
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to