nevzheng opened a new issue, #11963:
URL: https://github.com/apache/gravitino/issues/11963
### What would you like to be improved?
The native `variant` type was added to Gravitino's type system (#11932), but
the CLI's string→type parsing was never updated to recognize it. Running the
CLI with `--datatype variant` fails:
```
gravitino table create ... --columns col --datatype variant
# → IllegalArgumentException: Unknown or unsupported type: variant
```
Every other primitive (`string`, `binary`, `uuid`, …) is accepted, so
`variant` is an inconsistent gap in the "specify a column by typing its type
string" flow.
### How should we improve?
Map `"variant"` to `Types.VariantType.get()` in
`TypeConverter.convert(String)` (`clients/cli/`), so the CLI accepts `variant`
as a column datatype (case-insensitive). `variant` is a parameterless
primitive, so parsing is lossless. Add unit tests covering
`TypeConverter.convert("variant")` and `ParseType.toType("variant")`.
--
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]
