squito commented on issue #23438: [SPARK-26525][SHUFFLE]Fast release ShuffleBlockFetcherIterator on completion of the iteration URL: https://github.com/apache/spark/pull/23438#issuecomment-458334792 @szhem thanks for pointing to that PR, you are right this is really similar. IMO, its not unreasonable to add this api. Users aren't implementing `TaskContext` themselves, they are just using it, so I don't think there is a compatibility problem. We could mark as DeveloperApi. Or really make it an internal thing, by have it only be a method on `TaskContextImpl`, and have `ShuffleBlockFetcherIterator` call it via an instanceof check.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
