Github user chutium commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-49159677
PR submitted #1442
---
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 fe
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-49155240
Awesome! Please submit a pull request with that addition.
On Jul 16, 2014 7:53 AM, "Teng Qiu" wrote:
> hi, it is really very useful for us, i tried this impl
Github user chutium commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-49154812
hi, it is really very useful for us, i tried this implementation from
@willb , in spark-shell, i still got java.lang.UnsupportedOperationException by
Query Plan, i made s
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/1359
---
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 enab
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-49093280
Thanks! Merged into master and 1.0.
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-49093085
I'm going to go ahead and merge this. Someone can make a follow-up with
nullability narrowing.
---
If your project is set up for it, you can reply to this email and ha
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48991417
QA results for PR 1359:- This patch PASSES unit tests.- This patch
merges cleanly- This patch adds the following public classes
(experimental):case class Substring(str: E
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48987029
QA tests have started for PR 1359. This patch merges cleanly. View
progress:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16652/consoleFull
---
If
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48986821
add to whitelist
---
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 featu
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14915562
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -205,3 +207,72 @@ case class EndsWith(left: Expr
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48971609
Thanks! Will merge as soon as Jenkins comes back happy.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user willb commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48971559
Thanks, Michael. I've fixed the conflict.
---
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 d
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48969318
Hey Will, this looks great. Thanks for implementing! Mind fixing the
conflict with master so I can merge?
---
If your project is set up for it, you can reply to this
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48690227
QA results for PR 1359:- This patch PASSES unit tests.- This patch
merges cleanly- This patch adds the following public classes
(experimental):case class Substring(str: E
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48685587
QA tests have started for PR 1359. This patch merges cleanly. View
progress:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16544/consoleFull
---
If
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48685347
QA results for PR 1359:- This patch FAILED unit tests.- This patch
merges cleanly- This patch adds the following public classes
(experimental):case class Substring(str: E
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48685301
QA tests have started for PR 1359. This patch merges cleanly. View
progress:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16543/consoleFull
---
If
Github user willb commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14802394
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -207,3 +207,71 @@ case class StartsWith(left: Exp
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14801757
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -207,3 +207,71 @@ case class StartsWith(left:
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48665810
Merged build finished. All automated tests passed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48665803
QA results for PR 1359:- This patch PASSES unit tests.- This patch
merges cleanly- This patch adds the following public classes
(experimental):case class Substring(str: E
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48665812
All automated tests passed.
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16513/
---
If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48654045
QA tests have started for PR 1359. This patch merges cleanly. View
progress:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16513/consoleFull
---
If
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48653881
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48653871
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 h
Github user willb commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14787712
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -207,3 +210,64 @@ case class StartsWith(left: Exp
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48651897
All automated tests passed.
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16506/
---
If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48651882
QA results for PR 1359:- This patch PASSES unit tests.- This patch
merges cleanly- This patch adds the following public classes
(experimental):case class Substring(str: E
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48651895
Merged build finished. All automated tests passed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user concretevitamin commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48648864
This sounds pretty cool!
---
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 willb commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48648670
@concretevitamin Inlining `Substring.slice` seems to make a nontrivial
difference (~11.5% speedup) on a very simple `Substring.eval()` microbenchmark.
---
If your project
Github user concretevitamin commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14783724
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -207,3 +210,64 @@ case class StartsWith
Github user willb commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14783626
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -207,3 +210,64 @@ case class StartsWith(left: Exp
Github user willb commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14783468
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -207,3 +210,64 @@ case class StartsWith(left: Exp
Github user willb commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48643276
Thanks for the comments, @concretevitamin. I've made the changes from your
comments and will think about reducing branch overhead before pushing an update.
---
If your pr
Github user concretevitamin commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48642397
Hey @willb - thanks for working on this, which is going to be very useful
for Spark SQL. I left a couple minor comments. Another general concern is the
performanc
Github user concretevitamin commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14782682
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -207,3 +210,64 @@ case class StartsWith
Github user concretevitamin commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14782583
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -207,3 +210,64 @@ case class StartsWith
Github user willb commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14782431
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -860,6 +860,7 @@ private[hive] object HiveQl {
val BETWEEN = "(?i)BETWEEN"
Github user concretevitamin commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14782267
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -860,6 +860,7 @@ private[hive] object HiveQl {
val BETWEEN = "(?
Github user willb commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14782197
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -207,3 +210,64 @@ case class StartsWith(left: Exp
Github user concretevitamin commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14781542
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -207,3 +210,64 @@ case class StartsWith
Github user concretevitamin commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14781690
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -207,3 +210,64 @@ case class StartsWith
Github user concretevitamin commented on a diff in the pull request:
https://github.com/apache/spark/pull/1359#discussion_r14781579
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala
---
@@ -19,8 +19,11 @@ package org.apache.spar
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48639097
QA tests have started for PR 1359. This patch merges cleanly. View
progress:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16506/consoleFull
---
If
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48638963
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1359#issuecomment-48638949
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 have
GitHub user willb opened a pull request:
https://github.com/apache/spark/pull/1359
SPARK-2407: Added internal implementation of SQL SUBSTR()
This replaces the Hive UDF for SUBSTR(ING) with an implementation in
Catalyst
and adds tests to verify correct operation.
You can merge t
48 matches
Mail list logo