[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-09-05 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 7: Nice work Will! -- To view, visit http://gerrit.cloudera.org:8080/4034 To unsubscribe, visit

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-09-05 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Add AvroKuduEventProducer to Kudu-Flume integration This adds a KuduEventProducer that accepts

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-09-05 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/3231/ -- To view, visit

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-09-05 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 6: (7 comments) http://gerrit.cloudera.org:8080/#/c/4034/6/java/kudu-flume-sink/pom.xml File

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-09-05 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4034 to look at the new patch set (#7). Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Add

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-09-05 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 6: (7 comments) Found a few more test nits. I think this is nearly ready to commit now.

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-09-04 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 5: > BTW this producer is starting to look great. :D -- To view, visit

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-09-04 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/3230/ -- To view, visit

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-09-04 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 5: (6 comments)

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-09-04 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4034 to look at the new patch set (#6). Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Add

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-09-02 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 5: (6 comments)

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-30 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 5: > Build Failed > > http://104.196.14.100/job/kudu-gerrit/3158/ : FAILURE All the

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-30 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4034 to look at the new patch set (#5). Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Add

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-30 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/3158/ -- To view, visit

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-30 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 4: (7 comments) Did a significant rewrite to make it more similar to the DataSet Sink

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-30 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 4: (7 comments)

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-26 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/3089/ -- To view, visit

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-26 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4034 to look at the new patch set (#4). Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Add

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-25 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4034 to look at the new patch set (#3). Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Add

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-25 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 1: (6 comments) This is good stuff.

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-18 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2982/ -- To view, visit

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-18 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4034 to look at the new patch set (#2). Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Add

[kudu-CR] Add AvroKuduEventProducer to Kudu-Flume integration

2016-08-18 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add AvroKuduEventProducer to Kudu-Flume integration .. Patch Set 1: (7 comments) http://gerrit.cloudera.org:8080/#/c/4034/1/java/kudu-flume-sink/pom.xml File