Marton Greber has posted comments on this change. ( http://gerrit.cloudera.org:8080/24139 )
Change subject: docs: add Flink replication guide ...................................................................... Patch Set 1: (8 comments) Thank you Alexey for the valuable comments! http://gerrit.cloudera.org:8080/#/c/24139/1/docs/kudu_flink_replication.adoc File docs/kudu_flink_replication.adoc: http://gerrit.cloudera.org:8080/#/c/24139/1/docs/kudu_flink_replication.adoc@49 PS1, Line 49: . All writes to the sink use upsert-ignore semantics, making replication idempotent and : compatible with at-least-once Flink checkpointing. > IIUC, the replication job doesn't handle table's schema drift. Done http://gerrit.cloudera.org:8080/#/c/24139/1/docs/kudu_flink_replication.adoc@67 PS1, Line 67: The replication job JAR (e.g. `kudu-replication-<version>.jar`). > Is it a fat JAR without extra dependencies or some extra Kudu JARs are requ Done http://gerrit.cloudera.org:8080/#/c/24139/1/docs/kudu_flink_replication.adoc@153 PS1, Line 153: | All other users | Sink tables | _(no insert/update/delete)_ : | Explicitly deny or omit write privileges > What about permissions to perform various DDL operations on the target tabl Done http://gerrit.cloudera.org:8080/#/c/24139/1/docs/kudu_flink_replication.adoc@391 PS1, Line 391: |=== > What about VARCHAR columns? Done http://gerrit.cloudera.org:8080/#/c/24139/1/docs/kudu_flink_replication.adoc@393 PS1, Line 393: The `ARRAY` column type is *not supported*. Tables containing array columns cannot be : replicated. > Do we consider this as a temporary limitation? If so, maybe mention that t Done http://gerrit.cloudera.org:8080/#/c/24139/1/docs/kudu_flink_replication.adoc@420 PS1, Line 420: == Monitoring > This seems to be a substantial chunk of information specific to monitoring Done http://gerrit.cloudera.org:8080/#/c/24139/1/docs/kudu_flink_replication.adoc@422 PS1, Line 422: A working reference monitoring stack (Prometheus + Grafana + json_exporter) is included in : `examples/flink-replication/monitoring/` > Please also mention what project's repo this refers to. Done http://gerrit.cloudera.org:8080/#/c/24139/1/docs/kudu_flink_replication.adoc@766 PS1, Line 766: > It would be nice to provide at least a minimal guidance on handling table r Done -- 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: 1 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: Fri, 27 Mar 2026 15:06:32 +0000 Gerrit-HasComments: Yes
