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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org