Rodrigo
Rodrigo
Hi @Guosmilesmile Thanks for reporting this issue. I have a couple question/comments: 1. How "flaky" is this unit test? is it flaky in a particular Flink Version? Have you been...
> The scope of this PR is to provide unit tests for the new implementation of Flink's sink (`IcebergSink`) based on Flink's v2 sinks API. This PR is the follow-up...
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...
> > 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...
Hi @arkadius is this still needed/relevant ? please advise, as it will be closed due to inactivity soon cc @pvary
Hi @arkadius is this still needed/relevant ? please advise, as it will be closed due to inactivity soon cc @pvary
> Do the schema conversions handle the nano ts correctly? Do we get a `Timestamp(9)` when we convert to Flink schema, or a nano timestamp when we convert from the...
@pvary @stevenzwu could you guys please start the CI pipelines on this PR? Thanks
This one looks good to me. @pvary could you please add me as reviewer so I can register my approval ? Thanks