[GitHub] spark pull request: [SPARK-11914] [SQL] Support coalesce and repar...

2015-11-24 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/9899


---
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 pull request: [SPARK-11914] [SQL] Support coalesce and repar...

2015-11-24 Thread rxin
Github user rxin commented on the pull request:

https://github.com/apache/spark/pull/9899#issuecomment-159442306
  
Thanks - I'm merging this in.



---
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 pull request: [SPARK-11914] [SQL] Support coalesce and repar...

2015-11-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9899#issuecomment-158859749
  
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
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 pull request: [SPARK-11914] [SQL] Support coalesce and repar...

2015-11-22 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9899#issuecomment-158859716
  
**[Test build #46507 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46507/consoleFull)**
 for PR 9899 at commit 
[`db4accc`](https://github.com/apache/spark/commit/db4accceb3fff4c4018662da3d12df40015550f8).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 pull request: [SPARK-11914] [SQL] Support coalesce and repar...

2015-11-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/9899#issuecomment-158859750
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46507/
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
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 pull request: [SPARK-11914] [SQL] Support coalesce and repar...

2015-11-22 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/9899#issuecomment-158847634
  
**[Test build #46507 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46507/consoleFull)**
 for PR 9899 at commit 
[`db4accc`](https://github.com/apache/spark/commit/db4accceb3fff4c4018662da3d12df40015550f8).


---
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 pull request: [SPARK-11914] [SQL] Support coalesce and repar...

2015-11-22 Thread gatorsmile
GitHub user gatorsmile opened a pull request:

https://github.com/apache/spark/pull/9899

[SPARK-11914] [SQL] Support coalesce and repartition in Dataset APIs

This PR is to provide two common `coalesce` and `repartition` in Dataset 
APIs. 

After reading the comments of SPARK-, I am unclear about the plan for 
supporting re-partitioning in Dataset APIs. Currently, both RDD APIs and 
Dataframe APIs provide users such a flexibility to control the number of 
partitions. 

In most traditional RDBMS, they expose the number of partitions, the 
partitioning columns, the table partitioning methods to DBAs for performance 
tuning and storage optimization. Normally, the changes could largely affect the 
query performance. Since the actual performance depends on the workload types, 
I do not think we can always find the best partitioning strategy for all the 
scenarios. 

I am wondering if Dataset APIs are planning to hide these APIs from users? 
Feel free to reject my PR if it does not match your plan. 

Thank you for your answers. @marmbrus @rxin @cloud-fan 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/gatorsmile/spark coalesce

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/9899.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #9899


commit 41d3adecfa1dbbab8165587d38f99036e443b465
Author: gatorsmile 
Date:   2015-11-23T03:12:42Z

Support coalesce and repartition in Dataset APIs

commit db4accceb3fff4c4018662da3d12df40015550f8
Author: gatorsmile 
Date:   2015-11-23T03:14:42Z

style clean




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