AngersZhuuuu commented on issue #25694: [SPARK-28982][SQL] Implementation Spark's own GetTypeInfoOperation URL: https://github.com/apache/spark/pull/25694#issuecomment-529051363 > LGTM. > I think it's good now. > It can be thought between "NULL" and "VOID", but I think it's fine as is - NULL is a weird type, and it's name cannot be directly used as a type name in a CREATE TABLE statement anyway (`CREATE TABLE foo (col NULL)` would fail), so I think it doesn't matter if the name is NULL or VOID. Yes, and in `SparkExecuteStatementOperation` it get NullType's attrTypeString as `void` too
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
