fvaleri commented on code in PR #22977:
URL: https://github.com/apache/kafka/pull/22977#discussion_r3688992465
##########
storage/src/testFixtures/java/org/apache/kafka/server/log/remote/storage/LocalTieredStorage.java:
##########
@@ -132,9 +132,9 @@ public final class LocalTieredStorage implements
RemoteStorageManager {
public static final String ENABLE_DELETE_API_CONFIG = "delete.enable";
/**
- * The ID of the broker which owns this instance of {@link
LocalTieredStorage}.
+ * The ID of the node which owns this instance of {@link
LocalTieredStorage}.
*/
- public static final String BROKER_ID = "broker.id";
Review Comment:
Should we also deprecate this one in 4.4 and remove only in 5.0? External
consumers of Kafka test JAR would get a compile error.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
