Jimexist commented on PR #2843: URL: https://github.com/apache/thrift/pull/2843#issuecomment-1674098784
> From the cross test log it seems that this change fixed the cross tests regarding uuid, and didn't add new failures. But I'm still not sure how `TType.ENUM` is actually used in Java code and whether -1 is going to be ok. cc @Jimexist not sure about the `TType.ENUM` myself either. regarding cross tests, since both java and kotlin uses the same convention so either ordinal will work since they are consistent. -- 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]
