fvaleri opened a new pull request, #14847: URL: https://github.com/apache/kafka/pull/14847
This change move the StorageTool from core to tools module. We have wrapper script, so there is no need to create a redirect object in core (see KIP-906). There is no impact on system tests. The tool uses some KafkaConfig properties (core), which are also migrated to the following classes: - LogConfig (storage): log.dir, logs.dirs, metadata.log.dir, inter.broker.protocol.version - RaftConfig (raft): node.id, process.roles, controller.listener.names The tool calls KafkaConfig.validateValues which runs the full set of configuration validations. Here we only have the ones that makes sense with the tool commands, the rest will be performed at startup. If this is not fine, then we would need to migrate the whole KafkaConfig first. -- 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
