You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2020/05/22 07:50:11 UTC

[GitHub] [kafka] jiameixie opened a new pull request #8711: KAFKA-9904:Use ThreadLocalConcurrent to Replace Random

jiameixie opened a new pull request #8711:
URL: https://github.com/apache/kafka/pull/8711


   When applicable, use of ThreadLocalRandom rather
   than shared Random objects in concurrent programs
   will typically encounter much less overhead and
   contention.
   
   Change-Id: Idf56adc8cbb4c611e327e639a49d90827a23d947
   Signed-off-by: Jiamei Xie <ji...@arm.com>
   
   *More detailed description of your change,
   if necessary. The PR title and PR message become
   the squashed commit message, so use a separate
   comment to ping reviewers.*
   
   *Summary of testing strategy (including rationale)
   for the feature or bug fix. Unit and/or integration
   tests are expected for any behaviour change and
   system tests should be considered for larger changes.*
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


----------------------------------------------------------------
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.

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



[GitHub] [kafka] jiameixie commented on pull request #8711: KAFKA-9904:Use ThreadLocalConcurrent to Replace Random

Posted by GitBox <gi...@apache.org>.
jiameixie commented on pull request #8711:
URL: https://github.com/apache/kafka/pull/8711#issuecomment-632558263


   > Is this PR different from #8531?
   
   @chia7712 It's the same issue. #8531 has wrong spell for KAFKA.  And I didn't see any PR about this issue in https://issues.apache.org/jira/browse/KAFKA-9904.  So I thought there was no PR.  I'll close this PR. 


----------------------------------------------------------------
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.

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



[GitHub] [kafka] chia7712 commented on pull request #8711: KAFKA-9904:Use ThreadLocalConcurrent to Replace Random

Posted by GitBox <gi...@apache.org>.
chia7712 commented on pull request #8711:
URL: https://github.com/apache/kafka/pull/8711#issuecomment-632552331


   Is this PR different from #8531?


----------------------------------------------------------------
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.

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



[GitHub] [kafka] jiameixie closed pull request #8711: KAFKA-9904:Use ThreadLocalConcurrent to Replace Random

Posted by GitBox <gi...@apache.org>.
jiameixie closed pull request #8711:
URL: https://github.com/apache/kafka/pull/8711


   


----------------------------------------------------------------
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.

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