waitinfuture commented on PR #2373:
URL: https://github.com/apache/celeborn/pull/2373#issuecomment-2040911125

   > It has been a while since I looked at this PR - but as formulated, the 
split into subranges is deterministic (if it is not, it should be made so). 
With that in place, this would not be an issue ... (I will take a deeper look 
later next week, but do let me know if I am missing something so that I can add 
that to my analysis)
   
   > the split into subranges is deterministic
   
   The way Celeborn splits partition is not deterministic with stage rerun, for 
example any push failure will cause split, so I'm afraid this statement does 
not hold...


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

Reply via email to