JTidbury2 commented on issue #18262: URL: https://github.com/apache/iceberg/issues/18262#issuecomment-6079765690
@mxm, could you advise on the best way forward here? As I understand it: - #18288 implements the shared-tracker approach, but there's been no response to the review, and the items @huaxingao marked as required on 1 Oct (key-type comparison by field ID, CDC without upsert, per-partition trackers) are still open. - @abrarsher23's implementation (Affirm/iceberg#17) covers all three, plus the extra test cases, and has been reviewed and merged on their fork. It currently targets `flink/v2.0` / `v2.1` on 1.11.0, so it would need porting to `main` (`flink/v2.3`). Would it be acceptable for @abrarsher23 to open a separate upstream PR with that implementation, with credit to @Asthenia0412 for #18288? Purely in the interest of turnaround and getting this issue fixed. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
