HyukjinKwon edited a comment on pull request #28833:
URL: https://github.com/apache/spark/pull/28833#issuecomment-655277714


   Re: https://github.com/apache/spark/pull/28833#discussion_r448165084
   
   Sorry I read the comments just now. So the decision here is we allow to 
parse `void` as `NullType` but doesn't allow it in some commands like `CREATE 
TABLE`s.
   
   How about other cases when we directly use DDL-formatted string as its type? 
These simple type strings can be used in many places such as `from_csv` (schema 
as DDL formatted string), `from_json` (schema as DDL formatted string), 
`createDataFrame` (Python), etc. However, `StructType.simpleString` cannot 
still be parsed as the valid types.


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