ableegoldman commented on code in PR #18150:
URL: https://github.com/apache/kafka/pull/18150#discussion_r1885482579
##########
streams/src/main/java/org/apache/kafka/streams/kstream/internals/graph/StatefulProcessorNode.java:
##########
@@ -47,11 +47,9 @@ public StatefulProcessorNode(final ProcessorParameters<K, V,
?, ?> processorPara
* Create a node representing a stateful processor, where the named stores
have already been registered.
*/
public StatefulProcessorNode(final String nodeName,
- final ProcessorParameters<K, V, ?, ?>
processorParameters,
- final String[] storeNames) {
+ final ProcessorParameters<K, V, ?, ?>
processorParameters) {
super(nodeName, processorParameters);
-
- this.storeNames = storeNames;
+ this.storeNames = null;
Review Comment:
I don't think we can change this yet since not all StatefulProcessorNode
users have been migrated yet. For example the FKJ. Pretty sure this is what's
causing the test failures.
--
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]
