GitHub user rmetzger opened a pull request:

    https://github.com/apache/flink/pull/500

    Improvements to Kafka Source/Sink & some bugfixes in Flink Streaming

    Hi,
    I've spend the last two days on improving the Kafka connectors and the 
streaming fault tolerance.
    Since its the first time I'm touching this code, I would like to get some 
feedback from the streaming guys (in particular on the changes with the state 
handling).
    
    Please note that this PR is based on 
https://github.com/apache/flink/pull/474, so it also contains commits from 
@gaborhermann.
    I'll squash the commits after the first round of feedback.
    
    If you want I can separate the the critical fixes from the Kafka connector 
rewrite, but i would prefer merging everything at once.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/rmetzger/flink kafka-gabor

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/500.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #500
    
----
commit b77196d73ef152330e172644aa83b54c44e05906
Author: Gábor Hermann <reckone...@gmail.com>
Date:   2015-03-09T18:03:11Z

    [streaming] Added topic distribution to low level Kafka consumer

commit f70f31a5dc69f22c419911cf43062dd218308d4c
Author: Gábor Hermann <reckone...@gmail.com>
Date:   2015-03-10T14:12:29Z

    [streaming] Added leader searching to Kafka connector at broker failure

commit 5de12983a7343a05f8850f9078a157b229a71136
Author: Gábor Hermann <reckone...@gmail.com>
Date:   2015-03-11T09:10:45Z

    [streaming] Updated Kafka connector documentation

commit 9310711766bdfb41f19d89f776195cc02681f809
Author: Gábor Hermann <reckone...@gmail.com>
Date:   2015-03-11T15:50:45Z

    [streaming] Kafka consumers connecting to Zookeeper instead of a broker

commit c421215ffdfc4bc8761d1403d3c37cd6ff875cd5
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-16T16:16:05Z

    Let the KafkaConfigWrapper use the correct classloader

commit d3d08122c90c9299f78980fa75b5967f23c3f12d
Author: Gábor Hermann <reckone...@gmail.com>
Date:   2015-03-17T01:45:11Z

    [streaming] [wip] Kafka integration test

commit fb165f6f8f8fbb4dfc6cea939ca8121ac4668275
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-17T09:14:34Z

    fix

commit 4cdac3dc56276b3061ea49c8ccde50af4c235927
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-17T10:57:44Z

    pom cleanup

commit f6d3d41402d434a364cf3700567fd1f521e46acd
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-17T13:45:37Z

    wip

commit 2d5a8e86e90b42b8d86c7a2e870dc679f56e79db
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-17T14:03:31Z

    local test is able to reproduce issue

commit 38bb7b44e862f5f61d72bc3db3ba35fc2ca63e3f
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-17T14:59:59Z

    Fixed exception forwarding in streaming code

commit d2909a584d085131eff9cfb4c304c034f5049c12
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-17T18:40:20Z

    wip

commit c74a558dcc3f179cfa358d703cf913503ac5ead4
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-17T21:06:19Z

    very close

commit 3ad0fa830e86195a0fa6daaa1b212bd77825b1a2
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-17T21:49:56Z

    That should work

commit f6c95d9a73472eccb68cf363ac3fe712d35c8a3c
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-17T22:00:21Z

    current

commit f7603fad2f644bf40d2c6d7ae454838dcbca170e
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-18T10:24:35Z

    checkstyle

commit d38c7befe239d5cbe5983ffa26d45e53edf0f8cc
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-18T13:31:46Z

    make it ser

commit 005ce8c43333f660d8c66f5ac3eb8534d6d04b40
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-18T14:09:39Z

    use usercode classloader to deserialize state

commit 57159e6e9aa7d8da9b9c88bec33bb2adc92879be
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-18T14:13:21Z

    checkstyle

commit 29cd3d010ff058b52a253e405ae3a5d65d879301
Author: Robert Metzger <rmetz...@apache.org>
Date:   2015-03-18T17:13:04Z

    wip

----


---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to