You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Piotr Nowojski (JIRA)" <ji...@apache.org> on 2018/11/30 08:59:00 UTC

[jira] [Closed] (FLINK-10838) Rewrite Kafka tests that fail Kafka brokers

     [ https://issues.apache.org/jira/browse/FLINK-10838?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Piotr Nowojski closed FLINK-10838.
----------------------------------
       Resolution: Won't Fix
    Fix Version/s:     (was: 1.8.0)

After further investigation, we can not rewrite those tests that are failing kafka brokers, because their whole point is to test the behaviour when Kafka brokers are failing. We already have very similar tests that are simply not closing test harness to simulate Flink's failure.

Rewriting tests that are using network failure proxy is still a valid thing and was extracted to separate issue https://issues.apache.org/jira/browse/FLINK-11038

> Rewrite Kafka tests that fail Kafka brokers
> -------------------------------------------
>
>                 Key: FLINK-10838
>                 URL: https://issues.apache.org/jira/browse/FLINK-10838
>             Project: Flink
>          Issue Type: Improvement
>          Components: Kafka Connector
>    Affects Versions: 1.7.0
>            Reporter: Piotr Nowojski
>            Priority: Critical
>
> Currently we have many tests that in order to test for example `at-least-once`, they fail Kafka brokers, or break network connection to them. It seems that those tests are more testing Kafka brokers then our own code and in the process (because of bugs in Kafka) cause lots of false positive results.
> We could try to rewrite those tests to spawn one of the Flink's Task Managers in separate process and instead of failing network/kafka broker we could SIGKILL Task Manager, without touching/affecting Zookeeper/Kafka brokers. It should improve stability of our tests.
> CC [~till.rohrmann] [~aljoscha]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)