You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by Balázs Donát Bessenyei <be...@apache.org> on 2019/11/02 20:46:04 UTC
Re: Review Request 71514: FLUME-3345 Add Kudu Flume Sinks
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/71514/#review218487
-----------------------------------------------------------
Hi Grant,
Thank you for the patch!
Do you mind opening a PR in https://github.com/apache/flume/pulls ?
That should make the review process and merging easier.
- Balázs Donát Bessenyei
On Sept. 23, 2019, 7:59 p.m., Grant Henke wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/71514/
> -----------------------------------------------------------
>
> (Updated Sept. 23, 2019, 7:59 p.m.)
>
>
> Review request for Flume and Ferenc Szabo.
>
>
> Bugs: FLUME-3345
> https://issues.apache.org/jira/browse/FLUME-3345
>
>
> Repository: flume-git
>
>
> Description
> -------
>
> This patch adds the Flume sinks that were previously maintined
> in the Apache Kud project.
>
>
> Diffs
> -----
>
> build-support/src/main/resources/config/checkstyle/checkstyle-suppressions.xml 59814687
> flume-ng-sinks/flume-ng-kudu-sink/pom.xml PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/main/java/org/apache/flume/sink/kudu/AvroKuduOperationsProducer.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/main/java/org/apache/flume/sink/kudu/KuduOperationsProducer.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/main/java/org/apache/flume/sink/kudu/KuduSink.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/main/java/org/apache/flume/sink/kudu/KuduSinkConfigurationConstants.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/main/java/org/apache/flume/sink/kudu/RegexpKuduOperationsProducer.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/main/java/org/apache/flume/sink/kudu/SimpleKeyedKuduOperationsProducer.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/main/java/org/apache/flume/sink/kudu/SimpleKuduOperationsProducer.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/test/java/org/apache/flume/sink/kudu/KuduSinkTestUtil.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/test/java/org/apache/flume/sink/kudu/TestAvroKuduOperationsProducer.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/test/java/org/apache/flume/sink/kudu/TestKeyedKuduOperationsProducer.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/test/java/org/apache/flume/sink/kudu/TestKuduSink.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/test/java/org/apache/flume/sink/kudu/TestRegexpKuduOperationsProducer.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/test/java/org/apache/flume/sink/kudu/TestRegexpKuduOperationsProducerParseError.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/test/java/org/apache/flume/sink/kudu/TestSecureKuduSink.java PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/test/resources/log4j2.properties PRE-CREATION
> flume-ng-sinks/flume-ng-kudu-sink/src/test/resources/testAvroKuduOperationsProducer.avsc PRE-CREATION
> flume-ng-sinks/pom.xml 99d21268
> pom.xml be7edbcb
>
>
> Diff: https://reviews.apache.org/r/71514/diff/1/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Grant Henke
>
>