liziyan-lzy commented on PR #12254:
URL: https://github.com/apache/iceberg/pull/12254#issuecomment-2933062276

   > > Hi @pvary , I've noticed there are some conflicts in this PR. Would you 
recommend that I rebase onto the current main branch and resolve conflicts?
   > 
   > Definitely do it please. We can't merge with conflicts 😄
   > 
   > > Also, I see Spark 4.0 is now the default version. Should we keep these 
changes focused on the current target version and create a follow-up PR for 
Spark 4.0 compatibility? Thanks for your guidance!
   > 
   > Since rebase is needed anyway, you might be able to move the changes to 
Spark 4.0. If you find it difficult, we can merge this to Spark 3.5, and then 
port the changes to the other supported versions.
   > 
   > We generally try to work on the latest version, so the changes slowly make 
to all of the supported versions. Even if we forget to backport the changes. 
But this one is only the backup plan. The work is finished on a feature when it 
is ported to every supported version.
   
   Hi, I have resolved the merge conflicts. Would it make sense to first land 
this in Spark 3.5, then create a subsequent PR to port these changes to our 
other supported version branches?


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

Reply via email to