[
https://issues.apache.org/jira/browse/KAFKA-2056?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14493495#comment-14493495
]
Fangmin Lv commented on KAFKA-2056:
-----------------------------------
Hi [~guozhang], I agree with you, it's more robust to return a kv pair with an
empty Map if no partitions are assigned. To make sure we don't break anything,
I will check the code base before changing the behaviour of
PartitionAssignor.assign().
> PartitionAssignorTest.testRangePartitionAssignor transient failure
> ------------------------------------------------------------------
>
> Key: KAFKA-2056
> URL: https://issues.apache.org/jira/browse/KAFKA-2056
> Project: Kafka
> Issue Type: Sub-task
> Reporter: Guozhang Wang
> Assignee: Fangmin Lv
> Labels: newbie
> Attachments: KAFKA-2056.patch
>
>
> {code}
> unit.kafka.consumer.PartitionAssignorTest > testRangePartitionAssignor FAILED
> java.lang.NullPointerException
> at
> unit.kafka.consumer.PartitionAssignorTest$$anonfun$unit$kafka$consumer$PartitionAssignorTest$$assignAndVerify$1.apply(PartitionAssignorTest.scala:173)
> at
> unit.kafka.consumer.PartitionAssignorTest$$anonfun$unit$kafka$consumer$PartitionAssignorTest$$assignAndVerify$1.apply(PartitionAssignorTest.scala:172)
> at scala.collection.immutable.List.foreach(List.scala:318)
> at
> unit.kafka.consumer.PartitionAssignorTest$.unit$kafka$consumer$PartitionAssignorTest$$assignAndVerify(PartitionAssignorTest.scala:172)
> at
> unit.kafka.consumer.PartitionAssignorTest$$anonfun$testRangePartitionAssignor$1.apply$mcVI$sp(PartitionAssignorTest.scala:79)
> at scala.collection.immutable.Range.foreach$mVc$sp(Range.scala:141)
> at
> unit.kafka.consumer.PartitionAssignorTest.testRangePartitionAssignor(PartitionAssignorTest.scala:60)
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
