[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15799721#comment-15799721 ] ASF GitHub Bot commented on TWILL-199: -- Github user chtyim commented on a diff in the pull request:

[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15799722#comment-15799722 ] ASF GitHub Bot commented on TWILL-199: -- Github user chtyim commented on a diff in the pull request:

[GitHub] twill pull request #16: [TWILL-199] Return the offset to read next message i...

2017-01-04 Thread chtyim
Github user chtyim commented on a diff in the pull request: https://github.com/apache/twill/pull/16#discussion_r94688950 --- Diff: twill-core/src/test/java/org/apache/twill/kafka/client/KafkaTest.java --- @@ -170,11 +172,57 @@ public void testKafkaClient() throws Exception {

[GitHub] twill pull request #16: [TWILL-199] Return the offset to read next message i...

2017-01-04 Thread chtyim
Github user chtyim commented on a diff in the pull request: https://github.com/apache/twill/pull/16#discussion_r94688058 --- Diff: twill-core/src/main/java/org/apache/twill/internal/kafka/client/SimpleKafkaConsumer.java --- @@ -273,17 +274,19 @@ private MessageCallback

[GitHub] twill pull request #16: [TWILL-199] Return the offset to read next message i...

2017-01-04 Thread chtyim
Github user chtyim commented on a diff in the pull request: https://github.com/apache/twill/pull/16#discussion_r94687750 --- Diff: twill-core/src/main/java/org/apache/twill/kafka/client/KafkaConsumer.java --- @@ -35,12 +35,14 @@ * Invoked when new messages is available.

[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15799233#comment-15799233 ] ASF GitHub Bot commented on TWILL-199: -- Github user maochf commented on a diff in the pull request:

[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15797904#comment-15797904 ] ASF GitHub Bot commented on TWILL-199: -- Github user maochf commented on a diff in the pull request:

[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15797901#comment-15797901 ] ASF GitHub Bot commented on TWILL-199: -- Github user chtyim commented on a diff in the pull request:

[GitHub] twill pull request #16: [TWILL-199] Handle offset error and return next offs...

2017-01-04 Thread chtyim
Github user chtyim commented on a diff in the pull request: https://github.com/apache/twill/pull/16#discussion_r94561337 --- Diff: twill-core/src/main/java/org/apache/twill/internal/kafka/client/SimpleKafkaConsumer.java --- @@ -273,17 +274,19 @@ private MessageCallback

[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15797882#comment-15797882 ] ASF GitHub Bot commented on TWILL-199: -- Github user maochf commented on a diff in the pull request:

[GitHub] twill pull request #16: [TWILL-199] Handle offset error and return next offs...

2017-01-04 Thread maochf
Github user maochf commented on a diff in the pull request: https://github.com/apache/twill/pull/16#discussion_r94559673 --- Diff: twill-core/src/test/java/org/apache/twill/kafka/client/KafkaTest.java --- @@ -170,11 +174,128 @@ public void testKafkaClient() throws Exception {

[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15797867#comment-15797867 ] ASF GitHub Bot commented on TWILL-199: -- Github user maochf commented on a diff in the pull request:

[GitHub] twill pull request #16: [TWILL-199] Handle offset error and return next offs...

2017-01-04 Thread maochf
Github user maochf commented on a diff in the pull request: https://github.com/apache/twill/pull/16#discussion_r94558645 --- Diff: twill-core/src/main/java/org/apache/twill/internal/kafka/client/SimpleKafkaConsumer.java --- @@ -68,7 +69,7 @@ /** * A {@link

[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15797861#comment-15797861 ] ASF GitHub Bot commented on TWILL-199: -- Github user maochf commented on a diff in the pull request:

[GitHub] twill pull request #16: [TWILL-199] Handle offset error and return next offs...

2017-01-04 Thread maochf
Github user maochf commented on a diff in the pull request: https://github.com/apache/twill/pull/16#discussion_r94558457 --- Diff: twill-core/src/main/java/org/apache/twill/internal/kafka/client/SimpleKafkaConsumer.java --- @@ -273,17 +274,19 @@ private MessageCallback

[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15797849#comment-15797849 ] ASF GitHub Bot commented on TWILL-199: -- Github user chtyim commented on a diff in the pull request:

[GitHub] twill pull request #16: [TWILL-199] Handle offset error and return next offs...

2017-01-04 Thread chtyim
Github user chtyim commented on a diff in the pull request: https://github.com/apache/twill/pull/16#discussion_r94557551 --- Diff: twill-core/src/main/java/org/apache/twill/internal/kafka/client/SimpleKafkaConsumer.java --- @@ -68,7 +69,7 @@ /** * A {@link

[GitHub] twill pull request #16: [TWILL-199] Handle offset error and return next offs...

2017-01-04 Thread chtyim
Github user chtyim commented on a diff in the pull request: https://github.com/apache/twill/pull/16#discussion_r94557480 --- Diff: twill-core/src/main/java/org/apache/twill/internal/kafka/client/SimpleKafkaConsumer.java --- @@ -273,17 +274,19 @@ private MessageCallback

[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15797840#comment-15797840 ] ASF GitHub Bot commented on TWILL-199: -- Github user maochf commented on a diff in the pull request:

[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15797824#comment-15797824 ] ASF GitHub Bot commented on TWILL-199: -- Github user maochf commented on a diff in the pull request:

[GitHub] twill pull request #16: [TWILL-199] Handle offset error and return next offs...

2017-01-04 Thread chtyim
Github user chtyim commented on a diff in the pull request: https://github.com/apache/twill/pull/16#discussion_r94551779 --- Diff: twill-core/src/test/java/org/apache/twill/kafka/client/KafkaTest.java --- @@ -170,11 +174,128 @@ public void testKafkaClient() throws Exception {

[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15797757#comment-15797757 ] ASF GitHub Bot commented on TWILL-199: -- Github user chtyim commented on a diff in the pull request:

[jira] [Commented] (TWILL-199) Get next offset and handle offset error in KafkaConsumer.MessageCallback

2017-01-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15797752#comment-15797752 ] ASF GitHub Bot commented on TWILL-199: -- Github user chtyim commented on a diff in the pull request: