[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-11 Thread janewangfb
Github user janewangfb commented on the issue: https://github.com/apache/spark/pull/18023 @gatorsmile Sure, I could have a follow-up PR to resolve DataFrameNaFunctions.fill. thanks for reviewing this PR. --- If your project is set up for it, you can reply to this e

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-11 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-11 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18023 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 f

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-11 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18023 The last comment is about `DataFrameNaFunctions.fill`. It does not work when `spark.sql.parser.quotedRegexColumnNames` is on. Could you resolve that in the follow-up PR? --- If your project is

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-11 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79535 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79535/testReport)** for PR 18023 at commit [`a5f9c44`](https://github.com/apache/spark/commit/a

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-11 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79535 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79535/testReport)** for PR 18023 at commit [`a5f9c44`](https://github.com/apache/spark/commit/a5

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-11 Thread janewangfb
Github user janewangfb commented on the issue: https://github.com/apache/spark/pull/18023 @gatorsmile regarding: > Could you revert back all the unneeded changes? (in JsonSuite.scala). (I saw this comment in email but didn't find in the PR) I have reverted the unneeded ch

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-11 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79504 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79504/testReport)** for PR 18023 at commit [`d613ff9`](https://github.com/apache/spark/commit/d

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-10 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79504 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79504/testReport)** for PR 18023 at commit [`d613ff9`](https://github.com/apache/spark/commit/d6

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-10 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79503 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79503/testReport)** for PR 18023 at commit [`56e2b83`](https://github.com/apache/spark/commit/5

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-10 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79503 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79503/testReport)** for PR 18023 at commit [`56e2b83`](https://github.com/apache/spark/commit/56

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-10 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79476 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79476/testReport)** for PR 18023 at commit [`8adad7c`](https://github.com/apache/spark/commit/8

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-10 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79476 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79476/testReport)** for PR 18023 at commit [`8adad7c`](https://github.com/apache/spark/commit/8a

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-10 Thread janewangfb
Github user janewangfb commented on the issue: https://github.com/apache/spark/pull/18023 @gatorsmile regarding: > Could you add some test cases in the other parts of the query? For example, group by clauses. Yes, added. --- If your project is set up for it, you can reply to

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-10 Thread janewangfb
Github user janewangfb commented on the issue: https://github.com/apache/spark/pull/18023 @gatorsmile, regarding > That is fine we do not support it, but we still need to add test cases for these negative cases. Thanks! Yes, I have added testcases. --- If your project i

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-10 Thread janewangfb
Github user janewangfb commented on the issue: https://github.com/apache/spark/pull/18023 @gatorsmile, regarding: > @janewangfb If we turn on the flag spark.sql.parser.quotedRegexColumnNames by default, the > following test cases failed. Could you do some investigations? Thanks!

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-07 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18023 @janewangfb That is fine we do not support, but we still need to add test cases for these negative cases. Thanks! --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-07 Thread janewangfb
Github user janewangfb commented on the issue: https://github.com/apache/spark/pull/18023 @gatorsmile regarding > Could you add some test cases in the other parts of the query? For example, group by clauses. I added some tests for aggreation. But for group by, we cannot ha

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-07 Thread janewangfb
Github user janewangfb commented on the issue: https://github.com/apache/spark/pull/18023 @viirya regarding > Do we still care hive.support.quoted.identifiers? If no, please update the PR description accordingly. Yes, we still have hive.support.quoted.identifiers. only whe

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-07 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18023 Could you add some test cases in the other parts of the query? For example, group by clauses. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-07 Thread janewangfb
Github user janewangfb commented on the issue: https://github.com/apache/spark/pull/18023 @gatorsmile, regarding: > Left a comment #18023 (comment) I have already fixed the issue. --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-07 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18023 @janewangfb If we turn on the flag `spark.sql.parser.quotedRegexColumnNames` by default, the following test cases failed. Could you do some investigations? Thanks! ``` org.apache.spark.s

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-07 Thread viirya
Github user viirya commented on the issue: https://github.com/apache/spark/pull/18023 Do we still care `hive.support.quoted.identifiers`? If no, please update the PR description accordingly. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-06 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18023 Left a comment https://github.com/apache/spark/pull/18023#discussion_r126079978 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-05 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79240 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79240/testReport)** for PR 18023 at commit [`65886cd`](https://github.com/apache/spark/commit/6

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-05 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79240 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79240/testReport)** for PR 18023 at commit [`65886cd`](https://github.com/apache/spark/commit/65

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-05 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79239 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79239/testReport)** for PR 18023 at commit [`65e5eec`](https://github.com/apache/spark/commit/6

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-05 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79239 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79239/testReport)** for PR 18023 at commit [`65e5eec`](https://github.com/apache/spark/commit/65

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-05 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79238 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79238/testReport)** for PR 18023 at commit [`d65c462`](https://github.com/apache/spark/commit/d

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-05 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79238 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79238/testReport)** for PR 18023 at commit [`d65c462`](https://github.com/apache/spark/commit/d6

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-07-05 Thread janewangfb
Github user janewangfb commented on the issue: https://github.com/apache/spark/pull/18023 Fixed the failed testcases pointed out by gatorsmile. --- 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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-30 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18023 Let me post a confusing error message. ```Scala withSQLConf(SQLConf.SUPPORT_QUOTED_REGEX_COLUMN_NAME.key -> "true") { Seq((1, 1)).toDF("key", "value").createOrReplaceTempView(

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79004 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79004/testReport)** for PR 18023 at commit [`448c3e2`](https://github.com/apache/spark/commit/4

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79003 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79003/testReport)** for PR 18023 at commit [`4e36ed9`](https://github.com/apache/spark/commit/4

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79004 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79004/testReport)** for PR 18023 at commit [`448c3e2`](https://github.com/apache/spark/commit/44

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #79003 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79003/testReport)** for PR 18023 at commit [`4e36ed9`](https://github.com/apache/spark/commit/4e

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-30 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18023 retest this please --- 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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-30 Thread janewangfb
Github user janewangfb commented on the issue: https://github.com/apache/spark/pull/18023 Jenkins test this please --- 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 wis

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #78955 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78955/testReport)** for PR 18023 at commit [`4e36ed9`](https://github.com/apache/spark/commit/4

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-29 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #78955 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78955/testReport)** for PR 18023 at commit [`4e36ed9`](https://github.com/apache/spark/commit/4e

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #78815 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78815/testReport)** for PR 18023 at commit [`321211d`](https://github.com/apache/spark/commit/3

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #78815 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78815/testReport)** for PR 18023 at commit [`321211d`](https://github.com/apache/spark/commit/32

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #78773 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78773/testReport)** for PR 18023 at commit [`f98207b`](https://github.com/apache/spark/commit/f

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #78773 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78773/testReport)** for PR 18023 at commit [`f98207b`](https://github.com/apache/spark/commit/f9

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-27 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18023 Yes. We need to consider it. > (?i) makes the regex case insensitive. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-27 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/18023 do we need to consider case sensitivity when matching the regex? --- 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #78645 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78645/testReport)** for PR 18023 at commit [`da60368`](https://github.com/apache/spark/commit/d

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #78645 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78645/testReport)** for PR 18023 at commit [`da60368`](https://github.com/apache/spark/commit/da

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-25 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18023 Could you please add another case when no column is matched into `query_regex_column.sql`? Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-23 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18023 Will review it this weekend. Thanks for your work! --- 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 th

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-06-21 Thread janewangfb
Github user janewangfb commented on the issue: https://github.com/apache/spark/pull/18023 Ping. I went through the individual reviewers' comment, it seems I addressed every one. Not sure if I am missing anyone, please let me know. Thanks! --- If your project is set up for it, you c

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #77548 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77548/testReport)** for PR 18023 at commit [`a0e3773`](https://github.com/apache/spark/commit/a

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #77548 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77548/testReport)** for PR 18023 at commit [`a0e3773`](https://github.com/apache/spark/commit/a0

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #77443 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77443/testReport)** for PR 18023 at commit [`a27023c`](https://github.com/apache/spark/commit/a

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #77443 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77443/testReport)** for PR 18023 at commit [`a27023c`](https://github.com/apache/spark/commit/a2

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #77442 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77442/testReport)** for PR 18023 at commit [`779724d`](https://github.com/apache/spark/commit/7

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #77442 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77442/testReport)** for PR 18023 at commit [`779724d`](https://github.com/apache/spark/commit/77

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-23 Thread janewangfb
Github user janewangfb commented on the issue: https://github.com/apache/spark/pull/18023 @cloud-fan, there are difference between `xyz` and xyz. we always need to extract xyz part from `xyz`, so we will need to pattern match. if we do not get pattern matched, we know it is not regex,

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-22 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #77192 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77192/testReport)** for PR 18023 at commit [`0284d01`](https://github.com/apache/spark/commit/0

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-22 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #77192 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77192/testReport)** for PR 18023 at commit [`0284d01`](https://github.com/apache/spark/commit/02

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #77109 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77109/testReport)** for PR 18023 at commit [`48c54aa`](https://github.com/apache/spark/commit/4

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18023 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 #18023: [SPARK-12139] [SQL] REGEX Column Specification

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

[GitHub] spark issue #18023: [SPARK-12139] [SQL] REGEX Column Specification

2017-05-19 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18023 **[Test build #77108 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77108/testReport)** for PR 18023 at commit [`612bedf`](https://github.com/apache/spark/commit/6

  1   2   >