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

Reply via email to