[GitHub] incubator-spark pull request: Patch for SPARK-942

[pwendell]

Github user pwendell commented on the pull request:

    https://github.com/apache/incubator-spark/pull/180#issuecomment-35968583
  
    Hmm okay I'm still wondering if there is a path for in-memory data where 
two copies are created:
    
    Cache manager runs:
    ```
    val elements = new ArrayBuffer[Any]
    elements ++= computedValues
    blockManager.put(key, elements, storageLevel, tellMaster = true)
    ```
    
    Block manager runs:
    ```
    doPut(blockId, Left(values.toIterator), level, tellMaster)
    ```
    
    DoPut runs:
    ```
    val res = memoryStore.putValues(blockId, values, level, true)
    ```
    
    MemoryStore runs:
    ```
    val valueEntries = new ArrayBuffer[Any]()
          valueEntries ++= values
    ```


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---