lct45 commented on a change in pull request #10813:
URL: https://github.com/apache/kafka/pull/10813#discussion_r646974657



##########
File path: 
streams/src/test/java/org/apache/kafka/streams/processor/internals/ProcessorNodeTest.java
##########
@@ -160,8 +160,8 @@ private void testMetrics(final String 
builtInMetricsVersion) {
     }
 
     @Test
-    public void testTopologyLevelClassCastException() {
-        // Serdes configuration is missing (default will be used which don't 
match the DSL below), which will trigger the new exception
+    public void testTopologyLevelConfigException() {

Review comment:
       I see what you mean, I think it would be valuable to have both. The 
catch for me was trying to figure out how to set streams configs in this test 
set up, is there a way to pass them in somewhere that I'm missing? I haven't 
been able to find anything @mjsax 




-- 
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:
us...@infra.apache.org


Reply via email to