Christo Lolov
Christo Lolov
I find this refactor quite readable. Looks good to me :)
Hey @ijuma! Divij is taking some time off, but I can have a look in the meantime and try to get it over the line.
Okay @dplavcic! Thanks both for picking it up and for letting us know :)
Hello @ijuma, @ableegoldman, @wycccccc! I have been working on trying to upgrade Kafka Streams from JUnit 4 to JUnit 5 (https://issues.apache.org/jira/browse/KAFKA-7342) and I was wondering what is the state of...
Thank you for the reply @cadonna! Okay, I will wait for a week starting today and I will try to reach out to @wycccccc over email as well should I...
Okay, your reasoning makes sense to me. Would you not need to add the class to the list in https://github.com/apache/kafka/blob/trunk/build.gradle#L397-L416?
Got it, okay, looks good to me, thanks for the detailed explanation :)
Looks great! 👍
Hello! This looks like a very interesting find. Could you write a test which tests this? I would personally also be interested to learn what tools you used to fuzz...
Tagging @mdedetrich @cadonna @ijuma for visibility and reviews :)