[GitHub] spark issue #15281: [DOCS] Reorganize explanation of Accumulators and Broadc...

2016-09-29 Thread vanzin
Github user vanzin commented on the issue:

https://github.com/apache/spark/pull/15281
  
Did not merge to branch-2.0, so just left it in master.


---
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.
---

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



[GitHub] spark issue #15281: [DOCS] Reorganize explanation of Accumulators and Broadc...

2016-09-29 Thread vanzin
Github user vanzin commented on the issue:

https://github.com/apache/spark/pull/15281
  
I assume there's no need to run tests for doc changes, so merging to master 
/ 2.0.


---
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.
---

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



[GitHub] spark issue #15281: [DOCS] Reorganize explanation of Accumulators and Broadc...

2016-09-29 Thread jsoltren
Github user jsoltren commented on the issue:

https://github.com/apache/spark/pull/15281
  
There is just one change and it is to the section name:
Strike "Accumulators and Broadcast Variables"
Replace with "Accumulators, Broadcast Variables, and Checkpoints"


---
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.
---

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



[GitHub] spark issue #15281: [DOCS] Reorganize explanation of Accumulators and Broadc...

2016-09-28 Thread vanzin
Github user vanzin commented on the issue:

https://github.com/apache/spark/pull/15281
  
Looks fine but I second Reynold's question. Sounds like no modifications 
from your summary.


---
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.
---

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



[GitHub] spark issue #15281: [DOCS] Reorganize explanation of Accumulators and Broadc...

2016-09-28 Thread rxin
Github user rxin commented on the issue:

https://github.com/apache/spark/pull/15281
  
Thanks - this change makes sense.

Did you change any of the content besides moving them?


---
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.
---

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



[GitHub] spark issue #15281: [DOCS] Reorganize explanation of Accumulators and Broadc...

2016-09-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/15281
  
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 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.
---

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