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

Posted to dev@uniffle.apache.org by GitBox <gi...@apache.org> on 2023/01/12 09:29:26 UTC

[GitHub] [incubator-uniffle] zjf2012 commented on issue #475: [Improvement] It's unnecessary to use ConcurrentHashMap for "partitionToBlockIds" in RssShuffleWriter

zjf2012 commented on issue #475:
URL: https://github.com/apache/incubator-uniffle/issues/475#issuecomment-1380040161

   > Will it bring too much cost?
   
   I cannot tell how much cost it is. But you can imagine we need to get lock first before each map operation in each shuffle write partition. And it's counter-intuitive to use lock whilst it's unnecessary. We should avoid lock as much as possible.


-- 
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: dev-unsubscribe@uniffle.apache.org

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