[jira] [Commented] (FLUME-2897) AsyncHBase sink NPE when Channel.getTransaction() fails

2016-03-28 Thread Mike Percy (JIRA)
[ https://issues.apache.org/jira/browse/FLUME-2897?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15215006#comment-15215006 ] Mike Percy commented on FLUME-2897: --- The reason this patch is correct is because it woul

[jira] [Updated] (FLUME-2897) AsyncHBase sink NPE when Channel.getTransaction() fails

2016-03-28 Thread Mike Percy (JIRA)
[ https://issues.apache.org/jira/browse/FLUME-2897?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mike Percy updated FLUME-2897: -- Attachment: FLUME-2897-1.patch > AsyncHBase sink NPE when Channel.getTransaction() fails > -

[jira] [Created] (FLUME-2897) AsyncHBase sink NPE when Channel.getTransaction() fails

2016-03-28 Thread Mike Percy (JIRA)
Mike Percy created FLUME-2897: - Summary: AsyncHBase sink NPE when Channel.getTransaction() fails Key: FLUME-2897 URL: https://issues.apache.org/jira/browse/FLUME-2897 Project: Flume Issue Type: B

[jira] [Commented] (FLUME-2821) Flume-Kafka Source with new Consumer

2016-03-28 Thread Jarek Jarcec Cecho (JIRA)
[ https://issues.apache.org/jira/browse/FLUME-2821?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15214994#comment-15214994 ] Jarek Jarcec Cecho commented on FLUME-2821: --- Can you please upload the latest pa

[jira] [Updated] (FLUME-2821) Flume-Kafka Source with new Consumer

2016-03-28 Thread Jeff Holoman (JIRA)
[ https://issues.apache.org/jira/browse/FLUME-2821?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jeff Holoman updated FLUME-2821: Assignee: Grigoriy Rozhkov > Flume-Kafka Source with new Consumer >

Re: Review Request 41702: Kafka Source support for new Consumer API FLUME-2821

2016-03-28 Thread Ashish Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41702/#review125723 --- Ship it! Ship It! - Ashish Singh On March 28, 2016, 7:02 p.m

Re: Review Request 41702: Kafka Source support for new Consumer API FLUME-2821

2016-03-28 Thread Grigoriy Rozhkov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41702/ --- (Updated March 28, 2016, 7:02 p.m.) Review request for Flume, Grant Henke, Jare

Re: Review Request 41702: Kafka Source support for new Consumer API FLUME-2821

2016-03-28 Thread Ashish Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41702/#review125703 --- flume-ng-sources/flume-kafka-source/src/test/java/org/apache/flum

Re: Review Request 41702: Kafka Source support for new Consumer API FLUME-2821

2016-03-28 Thread Ashish Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41702/#review125702 --- flume-ng-sources/flume-kafka-source/src/test/java/org/apache/flum

Re: Review Request 41702: Kafka Source support for new Consumer API FLUME-2821

2016-03-28 Thread Grigoriy Rozhkov
> On March 28, 2016, 4:25 p.m., Ashish Singh wrote: > > flume-ng-sources/flume-kafka-source/src/test/java/org/apache/flume/source/kafka/KafkaSourceEmbeddedKafka.java, > > line 45 > > > > > > If we can not get hos

Re: Review Request 41702: Kafka Source support for new Consumer API FLUME-2821

2016-03-28 Thread Grigoriy Rozhkov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41702/ --- (Updated March 28, 2016, 5:42 p.m.) Review request for Flume, Grant Henke, Jare

Re: Review Request 41702: Kafka Source support for new Consumer API FLUME-2821

2016-03-28 Thread Ashish Singh
> On March 28, 2016, 4:26 p.m., Ashish Singh wrote: > > Ship It! Just one minor comment and we are good to go. Thanks for the awesome work Grigoriy! - Ashish --- This is an automatically generated e-mail. To reply, visit: https://revie

Re: Review Request 41702: Kafka Source support for new Consumer API FLUME-2821

2016-03-28 Thread Ashish Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41702/#review125668 --- Ship it! Ship It! - Ashish Singh On March 28, 2016, 9:38 a.m

Re: Review Request 41702: Kafka Source support for new Consumer API FLUME-2821

2016-03-28 Thread Ashish Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41702/#review125667 --- flume-ng-sources/flume-kafka-source/src/test/java/org/apache/flum

Re: Review Request 41702: Kafka Source support for new Consumer API FLUME-2821

2016-03-28 Thread Grigoriy Rozhkov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41702/ --- (Updated March 28, 2016, 9:38 a.m.) Review request for Flume, Grant Henke, Jare

Re: Review Request 41702: Kafka Source support for new Consumer API FLUME-2821

2016-03-28 Thread Grigoriy Rozhkov
> On March 25, 2016, 8:06 p.m., Ashish Singh wrote: > > flume-ng-doc/sphinx/FlumeUserGuide.rst, lines 1257-1260 > > > > > > Won't this require consumer group id? No, the default value 'flume' is used. > On Mar