[PR] KAFKA-19347 Don't update timeline data structures in createAcls [kafka]

["mumrah (via GitHub)" <gi...@apache.org>]

mumrah opened a new pull request, #19840:
URL: https://github.com/apache/kafka/pull/19840

   This patch fixes a problem in AclControlManager where we are updating the timeline data structures prematurely. 


-- 
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-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] KAFKA-19347 Don't update timeline data structures in createAcls [kafka]

["mumrah (via GitHub)" <gi...@apache.org>]

mumrah commented on PR #19840:
URL: https://github.com/apache/kafka/pull/19840#issuecomment-2919501774

   @ahuang98 i was reading through the ACLs code on the controller to remember how we deal with batches of ACLs and I just noticed the `put` call in createAcls method as well as in the replay method.


-- 
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-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] KAFKA-19347 Don't update timeline data structures in createAcls [kafka]

["cmccabe (via GitHub)" <gi...@apache.org>]

cmccabe merged PR #19840:
URL: https://github.com/apache/kafka/pull/19840


-- 
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-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org