chickenchickenlove commented on PR #21677: URL: https://github.com/apache/kafka/pull/21677#issuecomment-4047144636
@mjsax Thanks for the feedback! It looks like we need to take a closer look, but for now I think it’s fine to keep the asInternalProcessorContext(stateStoreContext) code as is. It seems to work as a guardrail to prevent certain processors from being executed in test code, but I can’t be fully sure since it may serve other purposes as well. The current PR is structured as (1). I’ll rebase it onto trunk and try implementing it as (2). The goal is to implement (2) so that MockProcessorContext properly implements the unimplemented methods required by InternalProcessorContext. What do 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]
