Mike Percy has posted comments on this change. Change subject: KUDU-1416 Upsert support for Flume sink ......................................................................
Patch Set 1: This looks pretty good to me, looks like there is some Jenkins compile error. Apparently it can't find Upsert? I wonder if it's pulling the wrong client jar down from Maven somehow. This is the compile error: 21:36:11 [INFO] ------------------------------------------------------------------------ 21:36:11 [INFO] Building Kudu Flume NG Sink 0.9.0-SNAPSHOT 21:36:11 [INFO] ------------------------------------------------------------------------ 21:36:11 [INFO] 21:36:11 [INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ kudu-flume-sink --- 21:36:11 [INFO] Deleting /home/jenkins-slave/workspace/kudu-0/java/kudu-flume-sink/target 21:36:11 [INFO] 21:36:11 [INFO] --- maven-resources-plugin:2.3:resources (default-resources) @ kudu-flume-sink --- 21:36:11 [INFO] Using 'UTF-8' encoding to copy filtered resources. 21:36:11 [INFO] skip non existing resourceDirectory /home/jenkins-slave/workspace/kudu-0/java/kudu-flume-sink/src/main/resources 21:36:11 [INFO] 21:36:11 [INFO] --- maven-compiler-plugin:2.3.2:compile (default-compile) @ kudu-flume-sink --- 21:36:11 [INFO] Compiling 5 source files to /home/jenkins-slave/workspace/kudu-0/java/kudu-flume-sink/target/classes 21:36:11 [INFO] ------------------------------------------------------------- 21:36:11 [ERROR] COMPILATION ERROR : 21:36:11 [INFO] ------------------------------------------------------------- 21:36:11 [ERROR] /home/jenkins-slave/workspace/kudu-0/java/kudu-flume-sink/src/main/java/org/kududb/flume/sink/SimpleKeyedKuduEventProducer.java:[27,24] error: cannot find symbol 21:36:11 [ERROR] package org.kududb.client 21:36:11 /home/jenkins-slave/workspace/kudu-0/java/kudu-flume-sink/src/main/java/org/kududb/flume/sink/SimpleKeyedKuduEventProducer.java:[81,37] error: cannot find symbol 21:36:11 [INFO] 2 errors -- To view, visit http://gerrit.cloudera.org:8080/3157 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ibe5b5df70687103ed6916d58148336882aa66d85 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Will Berkeley <wdberke...@gmail.com> Gerrit-Reviewer: Ara Ebrahimi <ara.ebrah...@argyledata.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: No