cadonna commented on a change in pull request #8902:
URL: https://github.com/apache/kafka/pull/8902#discussion_r444319623
##########
File path:
streams/src/main/java/org/apache/kafka/streams/state/internals/MeteredKeyValueStore.java
##########
@@ -103,8 +104,12 @@ public void init(final ProcessorContext context,
@SuppressWarnings("unchecked")
void initStoreSerde(final ProcessorContext context) {
+ final InternalProcessorContext internalProcessorContext =
(InternalProcessorContext) context;
+ final String storeName = name();
+ final String changelogTopic =
internalProcessorContext.changelogFor(storeName);
serdes = new StateSerdes<>(
- ProcessorStateManager.storeChangelogTopic(context.applicationId(),
name()),
+ changelogTopic != null
Review comment:
It would be `null` for the case where logging is disabled. In this case
we would still need the store serdes to read records from the store. We just
pass in the default changelog name, so that one can still use a changelog name
in the serde, e.g., as subject name for Confluent's schema registry.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
