You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by tzulitai <gi...@git.apache.org> on 2017/07/24 04:43:54 UTC

[GitHub] flink pull request #4386: (backport-1.3) [FLINK-7174] [kafka] Bump Kafka 0.1...

GitHub user tzulitai opened a pull request:

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

    (backport-1.3) [FLINK-7174] [kafka] Bump Kafka 0.10 dependency to 0.10.2.1

    Backport of #4321 to `release-1.3`, with the following things being different:
    1. No need to touch `KafkaConsumerThread`, because in 1.3 the code in `KafkaConsumerThread` will only ever be reached if there is partitions to subscribe to (therefore would not bump into the changed exception behaviour described in #4321).
    2. Some touched tests and classes do not exist in 1.3 (e.g. partition reassignment tests, `AbstractPartitionDiscoverer`) and therefore is not relevant for the backport.

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

    $ git pull https://github.com/tzulitai/flink FLINK-7174-flink13

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

    https://github.com/apache/flink/pull/4386.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 #4386
    
----
commit 9653b60974f68b14fd4be5ee5b9f0f687b764bdb
Author: Piotr Nowojski <pi...@gmail.com>
Date:   2017-07-13T09:07:28Z

    [FLINK-7174] [kafka connector] Bump Kafka 0.10 dependency to 0.10.2.1
    
    This closes #4321

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] flink pull request #4386: (backport-1.3) [FLINK-7174] [kafka] Bump Kafka 0.1...

Posted by tzulitai <gi...@git.apache.org>.
Github user tzulitai closed the pull request at:

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


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] flink issue #4386: (backport-1.3) [FLINK-7174] [kafka] Bump Kafka 0.10 depen...

Posted by pnowojski <gi...@git.apache.org>.
Github user pnowojski commented on the issue:

    https://github.com/apache/flink/pull/4386
  
    Looks good to me.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] flink issue #4386: (backport-1.3) [FLINK-7174] [kafka] Bump Kafka 0.10 depen...

Posted by tzulitai <gi...@git.apache.org>.
Github user tzulitai commented on the issue:

    https://github.com/apache/flink/pull/4386
  
    Thanks for the review @pnowojski!
    
    I'm also using this branch to collect some final backports for `release-1.3`. Will merge once Travis is green.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---