advancedxy commented on issue #407:
URL: 
https://github.com/apache/incubator-uniffle/issues/407#issuecomment-1352990027

   > 
https://github.com/apache/spark/blob/0fd1f85f16502d1d4222cf3d7abfd5e2b86464e6/sql/core/src/test/scala/org/apache/spark/sql/execution/adaptive/AdaptiveQueryExecSuite.scala#L228
   
   I didn't get a change to reproduce this case. I will do it in this week. 
   
   I still think it should be cached when necessary and selectively. 
   
   > If we cache it, we don't even need to flush it to the disk
   
   Well, I'm for this idea as I'd like to reduce small I/Os to disk as much as 
possible. But memory is still an important factor to consider, as we want to 
deploy Uniffle shuffle server with smaller memory, say 32GB or 48GB.


-- 
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-unsubscr...@uniffle.apache.org

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

Reply via email to