[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14104 Merged build finished. Test PASSed. --- 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

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14104 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62231/ Test PASSed. ---

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14104 **[Test build #62231 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62231/consoleFull)** for PR 14104 at commit

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14104 **[Test build #62231 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62231/consoleFull)** for PR 14104 at commit

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-13 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14104 Jenkins retest this please --- 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

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-12 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14104 @phalodi no need to ping. We don't merge things immediately, especially when non-essential. It gives more time for review. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-12 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen Is this is correct if it is then please merge it otherwise give comment if needed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-11 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen Below is final screen shot as you suggested i hope its looks good please review it and merge it. ![screenshot from 2016-07-11

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-11 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen I put back-tick-quote for foreach, foreachAsync and FutureAction. --- 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

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-11 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen Thanks man i know i am not giving correct language but my intention is also same i hope its final commit :) if its done can you merge it. --- If your project is set up for it, you can

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-11 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen @rxin ![screenshot from 2016-07-11 16-51-53](https://cloud.githubusercontent.com/assets/8075390/16729090/e860ff50-4787-11e6-915e-7e39da6558e3.png) --- If your project is set up for

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-11 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen now its perfect i think at last from table to one line but yeah you are right it good to just mention that spark provide it and not duplicate table of actions. --- If your project is set

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-11 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen sure i will make appropriate changes and push it again --- 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

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-11 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen so what do you think we should not add this api reference in document? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-11 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14104 Although I don't think it's essential to mention this API (it's in the API doc) it wouldn't hurt to point to it here. The mention should be accurate though. I think it's correct to say that the

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-11 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen i know its nothing major just call normal functions in future but still naive user first time who learn scala and spark dont know what are future and all so at least we should add reference

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-10 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen @rxin Please review it and give your comments. ![screenshot from 2016-07-11

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-10 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen i make changes as you suggest i remove the table and add a single below actions table and also change statement to be more clear about non blocking. --- If your project is set up for it,

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-10 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen ok no problem so what you suggest should i remove the table and just add the line below the actions table and give link to scala and java docs? And the line "Spark provide asynchronous

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-10 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14104 I fixed my comment above, sorry. They _do_ block the calling thread, but actions triggered in different threads on one RDD do not block each other. The first sentence isn't correct. My suggestion is

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-10 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen yeah i agree with you its same like non async actions but yeah we should list it because its more useful in real life application when single application running multiple jobs of different

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-10 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @srowen yeah you are right its not blocking calling thread but they execute sequentially right but in async action its return future so its running on different threads soo not run sequentially soo

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-10 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14104 Mostly this just duplicates the non-async documentation. I think it's worth mentioning the existence of the async methods with a pointer to the API docs, but, I don't think this table adds value.

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-09 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @rxin Below are screen shot of Asynchronous Actions inside actions and screen shot of index of programming guide. ![screenshot from 2016-07-09

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-08 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/14104 Can you paste a screenshot of the doc? --- 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

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14104 Merged build finished. Test PASSed. --- 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

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14104 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61979/ Test PASSed. ---

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14104 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61980/ Test PASSed. ---

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14104 **[Test build #61979 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61979/consoleFull)** for PR 14104 at commit

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14104 Merged build finished. Test PASSed. --- 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

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14104 **[Test build #61980 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61980/consoleFull)** for PR 14104 at commit

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14104 **[Test build #61980 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61980/consoleFull)** for PR 14104 at commit

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14104 **[Test build #61979 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61979/consoleFull)** for PR 14104 at commit

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-08 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14104 Jenkins test this please --- 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

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-08 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14104 @phalodi it's a bot, not a person. --- 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

[GitHub] spark issue #14104: [SPARK-16438] Add Asynchronous Actions documentation

2016-07-08 Thread phalodi
Github user phalodi commented on the issue: https://github.com/apache/spark/pull/14104 @AmplabJenkins I dont know i just create the issue on JIRA and make changes and make pull request for merge , I think it is valid pull request because in real world all application need non