You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@shardingsphere.apache.org by "sandynz (via GitHub)" <gi...@apache.org> on 2023/03/15 11:22:07 UTC

[GitHub] [shardingsphere] sandynz commented on pull request #24558: Avoid ConcurrentHashMap recursive update in ShardingSphereServiceLoader.getServiceInstances

sandynz commented on PR #24558:
URL: https://github.com/apache/shardingsphere/pull/24558#issuecomment-1469825958

   I tested different implementations with JMH
   - Replace ConcurrentHashMap to ConcurrentSkipListMap, one line changed (not committed). Performance is bad than before.
   - Use ConcurrentHashMap, lazy load (Commit reverted). Performance is a little bad than before.
   - Use ConcurrentHashMap, small range locks. Performance is similar as before.
   


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

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