amogh-jahagirdar commented on code in PR #4071:
URL: https://github.com/apache/iceberg/pull/4071#discussion_r841284999
##########
api/src/main/java/org/apache/iceberg/ManageSnapshots.java:
##########
@@ -80,4 +80,98 @@
* wapId
*/
ManageSnapshots cherrypick(long snapshotId);
+
+ /**
+ * Create a new branch pointing to the given snapshot id.
+ *
+ * @param name branch name
+ * @param snapshotId id of the snapshot which will be the head of the branch
+ * @return this for method chaining
+ * @throws IllegalArgumentException if a branch with the given name already
exists
+ */
+ ManageSnapshots createBranch(String name, long snapshotId);
+
+ /**
+ * Create a new tag pointing to the given snapshot id
+ *
+ * @param name tag name
+ * @param snapshotId snapshotId for the head of the new branch.
+ * @return this for method chaining
+ * @throws IllegalArgumentException if a tag with the given name already
exists
+ */
+ ManageSnapshots createTag(String name, long snapshotId);
+
+ /**
+ * Remove a branch by name
+ *
+ * @param name branch name
+ * @return this for method chaining
+ * @throws IllegalArgumentException if the branch does not exist
+ */
+ ManageSnapshots removeBranch(String name);
Review Comment:
I need to add checks to prevent removing/renaming the main branch.
--
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: dev-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
