[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-27 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/17397 Submitted the PR https://github.com/apache/spark/pull/17448. Please check whether the fix is appropriate. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-27 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/17397 Sorry, I did not catch it. Let me submit a follow-up to fix this test case issue. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-27 Thread ueshin
Github user ueshin commented on the issue: https://github.com/apache/spark/pull/17397 @hvanhovell `DataSourceScanExecRedactionSuite` introduced by this pr seems to cause another issue when the `basePath` is too long to redact: https://amplab.cs.berkeley.edu/jenkins/view/Spark

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread ueshin
Github user ueshin commented on the issue: https://github.com/apache/spark/pull/17397 I see, thanks! --- 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

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread hvanhovell
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/17397 @ueshin here you go: https://github.com/apache/spark/pull/17420 --- 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 doe

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread ueshin
Github user ueshin commented on the issue: https://github.com/apache/spark/pull/17397 @hvanhovell This seems to have broken the 2.10 build: ``` [error] /home/jenkins/workspace/spark-master-compile-maven-scala-2.10/core/src/main/scala/org/apache/spark/internal/config/Config

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/17397 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 w

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17397 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75174/ Test PASSed. ---

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17397 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 e

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17397 **[Test build #75174 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75174/testReport)** for PR 17397 at commit [`73cd8c4`](https://github.com/apache/spark/commit/7

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/17397 LGTM pending Jenkins --- 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

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17397 **[Test build #75174 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75174/testReport)** for PR 17397 at commit [`73cd8c4`](https://github.com/apache/spark/commit/73

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17397 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 e

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17397 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75166/ Test PASSed. ---

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17397 **[Test build #75166 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75166/testReport)** for PR 17397 at commit [`cff62f6`](https://github.com/apache/spark/commit/c

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/17397 Thanks! @hvanhovell Could we also output the conf name? ``` spark-[{. is not a valid regex java.lang.IllegalArgumentException: spark-[{. is not a valid regex at org.apache

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-24 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17397 **[Test build #75166 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75166/testReport)** for PR 17397 at commit [`cff62f6`](https://github.com/apache/spark/commit/cf

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17397 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75100/ Test PASSed. ---

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17397 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 e

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-23 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17397 **[Test build #75100 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75100/testReport)** for PR 17397 at commit [`2c160ea`](https://github.com/apache/spark/commit/2

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-23 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/17397 Maybe we can verify the user input or capture the exception? If users input incorrect value, it could get a confusing error message. ``` Unclosed group near index 19 spark-~!@#$%^

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-23 Thread liufengdb
Github user liufengdb commented on the issue: https://github.com/apache/spark/pull/17397 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 fea

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-23 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17397 **[Test build #75100 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75100/testReport)** for PR 17397 at commit [`2c160ea`](https://github.com/apache/spark/commit/2c

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17397 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75093/ Test FAILed. ---

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-23 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17397 **[Test build #75093 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75093/testReport)** for PR 17397 at commit [`2648ce5`](https://github.com/apache/spark/commit/2

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17397 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 e

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-23 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/17397 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

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-23 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17397 **[Test build #75093 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75093/testReport)** for PR 17397 at commit [`2648ce5`](https://github.com/apache/spark/commit/26

[GitHub] spark issue #17397: [SPARK-20070][SQL] Redact DataSourceScanExec treeString

2017-03-23 Thread hvanhovell
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/17397 cc @liufengdb --- 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