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/05/24 07:11:18 UTC
[kafka] branch trunk updated: MINOR: Fix a few compiler warnings
(#6767)
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 eefb25d MINOR: Fix a few compiler warnings (#6767)
eefb25d is described below
commit eefb25d4ca12a9c33ad654bf789eee4d451ebb0d
Author: Lee Dongjin <do...@apache.org>
AuthorDate: Fri May 24 16:11:04 2019 +0900
MINOR: Fix a few compiler warnings (#6767)
Reviewers: Boyang Chen <bc...@outlook.com>, Jason Gustafson <ja...@confluent.io>
---
core/src/main/scala/kafka/controller/KafkaController.scala | 2 ++
core/src/main/scala/kafka/coordinator/group/GroupCoordinator.scala | 6 ++++++
core/src/main/scala/kafka/server/KafkaConfig.scala | 2 +-
core/src/main/scala/kafka/utils/CommandLineUtils.scala | 1 -
core/src/test/scala/unit/kafka/server/RequestQuotaTest.scala | 1 -
5 files changed, 9 insertions(+), 3 deletions(-)
diff --git a/core/src/main/scala/kafka/controller/KafkaController.scala b/core/src/main/scala/kafka/controller/KafkaController.scala
index 2034c74..0880967 100644
--- a/core/src/main/scala/kafka/controller/KafkaController.scala
+++ b/core/src/main/scala/kafka/controller/KafkaController.scala
@@ -1602,6 +1602,8 @@ class KafkaController(val config: KafkaConfig,
processIsrChangeNotification()
case Startup =>
processStartup()
+ case ShutdownEventThread =>
+ // not handled here
}
} catch {
case e: ControllerMovedException =>
diff --git a/core/src/main/scala/kafka/coordinator/group/GroupCoordinator.scala b/core/src/main/scala/kafka/coordinator/group/GroupCoordinator.scala
index 55c90e5..daf38f3 100644
--- a/core/src/main/scala/kafka/coordinator/group/GroupCoordinator.scala
+++ b/core/src/main/scala/kafka/coordinator/group/GroupCoordinator.scala
@@ -401,6 +401,9 @@ class GroupCoordinator(val brokerId: Int,
val memberMetadata = group.get(memberId)
responseCallback(memberMetadata.assignment, Errors.NONE)
completeAndScheduleNextHeartbeatExpiration(group, group.get(memberId))
+
+ case Dead =>
+ throw new IllegalStateException(s"Reached unexpected condition for Dead group ${group.groupId}")
}
}
}
@@ -533,6 +536,9 @@ class GroupCoordinator(val brokerId: Int,
val member = group.get(memberId)
completeAndScheduleNextHeartbeatExpiration(group, member)
responseCallback(Errors.NONE)
+
+ case Dead =>
+ throw new IllegalStateException(s"Reached unexpected condition for Dead group $groupId")
}
}
}
diff --git a/core/src/main/scala/kafka/server/KafkaConfig.scala b/core/src/main/scala/kafka/server/KafkaConfig.scala
index 584620f..60af03c 100755
--- a/core/src/main/scala/kafka/server/KafkaConfig.scala
+++ b/core/src/main/scala/kafka/server/KafkaConfig.scala
@@ -29,7 +29,7 @@ import kafka.utils.CoreUtils
import kafka.utils.Implicits._
import org.apache.kafka.clients.CommonClientConfigs
import org.apache.kafka.common.Reconfigurable
-import org.apache.kafka.common.config.ConfigDef.{ConfigKey, ValidList, Validator}
+import org.apache.kafka.common.config.ConfigDef.{ConfigKey, ValidList}
import org.apache.kafka.common.config.internals.BrokerSecurityConfigs
import org.apache.kafka.common.config.{AbstractConfig, ConfigDef, ConfigException, SaslConfigs, SslClientAuth, SslConfigs, TopicConfig}
import org.apache.kafka.common.metrics.Sensor
diff --git a/core/src/main/scala/kafka/utils/CommandLineUtils.scala b/core/src/main/scala/kafka/utils/CommandLineUtils.scala
index 1bf7cdf..b19da3e 100644
--- a/core/src/main/scala/kafka/utils/CommandLineUtils.scala
+++ b/core/src/main/scala/kafka/utils/CommandLineUtils.scala
@@ -19,7 +19,6 @@
import java.util.Properties
import joptsimple.{OptionParser, OptionSet, OptionSpec}
-import org.apache.kafka.common.utils.AppInfoParser
import scala.collection.Set
diff --git a/core/src/test/scala/unit/kafka/server/RequestQuotaTest.scala b/core/src/test/scala/unit/kafka/server/RequestQuotaTest.scala
index 047188f..6b2fd29 100644
--- a/core/src/test/scala/unit/kafka/server/RequestQuotaTest.scala
+++ b/core/src/test/scala/unit/kafka/server/RequestQuotaTest.scala
@@ -14,7 +14,6 @@
package kafka.server
-import java.nio.ByteBuffer
import java.util.{Collections, LinkedHashMap, Optional, Properties}
import java.util.concurrent.{Executors, Future, TimeUnit}