Ashwani Raina has posted comments on this change. ( http://gerrit.cloudera.org:8080/24139 )
Change subject: docs: add Flink replication guide ...................................................................... Patch Set 3: (3 comments) Almost there. Just a nit and a question. http://gerrit.cloudera.org:8080/#/c/24139/2/docs/kudu_flink_replication.adoc File docs/kudu_flink_replication.adoc: http://gerrit.cloudera.org:8080/#/c/24139/2/docs/kudu_flink_replication.adoc@694 PS2, Line 694: eady state (between discovery cycles), `pendingCount`, `unassignedCount`, and > Yes, this is covered in a separate change already up for review: https://ge As long as that patch is merged first, things should be fine. Otherwise, there will be a window of possible dead-link. http://gerrit.cloudera.org:8080/#/c/24139/2/docs/kudu_flink_replication.adoc@881 PS2, Line 881: : [source] Not sure how this works. Is there no unit mismatch between 'timestamp(t)' and 't/1000'? http://gerrit.cloudera.org:8080/#/c/24139/3/docs/kudu_flink_replication.adoc File docs/kudu_flink_replication.adoc: http://gerrit.cloudera.org:8080/#/c/24139/3/docs/kudu_flink_replication.adoc@395 PS3, Line 395: has no practical effect in Here and below: nit: This has no practical effect on -- To view, visit http://gerrit.cloudera.org:8080/24139 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I684d608165af636bd4a799351926b68322469218 Gerrit-Change-Number: 24139 Gerrit-PatchSet: 3 Gerrit-Owner: Marton Greber <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Reviewer: Zoltan Martonka <[email protected]> Gerrit-Comment-Date: Wed, 01 Apr 2026 10:53:22 +0000 Gerrit-HasComments: Yes
