raminqaf commented on PR #29041: URL: https://github.com/apache/flink/pull/29041#issuecomment-5477958168
> `flink-table/flink-table-common/src/main/java/org/apache/flink/table/types/utils/ClassDataTypeConverter.java:83-84`: nit: it registers `Bitmap.class` and `Variant.class` but I don't see a `UUID.class -> DataTypes.UUID()` entry. Might be worth adding for consistency, unless there's a reason to leave it out? My plan is to add this entry together with FLINK-40490 so the reflective path is functional the moment `java.util.UUID` starts resolving to `UUID`. If you would rather keep parity with `Bitmap`/`Variant` now, I can add it, with that caveat. -- 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]
