xumanbu commented on PR #1423: URL: https://github.com/apache/incubator-uniffle/pull/1423#issuecomment-1882921508
> > > ShuffleHandleInfo seems immutable, We can't change it. > > > > > > ShuffleHandleInfo can't change in executor beacuse it is a broadcast value. but we can add a new server to ShuffleHandleInfo#partitionToServers in driver, and ShuffleReader in executor renew a ShuffleHandleInfo by ShuffleManagerClient get latest partitionToServers from Driver. > > Maybe we can use an another variable. If we use ShuffleHandleInfo, it's a little misunderstanding. good suggestion. let me think again. -- 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]
