fvaleri closed pull request #14847: KAFKA-14585: Move StorageTool to tools
URL: https://github.com/apache/kafka/pull/14847
--
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 un
fvaleri commented on code in PR #14847:
URL: https://github.com/apache/kafka/pull/14847#discussion_r1580757743
##
metadata/src/main/java/org/apache/kafka/metadata/properties/MetaProperties.java:
##
@@ -47,7 +47,7 @@ public final class MetaProperties {
/**
* The proper
fvaleri commented on code in PR #14847:
URL: https://github.com/apache/kafka/pull/14847#discussion_r1580759331
##
tools/src/main/java/org/apache/kafka/tools/StorageTool.java:
##
@@ -0,0 +1,555 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * cont
fvaleri commented on code in PR #14847:
URL: https://github.com/apache/kafka/pull/14847#discussion_r1580757743
##
metadata/src/main/java/org/apache/kafka/metadata/properties/MetaProperties.java:
##
@@ -47,7 +47,7 @@ public final class MetaProperties {
/**
* The proper
fvaleri commented on code in PR #14847:
URL: https://github.com/apache/kafka/pull/14847#discussion_r1580741403
##
tools/src/test/java/org/apache/kafka/tools/StorageToolTest.java:
##
@@ -0,0 +1,517 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ *
mimaison commented on code in PR #14847:
URL: https://github.com/apache/kafka/pull/14847#discussion_r1579785296
##
metadata/src/main/java/org/apache/kafka/metadata/properties/MetaProperties.java:
##
@@ -47,7 +47,7 @@ public final class MetaProperties {
/**
* The prope
fvaleri commented on PR #14847:
URL: https://github.com/apache/kafka/pull/14847#issuecomment-2068086501
@showuon @mimaison I think this is now ready for review.
I think now changes are well isolated. There is no code refactoring or Kafka
configuration changes, so comparison with the o
nizhikov commented on PR #14847:
URL: https://github.com/apache/kafka/pull/14847#issuecomment-1893726661
Can we extract changes regarding `LogConfig`, `RĐ°ftConfig` into separate PR
to simplify review?
--
This is an automated message from the Apache Git Service.
To respond to the message,
showuon commented on PR #14847:
URL: https://github.com/apache/kafka/pull/14847#issuecomment-1893320028
@fvaleri , I think you could extract the refactor part of code into another
PR, and ping me there when ready for review. Thanks.
--
This is an automated message from the Apache Git Serv
fvaleri commented on PR #14847:
URL: https://github.com/apache/kafka/pull/14847#issuecomment-1872986366
Rebased. Waiting for review. Thanks.
@showuon @mimaison @cmccabe
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
showuon commented on PR #14847:
URL: https://github.com/apache/kafka/pull/14847#issuecomment-1853758861
OK, let's wait until https://issues.apache.org/jira/browse/KAFKA-15853 is
merged, or we've got response from Colin.
--
This is an automated message from the Apache Git Service.
To
fvaleri commented on PR #14847:
URL: https://github.com/apache/kafka/pull/14847#issuecomment-1842545770
@showuon it is called when you instantiate the KafkaConfig object, so it's
the constructor:
https://github.com/apache/kafka/blob/trunk/core/src/main/scala/kafka/server/KafkaConfig.scala#L
showuon commented on PR #14847:
URL: https://github.com/apache/kafka/pull/14847#issuecomment-1842216032
> The tool calls KafkaConfig.validateValues which runs the full set of
configuration validations.
@fvaleri , sorry, I didn't see where we invoke `KafkaConfig.validateValues`
in St
fvaleri commented on PR #14847:
URL: https://github.com/apache/kafka/pull/14847#issuecomment-1831441198
> 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 perf
fvaleri commented on PR #14847:
URL: https://github.com/apache/kafka/pull/14847#issuecomment-1828371630
@mimaison @showuon this is big one, but changes are fairly isolated. It
works, but there are some open points which I mention in the PR description.
Let me know what you think.
--
This
15 matches
Mail list logo