Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12199
---
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 ena
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206488388
Merging into master 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
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/12199#discussion_r58697136
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/parser/ParseUtils.java
---
@@ -1,135 +0,0 @@
-/**
- * Licensed to the Apache Soft
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206341798
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206341797
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206341350
**[Test build #55106 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55106/consoleFull)**
for PR 12199 at commit
[`e28ddd2`](https://g
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/12199#discussion_r58688523
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/parser/ParseUtils.java
---
@@ -1,135 +0,0 @@
-/**
- * Licensed to the Apache S
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206289821
**[Test build #2758 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2758/consoleFull)**
for PR 12199 at commit
[`0619067`](https://
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206286924
**[Test build #55106 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55106/consoleFull)**
for PR 12199 at commit
[`e28ddd2`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206220796
**[Test build #2758 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2758/consoleFull)**
for PR 12199 at commit
[`0619067`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206214230
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206214232
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206214074
**[Test build #55095 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55095/consoleFull)**
for PR 12199 at commit
[`0619067`](https://g
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/12199#discussion_r58667228
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ParserUtilsSuite.scala
---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Ap
Github user hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206210010
Yay! 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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206146082
**[Test build #55095 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55095/consoleFull)**
for PR 12199 at commit
[`0619067`](https://gi
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206141117
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
enabl
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206138706
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206138708
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206138699
**[Test build #55090 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55090/consoleFull)**
for PR 12199 at commit
[`e54bcc6`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12199#issuecomment-206138185
**[Test build #55090 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55090/consoleFull)**
for PR 12199 at commit
[`e54bcc6`](https://gi
GitHub user sarutak opened a pull request:
https://github.com/apache/spark/pull/12199
[SPARK-14426][SQL] Merge PerserUtils and ParseUtils
## What changes were proposed in this pull request?
We have ParserUtils and ParseUtils which are both utility collections for
use during
22 matches
Mail list logo