rickyma commented on PR #1627: URL: https://github.com/apache/incubator-uniffle/pull/1627#issuecomment-2044343843
> > Putting the storageManager into the event as a member variable does not solve the problem, and it is essentially the same as using the eventOfUnderStorageManagers cache. > > The difference is that the key wont be expired. But it won't solve the following problem, the storageManager will always be the same within the event: > However, after the event fails, it is not appropriate to continue taking the original storageManager from the cache(because events usually fail due to high IO pressure or disk damage or disk full) -- 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]
