Github user squito commented on the issue:

    https://github.com/apache/spark/pull/19041
  
    thanks for the updates @brad-kaiser.  I think I understand and don't have 
any major concerns.  It doesn't seem easy to use the LRU from MemoryStore, so 
can set that aside for now
    
    btw as you push updates, I'd prefer to just add new commits on top (even 
merges to master), as that makes it easier for reviewers to see incremental 
changes.


---

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

Reply via email to