spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kiszk <...@git.apache.org>
Subject [GitHub] spark issue #19222: [SPARK-10399][CORE][SQL] Introduce multiple MemoryBlocks...
Date Fri, 13 Oct 2017 04:26:13 GMT
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-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Mime
View raw message