[GitHub] spark issue #19708: [WIP][SPARK-22479][SQL] Exclude credentials from Saveint...
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...
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...
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...
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...
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...
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...
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...
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...
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...
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...
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...
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...
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