maobaolong commented on PR #2132: URL: https://github.com/apache/incubator-uniffle/pull/2132#issuecomment-2430666596
@zuston Thanks for the discussion offline. I think I need to introduce the motivation of this PR. The motivation of this PR aimed to resolve the scenario for our regression test, it is a simple test with tiny shuffle data as the description given, without this PR, the coordinator cannot collect any application information. For the production scenario, it could be a little better, since lots of app have a huge mount of shuffle data, but the risky of lost last heartbeat still exist. As a tradeoff, I add a configure option(default to false to keep the original behavior), it can be set to true if we need to get the exact information of application, and keep to false by default. -- 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]
