k-apol commented on code in PR #20326:
URL: https://github.com/apache/kafka/pull/20326#discussion_r2280623447


##########
streams/src/main/java/org/apache/kafka/streams/processor/internals/InternalTopicManager.java:
##########
@@ -113,7 +113,7 @@ public InternalTopicManager(final Time time,
         }
     }
 
-    static class ValidationResult {
+    public static class ValidationResult {

Review Comment:
   I will need to use it from inside `KafkaStreams.java` - the new `init()` 
method will need to make decisions based off of the validation results of the 
passed topic config, and I made it public here so that I will be able to do so 
on the next PR. I suppose it can wait until the actual implementation of 
`init()` if you would prefer, or maybe you have an alternative approach? 
   
   [Here is the code I am referencing from the (now closed) previous 
PR](https://github.com/k-apol/kafka/pull/1/files#diff-ab27af136b0c45ed402ec44368a91380b018bf06f1a9722324fe6be8d5220f7dR388)



-- 
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

Reply via email to