[GitHub] spark pull request: [SPARK-5865] [WIP] [API DOC] Add doc warnings ...

2016-01-22 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10874#issuecomment-173956442 I think it's OK, though I might say something like "all the data is loaded into the driver's memory" rather than "the whole _thing_ ..." Very minor preference. --- If

[GitHub] spark pull request: [SPARK-5865] [WIP] [API DOC] Add doc warnings ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10874#issuecomment-173778315 Can one of the admins verify this patch? --- 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 p

[GitHub] spark pull request: [SPARK-5865] [WIP] [API DOC] Add doc warnings ...

2016-01-21 Thread Wenpei
GitHub user Wenpei opened a pull request: https://github.com/apache/spark/pull/10874 [SPARK-5865] [WIP] [API DOC] Add doc warnings for methods that return local data structures @rxin @srowen I work out note message for rdd.take function, please help to review. If it's