[GitHub] spark pull request: [SPARK-14912][SQL] Propagate data source optio...

2016-04-26 Thread andrewor14 
Github user andrewor14 commented on the pull request:

https://github.com/apache/spark/pull/12688#issuecomment-214830579
  
lgtm


---
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-14912][SQL] Propagate data source optio...

2016-04-26 Thread asfgit 
Github user asfgit closed the pull request at:

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


---
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-14912][SQL] Propagate data source optio...

2016-04-26 Thread yhuai 
Github user yhuai commented on the pull request:

https://github.com/apache/spark/pull/12688#issuecomment-214829104
  
Thanks. Merging to 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 pull request: [SPARK-14912][SQL] Propagate data source optio...

2016-04-26 Thread AmplabJenkins 
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12688#issuecomment-214666977
  
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-14912][SQL] Propagate data source optio...

2016-04-26 Thread AmplabJenkins 
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12688#issuecomment-214666978
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56979/
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-14912][SQL] Propagate data source optio...

2016-04-26 Thread SparkQA 
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12688#issuecomment-214666764
  
**[Test build #56979 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56979/consoleFull)**
 for PR 12688 at commit 
[`5c5e6a6`](https://github.com/apache/spark/commit/5c5e6a687017196b0ce54b4e14087a878f691f2b).
 * 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-14912][SQL] Propagate data source optio...

2016-04-26 Thread SparkQA 
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12688#issuecomment-214637792
  
**[Test build #56979 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56979/consoleFull)**
 for PR 12688 at commit 
[`5c5e6a6`](https://github.com/apache/spark/commit/5c5e6a687017196b0ce54b4e14087a878f691f2b).


---
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-14912][SQL] Propagate data source optio...

2016-04-26 Thread AmplabJenkins 
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12688#issuecomment-214636527
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56973/
Test FAILed.


---
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-14912][SQL] Propagate data source optio...

2016-04-26 Thread AmplabJenkins 
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12688#issuecomment-214636525
  
Merged build finished. Test FAILed.


---
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-14912][SQL] Propagate data source optio...

2016-04-26 Thread SparkQA 
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12688#issuecomment-214636420
  
**[Test build #56973 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56973/consoleFull)**
 for PR 12688 at commit 
[`3d53396`](https://github.com/apache/spark/commit/3d533968a0596a00cb53d8a8d1b228f543a15835).
 * This patch **fails PySpark unit 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-14912][SQL] Propagate data source optio...

2016-04-25 Thread yhuai 
Github user yhuai commented on the pull request:

https://github.com/apache/spark/pull/12688#issuecomment-214622278
  
LGTM


---
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-14912][SQL] Propagate data source optio...

2016-04-25 Thread rxin 
Github user rxin commented on the pull request:

https://github.com/apache/spark/pull/12688#issuecomment-214620464
  
cc @yhuai / @andrewor14 


---
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-14912][SQL] Propagate data source optio...

2016-04-25 Thread SparkQA 
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12688#issuecomment-214620503
  
**[Test build #56973 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56973/consoleFull)**
 for PR 12688 at commit 
[`3d53396`](https://github.com/apache/spark/commit/3d533968a0596a00cb53d8a8d1b228f543a15835).


---
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-14912][SQL] Propagate data source optio...

2016-04-25 Thread rxin 
GitHub user rxin opened a pull request:

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

[SPARK-14912][SQL] Propagate data source options to Hadoop configuration

## What changes were proposed in this pull request?
We currently have no way for users to propagate options to the underlying 
library that rely in Hadoop configurations to work. For example, there are 
various options in parquet-mr that users might want to set, but the data source 
API does not expose a per-job way to set it. This patch propagates the 
user-specified options also into Hadoop Configuration.

## How was this patch tested?
Added a unit test to test both the read path and the write path.

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

$ git pull https://github.com/rxin/spark SPARK-14912

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

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


commit 3d533968a0596a00cb53d8a8d1b228f543a15835
Author: Reynold Xin 
Date:   2016-04-26T05:30:33Z

[SPARK-14912][SQL] Propagate data source options to Hadoop configurations




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