aokolnychyi commented on a change in pull request #2314:
URL: https://github.com/apache/iceberg/pull/2314#discussion_r591984524
##########
File path: spark/src/main/java/org/apache/iceberg/actions/Actions.java
##########
@@ -79,7 +80,8 @@ public RewriteDataFilesAction rewriteDataFiles() {
}
public ExpireSnapshotsAction expireSnapshots() {
- return new ExpireSnapshotsAction(spark, table);
+ BaseExpireSnapshotsSparkAction delegate = new
BaseExpireSnapshotsSparkAction(spark, table);
+ return new ExpireSnapshotsAction(delegate);
Review comment:
Oh, yeah, we will mark `Actions` deprecated once we have a new
alternative. We will have a totally new entry point called `SparkActions` in
another package. Right now, I am migrating action by action to reduce the
amount of changes in a single pr. Once all actions are done, I'll create a new
public API for users to use and deprecate the rest.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
