zeroshade commented on code in PR #827:
URL: https://github.com/apache/iceberg-go/pull/827#discussion_r3024249540


##########
table/transaction.go:
##########
@@ -149,6 +149,12 @@ func (t *Transaction) SetProperties(props 
iceberg.Properties) error {
        return nil
 }
 
+// UpgradeFormatVersion upgrades the table to the given format version. 
Downgrading
+// is not allowed. If the table is already at the given version, this is a 
no-op.
+func (t *Transaction) UpgradeFormatVersion(version int) error {
+       return t.apply([]Update{NewUpgradeFormatVersionUpdate(version)}, nil)
+}

Review Comment:
   Don't we have to do work here in rewriting files? Or is it expected that the 
catalog does this?



-- 
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]

Reply via email to