Danny02 opened a new pull request, #12608: URL: https://github.com/apache/kafka/pull/12608
Why: It is an interesting question what should be the result when merging a KStream with itself. Should the merge duplicate the messages or should it be a noop. I think the only reasonable solution is to duplicate the messages because there are many different ways to disguise a KStream (e.g. adding peek operation on it). It is therefore impossible to implement the solution where it is a noop. How does it help with resolving the issue: This change makes the behavior of the merge operation consistent. Side effects: Using a list instead of a set for the parent nodes could have side effects. The test suit did not detect any. -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
