RussellSpitzer commented on code in PR #12897:
URL: https://github.com/apache/iceberg/pull/12897#discussion_r2060447565
##########
.palantir/revapi.yml:
##########
@@ -1177,6 +1177,10 @@ acceptedBreaks:
old: "class org.apache.iceberg.Metrics"
new: "class org.apache.iceberg.Metrics"
justification: "Java serialization across versions is not guaranteed"
+ - code: "java.class.defaultSerializationChanged"
Review Comment:
I think we are fine here, we don't expect a client/server using this code to
be on different library versions. I believe this currently is hit when using
Spark because we are going to serialize the table metadata around. For usages
like this I don't want to have to support multiple versions on driver /
executor.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
