mnegodaev commented on PR #11442: URL: https://github.com/apache/kafka/pull/11442#issuecomment-1163436844
> I've marked this as needing a KIP, though I'm not yet sure whether that is the case. These SMTs are technically part of the API, since they appear in STM configurations, which are considered part of the public API. I'll keep looking into this to figure out if this is really the case, but it's probably safer to assume a KIP is needed. @rhauch thank you for your review, I've really missed the fact I've broken current behavior on empty schema name! I've fixed this and added more corner tests cases on the schema update. Also I've made an effort to prettify our tests to make them looking more uniform and readable. -- 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