Github user gengliangwang commented on the issue:

    https://github.com/apache/spark/pull/20013
  
    The major concern is that with these code changes,  the memory usage will 
be much larger with `InMemoryStore`.
    Also building so many new indexes just for getting `computedQuantiles`,  
seems overkilling.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to