zuston commented on PR #1763:
URL: 
https://github.com/apache/incubator-uniffle/pull/1763#issuecomment-2167075936

   > ping @zuston @jerqi @advancedxy @EnricoMi
   
   Sorry for the late review. I have recogized this problem, but I have another 
away to optimize this by using the linkedHashMap, and could mix the flush + 
cache blocks into this only one linkedHashmap, the design is done in rust 
implement:
   
   
![image](https://github.com/apache/incubator-uniffle/assets/8609142/4dd84535-b84a-4365-901b-b90f6c044a34)
   
   Anyway, the current impl is OK for me if you don't want to bring too much 
modifications. @xianjingfeng 
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to