c-thiel commented on code in PR #12010: URL: https://github.com/apache/iceberg/pull/12010#discussion_r1922478361
########## api/src/main/java/org/apache/iceberg/UpdateStatistics.java: ########## @@ -27,9 +27,21 @@ public interface UpdateStatistics extends PendingUpdate<List<StatisticsFile>> { * the snapshot if any exists. * * @return this for method chaining + * @deprecated since 1.8.0, will be removed 1.9.0 or 2.0.0, use setStatistics(statisticsFile). Review Comment: fixed in latest commit ########## core/src/main/java/org/apache/iceberg/SetStatistics.java: ########## @@ -32,10 +32,22 @@ public SetStatistics(TableOperations ops) { this.ops = ops; } + /** + * Set the statistics file for a snapshot. + * + * @deprecated since 1.8.0, will be removed in 1.9.0 or 2.0.0, use setStatistics(statisticsFile). Review Comment: fixed in latest commit ########## open-api/rest-catalog-open-api.yaml: ########## @@ -2890,6 +2889,9 @@ components: snapshot-id: type: integer format: int64 + description: Review Comment: fixed in latest commit -- 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