Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22511
I think the deal with the dispose in TorrentBroadcast is that it's
definitely needed in the local read case, but may need adjustments in the
remote read case.
The local read case (`doGetLocalBytes`) can return a `ByteBufferBlockData`
that needs to be disposed by the caller (check comment "The block was not found
on disk, so serialize an in-memory copy:", where this happens).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
