[jira] [Commented] (KAFKA-4733) Improve Streams Reset Tool console output

2017-02-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-4733?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15853103#comment-15853103 ] ASF GitHub Bot commented on KAFKA-4733: --- GitHub user gwenshap opened a pull request:

[GitHub] kafka pull request #2503: KAFKA-4733: Improve Streams Reset Tool console out...

2017-02-04 Thread gwenshap
GitHub user gwenshap opened a pull request: https://github.com/apache/kafka/pull/2503 KAFKA-4733: Improve Streams Reset Tool console output Added general explanation of the tool and what it does. Also added few details to the arguments. You can merge this pull request into a Git re

[GitHub] kafka pull request #2502: Merge pull request #1 from apache/trunk

2017-02-04 Thread PierreCoquentin
Github user PierreCoquentin closed the pull request at: https://github.com/apache/kafka/pull/2502 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] kafka pull request #2502: Merge pull request #1 from apache/trunk

2017-02-04 Thread PierreCoquentin
GitHub user PierreCoquentin opened a pull request: https://github.com/apache/kafka/pull/2502 Merge pull request #1 from apache/trunk Update forked repo You can merge this pull request into a Git repository by running: $ git pull https://github.com/PierreCoquentin/kafka trunk A

Re: [DISCUSS] KIP-118: Drop Support for Java 7 in Kafka 0.11

2017-02-04 Thread Onur Karaman
LGTM. On Sat, Feb 4, 2017 at 4:54 AM, Tom Crayford wrote: > This is great, thanks. > > On Sat, Feb 4, 2017 at 12:22 PM, Dongjin Lee wrote: > > > It also makes sense to me. Thanks for your great work, Ismael. > > > > Add to this, it seems like there should be some following work on > utilizing >

Re: [DISCUSS] KIP-120: Cleanup Kafka Streams builder API

2017-02-04 Thread Matthias J. Sax
I think the right pattern should be to use TopologyBuilder as a class member, like new KStreamBuilder will do, instead of a class hierarchy, in case you want to offer your own higher level abstraction to describe a topology. Why is it important that you can derive a new class from it? I am not to

Re: [DISCUSS] KIP-118: Drop Support for Java 7 in Kafka 0.11

2017-02-04 Thread Tom Crayford
This is great, thanks. On Sat, Feb 4, 2017 at 12:22 PM, Dongjin Lee wrote: > It also makes sense to me. Thanks for your great work, Ismael. > > Add to this, it seems like there should be some following work on utilizing > some parts of the code by applying Java 8's features. > > Best, > Dongjin

Re: [DISCUSS] KIP-118: Drop Support for Java 7 in Kafka 0.11

2017-02-04 Thread Dongjin Lee
It also makes sense to me. Thanks for your great work, Ismael. Add to this, it seems like there should be some following work on utilizing some parts of the code by applying Java 8's features. Best, Dongjin On Sat, Feb 4, 2017 at 3:39 AM, Eno Thereska wrote: > Makes sense. > > Eno > > > On 3 F

[jira] [Updated] (KAFKA-4734) timeindex on old segments not trimmed to actual size

2017-02-04 Thread Jiangjie Qin (JIRA)
[ https://issues.apache.org/jira/browse/KAFKA-4734?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jiangjie Qin updated KAFKA-4734: Status: Patch Available (was: Open) > timeindex on old segments not trimmed to actual size > -