divijvaidya commented on code in PR #14266: URL: https://github.com/apache/kafka/pull/14266#discussion_r1300083853
########## storage/src/main/java/org/apache/kafka/server/log/remote/storage/RemoteLogManagerConfig.java: ########## @@ -146,111 +146,111 @@ public final class RemoteLogManagerConfig { public static final ConfigDef CONFIG_DEF = new ConfigDef(); static { - CONFIG_DEF.defineInternal(REMOTE_LOG_STORAGE_SYSTEM_ENABLE_PROP, + CONFIG_DEF.define(REMOTE_LOG_STORAGE_SYSTEM_ENABLE_PROP, BOOLEAN, DEFAULT_REMOTE_LOG_STORAGE_SYSTEM_ENABLE, null, MEDIUM, REMOTE_LOG_STORAGE_SYSTEM_ENABLE_DOC) - .defineInternal(REMOTE_STORAGE_MANAGER_CONFIG_PREFIX_PROP, + .define(REMOTE_STORAGE_MANAGER_CONFIG_PREFIX_PROP, STRING, DEFAULT_REMOTE_STORAGE_MANAGER_CONFIG_PREFIX, new ConfigDef.NonEmptyString(), MEDIUM, REMOTE_STORAGE_MANAGER_CONFIG_PREFIX_DOC) - .defineInternal(REMOTE_LOG_METADATA_MANAGER_CONFIG_PREFIX_PROP, + .define(REMOTE_LOG_METADATA_MANAGER_CONFIG_PREFIX_PROP, STRING, DEFAULT_REMOTE_LOG_METADATA_MANAGER_CONFIG_PREFIX, new ConfigDef.NonEmptyString(), MEDIUM, REMOTE_LOG_METADATA_MANAGER_CONFIG_PREFIX_DOC) - .defineInternal(REMOTE_STORAGE_MANAGER_CLASS_NAME_PROP, STRING, + .define(REMOTE_STORAGE_MANAGER_CLASS_NAME_PROP, STRING, null, new ConfigDef.NonEmptyString(), MEDIUM, REMOTE_STORAGE_MANAGER_CLASS_NAME_DOC) - .defineInternal(REMOTE_STORAGE_MANAGER_CLASS_PATH_PROP, STRING, + .define(REMOTE_STORAGE_MANAGER_CLASS_PATH_PROP, STRING, null, new ConfigDef.NonEmptyString(), MEDIUM, REMOTE_STORAGE_MANAGER_CLASS_PATH_DOC) - .defineInternal(REMOTE_LOG_METADATA_MANAGER_CLASS_NAME_PROP, + .define(REMOTE_LOG_METADATA_MANAGER_CLASS_NAME_PROP, STRING, DEFAULT_REMOTE_LOG_METADATA_MANAGER_CLASS_NAME, new ConfigDef.NonEmptyString(), MEDIUM, REMOTE_LOG_METADATA_MANAGER_CLASS_NAME_DOC) - .defineInternal(REMOTE_LOG_METADATA_MANAGER_CLASS_PATH_PROP, + .define(REMOTE_LOG_METADATA_MANAGER_CLASS_PATH_PROP, STRING, null, new ConfigDef.NonEmptyString(), MEDIUM, REMOTE_LOG_METADATA_MANAGER_CLASS_PATH_DOC) - .defineInternal(REMOTE_LOG_METADATA_MANAGER_LISTENER_NAME_PROP, STRING, + .define(REMOTE_LOG_METADATA_MANAGER_LISTENER_NAME_PROP, STRING, null, new ConfigDef.NonEmptyString(), MEDIUM, REMOTE_LOG_METADATA_MANAGER_LISTENER_NAME_DOC) - .defineInternal(REMOTE_LOG_METADATA_CUSTOM_METADATA_MAX_BYTES_PROP, + .define(REMOTE_LOG_METADATA_CUSTOM_METADATA_MAX_BYTES_PROP, INT, DEFAULT_REMOTE_LOG_METADATA_CUSTOM_METADATA_MAX_BYTES, atLeast(0), LOW, REMOTE_LOG_METADATA_CUSTOM_METADATA_MAX_BYTES_DOC) - .defineInternal(REMOTE_LOG_INDEX_FILE_CACHE_TOTAL_SIZE_BYTES_PROP, + .define(REMOTE_LOG_INDEX_FILE_CACHE_TOTAL_SIZE_BYTES_PROP, Review Comment: This is unused so far until https://issues.apache.org/jira/browse/KAFKA-14912 completes. Hence, this should be internal. Can you please verify the same for rest of the configs as well? ########## docs/toc.html: ########## @@ -52,7 +52,8 @@ <li><a href="#streamsconfigs">3.6 Kafka Streams Configs</a> <li><a href="#adminclientconfigs">3.7 AdminClient Configs</a> <li><a href="#mirrormakerconfigs">3.8 MirrorMaker Configs</a> - <li><a href="#systemproperties">3.9 System Properties</a> + <li><a href="#remotestorageconfigs">3.9 Tier Storage Configs</a></li> Review Comment: "Remote Storage Configs" Also please move this to 3.10 (instead of 3.9) so that existing hyperlinks for "System Properties" continue working properly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org