linhongliu-db opened a new pull request #33437:
URL: https://github.com/apache/spark/pull/33437
### What changes were proposed in this pull request?
Change the `NullType.simpleString` to "void" to set "void" as the formal
type name of `NullType`
### Why are the changes needed?
This PR is intended to address the type name discussion in PR #29152. Here
are the reasons:
1. The type name of NullType is displayed everywhere, e.g. schema string,
error message, document. Hence it's not possible to hide it from users and we
have to choose a proper name
2. The "void" is widely used as the type name of "NULL", e.g. Hive, pgSQL
3. Changing to "void" can enable the round trip of `toDDL`/`fromDDL` for
NullType
### Does this PR introduce _any_ user-facing change?
Yes, the type name of "NULL" is changed from "null" to "void". for example:
```
scala> sql("select null as a, 1 as b").schema.catalogString
res5: String = struct<a:void,b:int>
```
### How was this patch tested?
existing test cases
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
