AndrewJSchofield commented on code in PR #23988:
URL: https://github.com/apache/kafka/pull/23988#discussion_r4394595671
##########
streams/src/main/java/org/apache/kafka/streams/StreamsConfig.java:
##########
@@ -583,6 +583,22 @@ public class StreamsConfig extends AbstractConfig {
public static final String DEFAULT_VALUE_SERDE_CLASS_CONFIG =
"default.value.serde";
private static final String DEFAULT_VALUE_SERDE_CLASS_DOC = "Default
serializer / deserializer class for value that implements the
<code>org.apache.kafka.common.serialization.Serde</code> interface. ";
+ /** {@code default.list.key.serde.inner} */
+ @SuppressWarnings("WeakerAccess")
+ public static final String DEFAULT_LIST_KEY_SERDE_INNER_CLASS =
CommonClientConfigs.DEFAULT_LIST_KEY_SERDE_INNER_CLASS;
Review Comment:
There could perhaps be `o.a.k.common.serialization.SerializerConfigs` or
similar because the public classes in that package do still refer to
`CommonClientConfigs` and don't have any legitimate alternative as far as I can
see.
--
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]
