lianetm commented on PR #17918: URL: https://github.com/apache/kafka/pull/17918#issuecomment-2494880578
Comment regarding the mixed subscription error msg in `SubscriptionState`. https://github.com/apache/kafka/blob/33318bc0f4973b090b1b42dc21ab3ef388af5b91/clients/src/main/java/org/apache/kafka/clients/consumer/internals/SubscriptionState.java#L74 It's truly not clearly stating that we don't allow pattern and re2j pattern together (we don't), but I intentionally didn't change it wondering if it's more of a hassle given that this is expected to be a transition period until we deprecate (eventually remove) the old regex, and then the current msg will be fully accurate as it is. Let me know what you think. -- 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]
