Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19222
@tejasapatil Thank you for your comment. I hope that benchmark result is a
response to concern about virtual method raised by @hvanhovell.
@hvanhovell **What do you think?**
As a long term plan, it would be good to all (low-level) memory access
(i.e. thru `Platform`) would be in form of `MemoryBlock`. There are two
reasons. One is easy to understand. The other is performance.
This transition would be step by step. I prepared [another
refactoring](https://github.com/apache/spark/pull/19472) for some classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
