raminqaf commented on PR #29041: URL: https://github.com/apache/flink/pull/29041#issuecomment-5479657618
> Hi, thanks for the PR! Just a small question: FLIP-604 also lists `byte[]` and canonical `String` as supported conversion classes, while `UuidType` currently only supports `java.util.UUID`. Which PR is planned to add support for these two conversion classes? Thanks Dylan! `byte[] `(16 bytes) and the canonical `String` are alternative conversion classes, not the default. They only make sense together with the DataStructureConverters that translate between the internal 16-byte value and those external classes, which is runtime work. A conversion class without its converter would pass supportsConversion but then fail at planning when the converter lookup comes up empty, so they can't be advertised on their own. They land with FLINK-40490 "Runtime serialization and codegen for UUID", alongside the UUID `DataStructureConverter`. This PR keeps only the default `java.util.UUID` conversion on purpose, so the type never advertises a bridging class that has no working converter yet. -- 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]
