You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@kafka.apache.org by "Jun Rao (JIRA)" <ji...@apache.org> on 2014/04/22 01:34:26 UTC

[jira] [Commented] (KAFKA-1411) remove unused test cases

    [ https://issues.apache.org/jira/browse/KAFKA-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13976189#comment-13976189 ] 

Jun Rao commented on KAFKA-1411:
--------------------------------

Created reviewboard https://reviews.apache.org/r/20541/
 against branch origin/trunk

> remove unused test cases
> ------------------------
>
>                 Key: KAFKA-1411
>                 URL: https://issues.apache.org/jira/browse/KAFKA-1411
>             Project: Kafka
>          Issue Type: Test
>          Components: core
>    Affects Versions: 0.8.2
>            Reporter: Jun Rao
>            Assignee: Jun Rao
>         Attachments: KAFKA-1411.patch
>
>
> We have some redundant tests for producer.
> 1. Tests in LazyInitProducerTest are redundant from those in PrimitiveApiTest.
> 2. AsyncProducerTest.testBrokerListAndAsync() just returns immediately. There are other tests that cover the same functionality.
> 3. PrimitiveApiTest: The compression property wasn't set properly. Also, there is no need to test compression with other combinations such as encoder/multi-fetch.



--
This message was sent by Atlassian JIRA
(v6.2#6252)