This is an automated email from the ASF dual-hosted git repository.
AndrewJSchofield 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 1938d32ca1e MINOR: Removed `setMetadataLogDirectory` unused method
(#22128)
1938d32ca1e is described below
commit 1938d32ca1e809df013393d2a2e4e9998f99ca4a
Author: Paolo Patierno <[email protected]>
AuthorDate: Thu Apr 23 20:37:31 2026 +0200
MINOR: Removed `setMetadataLogDirectory` unused method (#22128)
Trivial PR to remove an overload of the
`Formatter.setMetadataLogDirectory` which takes the metadata log
directory as an `Optional<String>`, because it's not used. The overall
codebase uses the other overload taking just a String (because most of
the callers have a String to pass) and then wrapping it into an
`Optional`.
Reviewers: Andrew Schofield <[email protected]>
Signed-off-by: Paolo Patierno <[email protected]>
---
.../src/main/java/org/apache/kafka/metadata/storage/Formatter.java | 5 -----
1 file changed, 5 deletions(-)
diff --git
a/metadata/src/main/java/org/apache/kafka/metadata/storage/Formatter.java
b/metadata/src/main/java/org/apache/kafka/metadata/storage/Formatter.java
index a036192fabb..3d3d3226826 100644
--- a/metadata/src/main/java/org/apache/kafka/metadata/storage/Formatter.java
+++ b/metadata/src/main/java/org/apache/kafka/metadata/storage/Formatter.java
@@ -207,11 +207,6 @@ public class Formatter {
return this;
}
- public Formatter setMetadataLogDirectory(Optional<String>
metadataLogDirectory) {
- this.metadataLogDirectory = metadataLogDirectory;
- return this;
- }
-
public Formatter setInitialControllers(DynamicVoters initialControllers) {
this.initialControllers = Optional.of(initialControllers);
return this;
