mimaison commented on code in PR #18930:
URL: https://github.com/apache/kafka/pull/18930#discussion_r2140310506
##########
clients/src/main/java/org/apache/kafka/common/config/AbstractConfig.java:
##########
@@ -65,6 +65,11 @@ public class AbstractConfig {
public static final String AUTOMATIC_CONFIG_PROVIDERS_PROPERTY =
"org.apache.kafka.automatic.config.providers";
public static final String CONFIG_PROVIDERS_CONFIG = "config.providers";
+ public static final String CONFIG_PROVIDERS_DOC =
+ "Comma-separated names of <code>ConfigProvider</code> classes,
loaded and used "
+ + "in the order specified. Implementing the interface "
Review Comment:
The load order does not matter. What about something like:
```
Comma-separated names of classes implementing the
<code>ConfigProvider</code> interface. This allows loading to load
configuration data, such as passwords, API keys, or other credentials, from
external sources.
```
Maybe we can also include `See <a
href="https://kafka.apache.org/documentation/#config_providers";>Configuration
Providers</a> for examples`.
WDYT?
--
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]
