You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by jg...@apache.org on 2019/06/20 16:49:52 UTC
[kafka] branch trunk updated: MINOR: Fix Partition::toString method
(#6971)
This is an automated email from the ASF dual-hosted git repository.
jgus pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git
The following commit(s) were added to refs/heads/trunk by this push:
new d40805d MINOR: Fix Partition::toString method (#6971)
d40805d is described below
commit d40805deacc5416be89984550b82c0b5d9dd57c5
Author: Vikas Singh <50...@users.noreply.github.com>
AuthorDate: Thu Jun 20 09:49:32 2019 -0700
MINOR: Fix Partition::toString method (#6971)
The AllReplicas was only printing remote replica ids. This change prints
all ids, including local one.
Reviewers: José Armando García Sancio <js...@users.noreply.github.com>, Jason Gustafson <ja...@confluent.io>
---
core/src/main/scala/kafka/cluster/Partition.scala | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/core/src/main/scala/kafka/cluster/Partition.scala b/core/src/main/scala/kafka/cluster/Partition.scala
index 544fc0c..0574cae 100755
--- a/core/src/main/scala/kafka/cluster/Partition.scala
+++ b/core/src/main/scala/kafka/cluster/Partition.scala
@@ -1201,7 +1201,7 @@ class Partition(val topicPartition: TopicPartition,
partitionString.append("Topic: " + topic)
partitionString.append("; Partition: " + partitionId)
partitionString.append("; Leader: " + leaderReplicaIdOpt)
- partitionString.append("; AllReplicas: " + remoteReplicasMap.keys.mkString(","))
+ partitionString.append("; AllReplicas: " + allReplicaIds.mkString(","))
partitionString.append("; InSyncReplicas: " + inSyncReplicas.mkString(","))
partitionString.toString
}