leangjonathan commented on code in PR #10861:
URL: https://github.com/apache/iceberg/pull/10861#discussion_r1707621872
##########
core/src/main/java/org/apache/iceberg/TableMetadata.java:
##########
@@ -1020,8 +1020,12 @@ public Builder upgradeFormatVersion(int
newFormatVersion) {
return this;
}
+ // register incremental version changes separately to ensure all upgrade
requirements are met
Review Comment:
We are iterating in the loop from `this.formatVersion` to start so we'd need
to store it in another local var which is unnecessary IMO.
I did find a more concise way to write the loop though
```
// register incremental version changes separately to ensure all
upgrade requirements are met
do {
changes.add(new
MetadataUpdate.UpgradeFormatVersion(++formatVersion));
} while (formatVersion < newFormatVersion);
this.formatVersion = newFormatVersion;
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
