apoorvmittal10 commented on PR #17957: URL: https://github.com/apache/kafka/pull/17957#issuecomment-2512452127
> > thanks for the approval, can you please merge the PR? > > @apoorvmittal10 Do you have any feedback for this PR? Aside from the lock scope, all other changes are ready to be merged. Mine is more on the way the locks are acquired and release. There is no point to keep holding the locks when a async API has been called. @adixitconfluent are we saying to have that change in the next PR? If yes, then it seems fine to me. -- 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
