AndrewJSchofield commented on PR #18929: URL: https://github.com/apache/kafka/pull/18929#issuecomment-2785568108
@JimmyWang6 Thanks for taking another look at this. I don't think you've discovered problems, just realised the manner in which this operation will have to work. I would look at https://github.com/apache/kafka/pull/19328 to see how I approached a similar admin operation. I do not think you need a new method in the Persister interface. I think you'll be able to do it with the existing write operation. I may be wrong, but that's where I would start. The GroupMetadataManager does not have access to the Persister. This is why we have put the calls to the Persister in the GroupCoordinatorService, as you'll see in the PR I linked. Let me know if you need additional information. @smjn is also a good source of information on this area since he knows all about the Persister and how to use it. -- 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
