[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-14 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/19708
  
**[Test build #83864 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83864/testReport)**
 for PR 19708 at commit 
[`db565f6`](https://github.com/apache/spark/commit/db565f6b0a57f436f85c32fe7d05b027908c7a9b).


---

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



[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-14 Thread onursatici
Github user onursatici commented on the issue:

https://github.com/apache/spark/pull/19708
  
Yeah PR description reflects the latest changes


---

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



[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/19708
  
Merged build finished. Test PASSed.


---

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



[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/19708
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83811/
Test PASSed.


---

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



[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-13 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/19708
  
**[Test build #83811 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83811/testReport)**
 for PR 19708 at commit 
[`56f48f3`](https://github.com/apache/spark/commit/56f48f3aa0c33ddeee86ce330b54b6d3f712ffae).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

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



[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-13 Thread hvanhovell
Github user hvanhovell commented on the issue:

https://github.com/apache/spark/pull/19708
  
This looks good. I was wondering if we shouldn't also take a look at data 
source operations like `InsertIntoDataSourceCommand`?

Could you also add a test?


---

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



[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-13 Thread gatorsmile
Github user gatorsmile commented on the issue:

https://github.com/apache/spark/pull/19708
  
Generally, this looks good to me. 

cc @cloud-fan @hvanhovell 


---

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



[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-13 Thread gatorsmile
Github user gatorsmile commented on the issue:

https://github.com/apache/spark/pull/19708
  
Just want to confirm whether the examples in the PR description are the 
ones based on the latest updates?


---

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



[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-13 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/19708
  
**[Test build #83811 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83811/testReport)**
 for PR 19708 at commit 
[`56f48f3`](https://github.com/apache/spark/commit/56f48f3aa0c33ddeee86ce330b54b6d3f712ffae).


---

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



[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

https://github.com/apache/spark/pull/19708
  
Merged build finished. Test FAILed.


---

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



[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue:

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


---

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



[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-13 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/19708
  
**[Test build #83810 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83810/testReport)**
 for PR 19708 at commit 
[`ae091ec`](https://github.com/apache/spark/commit/ae091ec677ab84bb201d429ce829f9ef7035836f).
 * This patch **fails Scala style tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---

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



[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...

2017-11-13 Thread SparkQA
Github user SparkQA commented on the issue:

https://github.com/apache/spark/pull/19708
  
**[Test build #83810 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83810/testReport)**
 for PR 19708 at commit 
[`ae091ec`](https://github.com/apache/spark/commit/ae091ec677ab84bb201d429ce829f9ef7035836f).


---

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