amogh-jahagirdar commented on code in PR #9134:
URL: https://github.com/apache/iceberg/pull/9134#discussion_r1406602865
##########
core/src/main/java/org/apache/iceberg/UpdateSnapshotReferencesOperation.java:
##########
@@ -106,40 +106,41 @@ public UpdateSnapshotReferencesOperation
replaceBranch(String name, long snapsho
return this;
}
- public UpdateSnapshotReferencesOperation replaceBranch(String name, String
source) {
- return replaceBranch(name, source, false);
+ public UpdateSnapshotReferencesOperation replaceBranch(String from, String
to) {
+ return replaceBranch(from, to, false);
}
- public UpdateSnapshotReferencesOperation fastForward(String name, String
source) {
- return replaceBranch(name, source, true);
+ public UpdateSnapshotReferencesOperation fastForward(String from, String to)
{
+ return replaceBranch(from, to, true);
}
private UpdateSnapshotReferencesOperation replaceBranch(
- String name, String source, boolean fastForward) {
- Preconditions.checkNotNull(name, "Target branch cannot be null");
- Preconditions.checkNotNull(source, "Source ref cannot be null");
- SnapshotRef sourceRef = updatedRefs.get(source);
- SnapshotRef refToUpdate = updatedRefs.get(name);
- Preconditions.checkArgument(refToUpdate != null, "Target branch does not
exist: %s", name);
- Preconditions.checkArgument(sourceRef != null, "Ref does not exist: %s",
source);
- Preconditions.checkArgument(refToUpdate.isBranch(), "Ref %s is a tag not a
branch", name);
+ String from, String to, boolean fastForward) {
+ Preconditions.checkNotNull(from, "Branch to update cannot be null");
Review Comment:
Yeah you're right, and this unfortunately applies to more than just
replace/fastForward. (e.g. create, rename, as well throw an NPE instead of
IAE). I'll take this in a follow on.
--
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
