Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-122633649
@adrian-wang I had some time this weekend and added concat/concat_ws with
code gen. I'm going to close this one. Thanks a lot.
---
If your project is set up for it, you
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/6775
---
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
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32701067
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -313,3 +312,78 @@ case class StringLength(child:
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32701341
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -313,3 +312,78 @@ case class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32701661
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -313,3 +312,78 @@ case class
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32702637
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -313,3 +312,78 @@ case class
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32700753
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -313,3 +312,78 @@ case class StringLength(child:
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32700846
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala
---
@@ -172,7 +174,7 @@ object FunctionRegistry {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32700683
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -85,6 +86,52 @@ class DataFrameFunctionsSuite extends QueryTest {
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32691936
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -85,6 +86,52 @@ class DataFrameFunctionsSuite extends
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112999257
[Test build #35069 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35069/consoleFull)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112998904
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112998942
Merged build started.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-113022738
[Test build #35069 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35069/console)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-113022749
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
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32700704
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -313,3 +312,78 @@ case class StringLength(child:
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32700685
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/StringFunctionsSuite.scala
---
@@ -226,5 +226,13 @@ class StringFunctionsSuite
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112717485
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112717437
[Test build #35026 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35026/console)
for PR 6775 at commit
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32597670
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,76 @@ case class
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32597720
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,76 @@ case class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112671324
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112671304
[Test build #35022 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35022/console)
for PR 6775 at commit
Github user chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112668948
Can you rebase this PR?
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112670743
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112670726
Merged build triggered.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112670829
[Test build #35022 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35022/consoleFull)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112674011
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112674030
Merged build started.
---
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 adrian-wang commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112719572
@rxin
---
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 chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32626256
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -85,6 +86,52 @@ class DataFrameFunctionsSuite extends
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32626959
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -313,3 +312,78 @@ case class
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112675748
[Test build #35026 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35026/consoleFull)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112415109
Build started.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112416596
[Test build #34980 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34980/consoleFull)
for PR 6775 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112451125
[Test build #34980 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34980/console)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112451176
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112654961
[Test build #35015 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35015/console)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112654982
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112631381
Build triggered.
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112631467
[Test build #35015 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35015/consoleFull)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112631388
Build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112360032
Build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112360019
Build triggered.
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112360185
[Test build #34976 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34976/consoleFull)
for PR 6775 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112386752
[Test build #34976 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34976/console)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112386780
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-112392777
Build triggered.
---
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 user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32487610
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -914,6 +914,42 @@ object functions {
def ceil(columnName: String):
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32400671
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,65 @@ case class
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32400743
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -914,6 +914,42 @@ object functions {
def ceil(columnName: String):
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32395410
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -914,6 +914,42 @@ object functions {
def ceil(columnName: String):
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32395319
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,65 @@ case class
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32379119
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,65 @@ case class
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32379216
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,65 @@ case class
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32379043
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,65 @@ case class
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32379137
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -914,6 +914,42 @@ object functions {
def ceil(columnName: String):
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32379255
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,65 @@ case class
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32379147
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -85,6 +86,45 @@ class DataFrameFunctionsSuite extends
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32379071
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,65 @@ case class
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32379225
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -914,6 +914,42 @@ object functions {
def ceil(columnName: String):
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32379201
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -914,6 +914,42 @@ object functions {
def ceil(columnName: String):
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32389816
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,65 @@ case class
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32389968
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,65 @@ case class
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32389929
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,65 @@ case class
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32390080
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -914,6 +914,42 @@ object functions {
def ceil(columnName: String):
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32391151
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,65 @@ case class
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32391184
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -914,6 +914,42 @@ object functions {
def ceil(columnName: String):
Github user adrian-wang commented on a diff in the pull request:
https://github.com/apache/spark/pull/6775#discussion_r32390057
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -315,4 +315,65 @@ case class
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111396465
[Test build #34752 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34752/console)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111396481
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111369425
[Test build #34752 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34752/consoleFull)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111369374
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111369386
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111365825
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111365822
[Test build #34748 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34748/console)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111360739
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111360793
Merged build started.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111361943
[Test build #34748 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34748/consoleFull)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111347706
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111347663
[Test build #34746 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34746/console)
for PR 6775 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111340634
[Test build #34746 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34746/consoleFull)
for PR 6775 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111341026
[Test build #34741 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34741/console)
for PR 6775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111341032
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111332767
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111332779
Merged build started.
---
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 adrian-wang opened a pull request:
https://github.com/apache/spark/pull/6775
[SPARK-8240] [SPARK-8241] [SQL] string function: concat/concat_ws
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/adrian-wang/spark
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6775#issuecomment-111333511
[Test build #34741 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34741/consoleFull)
for PR 6775 at commit
88 matches
Mail list logo