mirror of https://github.com/apache/kafka.git
				
				
				
			KAFKA-18553: Update javadoc and comments of ConfigType (#18567)
Reviewers: Mickael Maison <mickael.maison@gmail.com>, Chia-Ping Tsai <chia7712@gmail.com>, Ismael Juma <ismael@juma.me.uk>, Andrew Schofield <aschofield@confluent.io>, Apoorv Mittal <amittal@confluent.io>
This commit is contained in:
		
							parent
							
								
									f917358275
								
							
						
					
					
						commit
						b14d686b87
					
				|  | @ -63,7 +63,7 @@ object ConfigCommand extends Logging { | |||
| 
 | ||||
|   private val BrokerDefaultEntityName = "" | ||||
|   val BrokerLoggerConfigType = "broker-loggers" | ||||
|   private val BrokerSupportedConfigTypes = ConfigType.ALL.asScala :+ BrokerLoggerConfigType :+ ConfigType.CLIENT_METRICS :+ ConfigType.GROUP | ||||
|   private val BrokerSupportedConfigTypes = ConfigType.ALL.asScala :+ BrokerLoggerConfigType | ||||
|   private val DefaultScramIterations = 4096 | ||||
| 
 | ||||
|   def main(args: Array[String]): Unit = { | ||||
|  |  | |||
|  | @ -16,11 +16,10 @@ | |||
|  */ | ||||
| package org.apache.kafka.server.config; | ||||
| 
 | ||||
| import java.util.Arrays; | ||||
| import java.util.List; | ||||
| 
 | ||||
| /** | ||||
|  * Represents all the entities that can be configured via ZK | ||||
|  * Represents all the entities that can be configured. | ||||
|  */ | ||||
| public class ConfigType { | ||||
|     public static final String TOPIC = "topics"; | ||||
|  | @ -31,6 +30,5 @@ public class ConfigType { | |||
|     public static final String CLIENT_METRICS = "client-metrics"; | ||||
|     public static final String GROUP = "groups"; | ||||
| 
 | ||||
|     // Do not include ClientMetrics and Groups in `all` as they are not supported on ZK. | ||||
|     public static final List<String> ALL = Arrays.asList(TOPIC, CLIENT, USER, BROKER, IP); | ||||
|     public static final List<String> ALL = List.of(TOPIC, CLIENT, USER, BROKER, IP, CLIENT_METRICS, GROUP); | ||||
| } | ||||
|  |  | |||
		Loading…
	
		Reference in New Issue