nastra commented on code in PR #6074: URL: https://github.com/apache/iceberg/pull/6074#discussion_r1081611444
########## api/src/main/java/org/apache/iceberg/Transaction.java: ########## @@ -155,6 +155,13 @@ default UpdateStatistics updateStatistics() { */ ExpireSnapshots expireSnapshots(); + /** + * Create a new {@link ManageSnapshots manage snapshot API} to manage snapshots in this table. + * + * @return a new {@link ManageSnapshots} + */ + ManageSnapshots manageSnapshots(); Review Comment: RevAPI is checking API and ABI compatibility, so it's not just tricking RevAPI, it's also to make sure that users of this particular `Transaction` API don't run into API/ABI breakages when they upgrade the Iceberg version. -- 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