[GitHub] [iceberg] gaborkaszab commented on a diff in pull request #5726: API: Extended some deprecation comments in API folder

[GitBox <gi...@apache.org>]

gaborkaszab commented on code in PR #5726:
URL: https://github.com/apache/iceberg/pull/5726#discussion_r966705594


##########
api/src/main/java/org/apache/iceberg/metrics/DefaultMetricsContext.java:
##########
@@ -24,6 +24,9 @@
 public class DefaultMetricsContext implements MetricsContext {
   private static final int DEFAULT_HISTOGRAM_RESERVOIR_SIZE = 10_000;
 
+  /**
+   * @deprecated will be removed in 2.0.0, use {@link org.apache.iceberg.metrics.Counter} instead.

Review Comment:
   In theory you are only allowed to break the API in a major release bump, hence I've 2.0.0 here



-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org