CatOrLeader commented on PR #3797: URL: https://github.com/apache/amoro/pull/3797#issuecomment-3342818673
@xxubai Hello! Thanks for your answer; I appreciate it a lot! About the issues you raised: I was trying to find any other part of the code that can be fixed to resolve this issue; however, it is directly connected to the [original Iceberg library](https://github.com/apache/iceberg), for example: * https://github.com/apache/iceberg/pull/14027 It is not fixed yet, but in my opinion, it is necessary to give users the ability to have UUID as a partition key even now because in some way it can be more optimised than other types. Additionally, we can see a similar issue with the UUID in the libraries that use Iceberg: * PyIceberg: https://github.com/apache/iceberg/issues/8248 I am open to any other possibility to fix this issue in the Amoro code; however, for now it seems like this dirty way of casting classes is the single possibility to resolve the issue on the Amoro code side -- 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]
