[GitHub] [iceberg] jackye1995 commented on pull request #3685: Revert "Spark 2.4: Add test cases for SparkActions and remove deprecated Actions (#3587)"

[GitBox <gi...@apache.org>]

jackye1995 commented on pull request #3685:
URL: https://github.com/apache/iceberg/pull/3685#issuecomment-990267862


   Instead of reverting the entire `Actions`, can we only revert the ones we have not provided implementations in Spark24? I think the list is `snapshotTable`, `migrateTable`, `rewriteDataFiles`, which corresponds to `CreateAction`, `SnapshotAction` and `RewriteDataFilesAction`.
   
   @RussellSpitzer I think what you said is definitely the right way to fix this, but I feel we should not try to implement that just for the sake of fixing feature regression across releases. People who actually want to use these actions in Spark24 should do the backport.
   
   Any thoughts? @ajantha-bhat @rdblue 


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org