arkadius commented on PR #11219:
URL: https://github.com/apache/iceberg/pull/11219#issuecomment-2404641938
> Hi @arkadius I have started working in backporting the RANGE distribution
to the IcebergSink. The unit tests in my code will benefit from the new marker
interface you are introducing in this PR, so I'd like to merge this one so that
I can rebase properly. I see this PR is still on "draft". Is anything pending?
>
> @pvary does this PR look good to you, or you have anything pending that
would like to be done? The PR is looking good to me.
>
> Also, please @pvary and @stevenzwu please help starting the CI pipelines
on this PR.
@rodmeneses I think that it would be better if you copied the parts of the
code from this PR into your PR. I can rebase into your changes after you merge
them.
This PR isn't finished yet. I have a plan how the final shape of unit test
could look like to make them easier to maintain and to fill some gaps in
`IcebergSink`'s tests but before I show it, I would like to merge this PR:
#11249 (and maybe this: #11244). Thanks to that, it will be more clear what is
the scope of this change.
--
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
