Skip to content

Commit 1938d32

Browse files
authored
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 <aschofield@confluent.io> Signed-off-by: Paolo Patierno <ppatierno@live.com>
1 parent c946802 commit 1938d32

1 file changed

Lines changed: 0 additions & 5 deletions

File tree

  • metadata/src/main/java/org/apache/kafka/metadata/storage

metadata/src/main/java/org/apache/kafka/metadata/storage/Formatter.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -207,11 +207,6 @@ public Formatter setMetadataLogDirectory(String metadataLogDirectory) {
207207
return this;
208208
}
209209

210-
public Formatter setMetadataLogDirectory(Optional<String> metadataLogDirectory) {
211-
this.metadataLogDirectory = metadataLogDirectory;
212-
return this;
213-
}
214-
215210
public Formatter setInitialControllers(DynamicVoters initialControllers) {
216211
this.initialControllers = Optional.of(initialControllers);
217212
return this;

0 commit comments

Comments
 (0)