hiboyang commented on pull request #31876: URL: https://github.com/apache/spark/pull/31876#issuecomment-815491516
> > Echoing on previous suggestion to expose BlockManagerId and make it extensible, do we want to try that approach? We could rename BlockManagerId to some other generic name like BlockLocation. > > According to [#31876 (comment)](https://github.com/apache/spark/pull/31876#discussion_r598465454) , [#31876 (comment)](https://github.com/apache/spark/pull/31876#discussion_r601270461), I think we're less prefer to the way of exposing `BlockManagerId`. Yeah, in this case, we need to deal with those `asInstanceOf[BlockManagerId]` type cast to make it not fail for other location type. Any thoughts there? -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
