Marton Greber has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23121 )
Change subject: KUDU-3662 [4/n] Add reader & writer config parsing ...................................................................... KUDU-3662 [4/n] Add reader & writer config parsing Adds configuration parsing for Kudu Source and Kudu Sink so that replication jobs can be tuned without code changes. * ReplicationEnvProvider now consumes the parsed configs. * ReplicationTestBase auto-generates default reader/writer configs. * Adds two unit tests in replication_config_parser_test to verify correct deserialization and application of defaults. Change-Id: I48633a52046c7b5e637786d8e3c72d89946dc3e9 Reviewed-on: http://gerrit.cloudera.org:8080/23121 Tested-by: Marton Greber <[email protected]> Reviewed-by: Gabriella Lotz <[email protected]> Reviewed-by: Zoltan Chovan <[email protected]> Reviewed-by: Ashwani Raina <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> Reviewed-by: Zoltan Martonka <[email protected]> --- M java/kudu-replication/src/main/java/org/apache/kudu/replication/ReplicationConfigParser.java M java/kudu-replication/src/main/java/org/apache/kudu/replication/ReplicationEnvProvider.java M java/kudu-replication/src/main/java/org/apache/kudu/replication/ReplicationJob.java M java/kudu-replication/src/test/java/org/apache/kudu/replication/ReplicationTestBase.java M java/kudu-replication/src/test/java/org/apache/kudu/replication/TestReplication.java M java/kudu-replication/src/test/java/org/apache/kudu/replication/TestReplicationConfigParser.java 6 files changed, 242 insertions(+), 15 deletions(-) Approvals: Marton Greber: Verified Gabriella Lotz: Looks good to me, but someone else must approve Zoltan Chovan: Looks good to me, but someone else must approve Ashwani Raina: Looks good to me, but someone else must approve Alexey Serbin: Looks good to me, approved Zoltan Martonka: Looks good to me, but someone else must approve -- To view, visit http://gerrit.cloudera.org:8080/23121 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I48633a52046c7b5e637786d8e3c72d89946dc3e9 Gerrit-Change-Number: 23121 Gerrit-PatchSet: 4 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]>
