PedroCorreiaLuis commented on pull request #26319:
URL: https://github.com/apache/spark/pull/26319#issuecomment-721298978


   @kiszk So a couple of things: 
       -  I couldn't find any Character.isJavaIdentifier.
       - if you have the case class Foo that you mentioned it will enter in my 
if, and it only breaks because it starts with a 1, if you try this example you 
will see: 
         ```case class Foo(`a+something`: String)```
         `val test = Seq(Foo("HelloWorld!")).toDS()`
   
   This won't throw any exception.
   


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