Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/6712
---
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 liancheng commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110635367
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110635283
OK I'm going to merge this one first and send a small cleanup pr for the
comments here.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32094129
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -1307,16 +1307,9 @@
https://cwiki.apache.org/confluence/display/Hive/Enhanced+Agg
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32094155
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -460,7 +460,7 @@ class Analyzer(
def apply(plan: Log
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32094110
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -1307,16 +1307,9 @@
https://cwiki.apache.org/confluence/display/Hive/Enhance
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32094058
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala ---
@@ -1307,16 +1307,9 @@
https://cwiki.apache.org/confluence/display/Hive/Enhance
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32093933
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -460,7 +460,7 @@ class Analyzer(
def apply(plan: Log
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32093619
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/pythonUdfs.scala ---
@@ -71,43 +71,49 @@ private[spark] case class PythonUDF(
private[
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32093509
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -460,7 +460,7 @@ class Analyzer(
def apply(plan
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32092163
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/pythonUdfs.scala ---
@@ -71,43 +71,49 @@ private[spark] case class PythonUDF(
private[spark
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32092137
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/pythonUdfs.scala ---
@@ -71,43 +71,49 @@ private[spark] case class PythonUDF(
private[spark
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32092072
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/pythonUdfs.scala ---
@@ -71,43 +71,49 @@ private[spark] case class PythonUDF(
private[
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32089187
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/pythonUdfs.scala ---
@@ -71,43 +71,49 @@ private[spark] case class PythonUDF(
private[spark
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32089158
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -460,7 +460,7 @@ class Analyzer(
def apply(plan: Log
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110588254
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110588248
[Test build #34559 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34559/console)
for PR 6712 at commit
[`f4c2df0`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110570582
[Test build #34559 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34559/consoleFull)
for PR 6712 at commit
[`f4c2df0`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110570191
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/6712#issuecomment-110570203
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110569053
[Test build #34556 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34556/console)
for PR 6712 at commit
[`0bd4127`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110569055
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110568706
[Test build #34556 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34556/consoleFull)
for PR 6712 at commit
[`0bd4127`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110568131
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/6712#issuecomment-110568104
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6712#discussion_r32080678
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/hiveUdfs.scala
---
@@ -40,36 +41,43 @@ import org.apache.spark.sql.catalyst.rules.Rule
import or
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110545265
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110545259
[Test build #34542 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34542/console)
for PR 6712 at commit
[`f9a0378`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110531154
[Test build #34542 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34542/consoleFull)
for PR 6712 at commit
[`f9a0378`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110530785
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/6712#issuecomment-110530773
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110530308
[Test build #34535 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34535/console)
for PR 6712 at commit
[`5609494`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110530318
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110523084
[Test build #34535 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34535/consoleFull)
for PR 6712 at commit
[`5609494`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110522418
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/6712#issuecomment-110522428
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/6712#issuecomment-110520800
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 n
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110520784
[Test build #34532 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34532/console)
for PR 6712 at commit
[`8326ead`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110512101
[Test build #34530 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34530/console)
for PR 6712 at commit
[`04e1217`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110512133
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 n
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110498810
[Test build #34532 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34532/consoleFull)
for PR 6712 at commit
[`8326ead`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110498641
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 thi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110498672
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 f
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110486732
[Test build #34530 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34530/consoleFull)
for PR 6712 at commit
[`04e1217`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110486140
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 f
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110486100
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 thi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110263102
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110263098
[Test build #34488 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34488/console)
for PR 6712 at commit
[`d554d60`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110241035
[Test build #34487 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34487/console)
for PR 6712 at commit
[`9f3b75a`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110241043
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110239326
[Test build #34488 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34488/consoleFull)
for PR 6712 at commit
[`d554d60`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110239056
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/6712#issuecomment-110239050
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/6712#issuecomment-110238898
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110238873
[Test build #34485 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34485/console)
for PR 6712 at commit
[`dea550b`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110228780
[Test build #34487 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34487/consoleFull)
for PR 6712 at commit
[`9f3b75a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110228708
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/6712#issuecomment-110228716
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110228207
[Test build #34485 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34485/consoleFull)
for PR 6712 at commit
[`dea550b`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6712#issuecomment-110227993
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/6712#issuecomment-110227979
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 rxin opened a pull request:
https://github.com/apache/spark/pull/6712
[SPARK-7886] Use FunctionRegistry for built-in expressions in HiveContext.
This builds on https://github.com/apache/spark/pull/6710
You can merge this pull request into a Git repository by runnin
62 matches
Mail list logo