jackye1995 opened a new pull request #3702: URL: https://github.com/apache/iceberg/pull/3702
In #3638 , we improved `MetricsConfig` construction and marked it as immutable. As @aokolnychyi pointed out the class was used for serialization before we introduced `SerializableTable`. Although it is not likely that people are still serializing the metrics config, because it implements the `Serializable` interface, it's still safer to use `SerializableMap` instead of `ImmutableMap` for this to avoid any potential trouble. -- 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]
