Marton Greber has posted comments on this change. ( http://gerrit.cloudera.org:8080/24127 )
Change subject: [examples] add replication demo ...................................................................... Patch Set 6: (9 comments) Thanks for looking into the details! http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/Makefile File examples/flink-replication/Makefile: http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/Makefile@110 PS4, Line 110: apache/kudu:${KUDU_PYTHON_VERSION} (s > this version is pinned in the envfile as kudu-python-1.18.1-ubuntu, maybe u Done http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/docker-compose.yml File examples/flink-replication/docker-compose.yml: http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/docker-compose.yml@268 PS4, Line 268: kudu:${KUDU_PYTHON_VERSION} (se > update to kudu-python-1.18.1-ubuntu Done http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/docker-compose.yml@329 PS4, Line 329: grafana/grafana:${GRAF > maybe this should be pinned too? along with the json-exporter as well Done http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/flink/submit-job.sh File examples/flink-replication/flink/submit-job.sh: http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/flink/submit-job.sh@29 PS4, Line 29: # DISCOVERY_INTERVAL_SECONDS KuduSource diff-scan period (default: 10) : # CHECKPOINT_INTERVAL_MS Flink checkpoint interval (default: 5000) > these defaults are correct when there is no .env file, however they are dec Done http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/ingest/ingest.py File examples/flink-replication/ingest/ingest.py: http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/ingest/ingest.py@41 PS4, Line 41: 16 bucket > this comment declares 4 buckets, however NUM_PARTITIONS is declared as 16 a Done http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/ingest/ingest.py@264 PS4, Line 264: ("INSERT_RATE", 0.4)), > this doesn't match line #50: Done http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/ingest/ingest.py@270 PS4, Line 270: ("UPDATE_RATE", 0.2)), > this doesn't match line #51: Done http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/monitoring/prometheus.yml File examples/flink-replication/monitoring/prometheus.yml: http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/monitoring/prometheus.yml@22 PS4, Line 22: > there is no monitoring-up target in the makefile, also the docker-compose.y Done http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/scripts/ready.sh File examples/flink-replication/scripts/ready.sh: http://gerrit.cloudera.org:8080/#/c/24127/4/examples/flink-replication/scripts/ready.sh@26 PS4, Line 26: Called auto > the makefile doesn't have a ready target Done -- To view, visit http://gerrit.cloudera.org:8080/24127 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iebf29efaebc91d9831f57fc884a2acbab31715ce Gerrit-Change-Number: 24127 Gerrit-PatchSet: 6 Gerrit-Owner: Marton Greber <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[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: Tue, 07 Apr 2026 10:53:59 +0000 Gerrit-HasComments: Yes
