lurnagao-dahua commented on PR #10119:
URL: https://github.com/apache/iceberg/pull/10119#issuecomment-2309175541
Hi,Is there a plan to port to Spark 3.3?
--
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
aokolnychyi commented on PR #10119:
URL: https://github.com/apache/iceberg/pull/10119#issuecomment-2050543397
Thanks for reviewing, @szehon-ho @tmnd1991 @nastra!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
aokolnychyi merged PR #10119:
URL: https://github.com/apache/iceberg/pull/10119
--
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...@icebe
aokolnychyi commented on PR #10119:
URL: https://github.com/apache/iceberg/pull/10119#issuecomment-2050541242
In 3.4, we use custom `ReplaceIcebergData` and rewrite the operations
differently. I think that logic is only needed in 3.5.
--
This is an automated message from the Apache Git Se
tmnd1991 commented on PR #10119:
URL: https://github.com/apache/iceberg/pull/10119#issuecomment-2049109276
> lgtm, ReplaceData is not available in Spark 3.4?
ReplaceData is there in 3.4 but has 1 less argument (`groupFilterCondition:
Option[Expression]`) but I see it's not used at all
aokolnychyi commented on PR #10119:
URL: https://github.com/apache/iceberg/pull/10119#issuecomment-2048595286
I excluded the unnecessary branch in `ReplaceStaticInvoke` and adapted the
tests.
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
aokolnychyi commented on PR #10119:
URL: https://github.com/apache/iceberg/pull/10119#issuecomment-2048536387
I forgot 3.4 is a bit different, I may need to make some changes. Let me see.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
