Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9215#issuecomment-150392433
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/9215#issuecomment-150392446
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 mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821931
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeansModel.scala
---
@@ -0,0 +1,137 @@
+/*
+ * Licensed to the Apache Softw
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821747
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821750
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821656
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821607
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821578
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821027
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42820990
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821021
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42820995
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821004
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821017
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42820987
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821008
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821012
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42820998
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42820997
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42820980
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821019
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/5267#discussion_r42821001
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -0,0 +1,690 @@
+/*
+ * Licensed to the Apache Software Fo
Github user vectorijk commented on the pull request:
https://github.com/apache/spark/pull/9233#issuecomment-150390355
cc @mengxr
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-150389761
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/9003#issuecomment-150389712
**[Test build #44179 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44179/consoleFull)**
for PR 9003 at commit
[`3ef2faa`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9003#issuecomment-150389765
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9236#issuecomment-150389357
Can one of the admins verify this patch?
---
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 pr
GitHub user manygrams opened a pull request:
https://github.com/apache/spark/pull/9236
[SPARK-11270] [STREAMING] Add improved equality testing for
TopicAndPartition from the Kafka Streaming API
@jerryshao @tdas
I know this is kind of minor, and I know you all are busy, but
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42820025
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -930,3 +930,327 @@ object HyperLogLogPlusPlus
Github user gliptak commented on the pull request:
https://github.com/apache/spark/pull/8323#issuecomment-150386525
As per https://github.com/xmlrunner/unittest-xml-reporting/issues/95 the
Python tests do not have packages (or possibly not being recognized by
https://github.com/xmlrun
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9127#issuecomment-150383049
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9127#issuecomment-150383050
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7910#issuecomment-150382998
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/9127#issuecomment-150382994
**[Test build #44174 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44174/consoleFull)**
for PR 9127 at commit
[`bba5550`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7910#issuecomment-150382999
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7910#issuecomment-150382915
**[Test build #44175 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44175/consoleFull)**
for PR 7910 at commit
[`e88f70d`](https://git
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/9222#issuecomment-150382796
Do you have benchmark numbers for this change?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If you
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/9234#issuecomment-150382185
Hi thanks for the contribution, you might want to check out the ongoing
work in https://github.com/apache/spark/pull/9099
and SPARK-11086
---
If your project is
Github user olarayej commented on the pull request:
https://github.com/apache/spark/pull/8984#issuecomment-150381685
@sun-rui Sure! Thank you!
---
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/9127#issuecomment-150380317
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9127#issuecomment-150380315
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/9127#issuecomment-150380260
**[Test build #44173 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44173/consoleFull)**
for PR 9127 at commit
[`d86f435`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150379910
**[Test build #44185 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44185/consoleFull)**
for PR 9210 at commit
[`da4d824`](https://gith
Github user tedyu commented on a diff in the pull request:
https://github.com/apache/spark/pull/8909#discussion_r42817261
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala ---
@@ -1196,46 +1174,90 @@ class SQLContext(@transient val sparkContext:
SparkContext)
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9235#issuecomment-150379127
**[Test build #44184 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44184/consoleFull)**
for PR 9235 at commit
[`b265ec2`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8690#issuecomment-150379119
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8690#issuecomment-150379121
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9209
---
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 a diff in the pull request:
https://github.com/apache/spark/pull/8909#discussion_r42817021
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala ---
@@ -1196,46 +1174,90 @@ class SQLContext(@transient val sparkContext:
SparkContex
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8690#issuecomment-150379000
**[Test build #44183 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44183/consoleFull)**
for PR 8690 at commit
[`eef61c6`](https://git
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/9209#issuecomment-150378938
I've merged this.
---
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 featur
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150378799
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8909#discussion_r42816917
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala ---
@@ -1196,46 +1174,90 @@ class SQLContext(@transient val sparkContext:
SparkContext)
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150378777
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 tedyu commented on a diff in the pull request:
https://github.com/apache/spark/pull/8909#discussion_r42816849
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala ---
@@ -1196,46 +1174,90 @@ class SQLContext(@transient val sparkContext:
SparkContext)
Github user tedyu commented on a diff in the pull request:
https://github.com/apache/spark/pull/8909#discussion_r42816551
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala ---
@@ -1196,46 +1174,90 @@ class SQLContext(@transient val sparkContext:
SparkContext)
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9235#issuecomment-150378078
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/9235#issuecomment-150378064
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 sun-rui commented on the pull request:
https://github.com/apache/spark/pull/9222#issuecomment-150377869
dfToCols is meant to be called from R side, it makes sense to test it in R.
Having a test case in R for it can test not only the logic in Scala, but also
helps to test
GitHub user vanzin opened a pull request:
https://github.com/apache/spark/pull/9235
[SPARK-11134] [core] Increase LauncherBackendSuite timeout.
This test can take a little while to finish on slow / loaded machines.
You can merge this pull request into a Git repository by running:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9230#issuecomment-150377481
**[Test build #1942 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1942/consoleFull)**
for PR 9230 at commit
[`23ba732`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9209#issuecomment-150376923
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9209#issuecomment-150376920
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/9209#issuecomment-150376800
**[Test build #44172 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44172/consoleFull)**
for PR 9209 at commit
[`d0fc050`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8690#issuecomment-150376605
**[Test build #44183 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44183/consoleFull)**
for PR 8690 at commit
[`eef61c6`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8690#issuecomment-150376436
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/8690#issuecomment-150376443
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 yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/8690#issuecomment-150375830
@noel-smith thank you for the review. I added them again. Could you review
it?
---
If your project is set up for it, you can reply to this email and have your
reply app
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9234#issuecomment-150374723
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9234#issuecomment-150374719
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/8323
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9234#issuecomment-150373617
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9234#issuecomment-150373621
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/8323#issuecomment-150373253
LGTM, merging this 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 do
Github user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/8643#issuecomment-150372929
@jkbradley sorry for the delay of my update. I tried to use DataFrame
serialization at
https://github.com/yu-iskw/spark/commit/2f701930677c7d359c0a706d16c2438509528b77.
Github user NarineK commented on a diff in the pull request:
https://github.com/apache/spark/pull/9012#discussion_r42814122
--- Diff: R/pkg/R/DataFrame.R ---
@@ -1457,15 +1457,142 @@ setMethod("join",
dataFrame(sdf)
})
-#' @rdname merge
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9190
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8643#issuecomment-150372009
**[Test build #44182 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44182/consoleFull)**
for PR 8643 at commit
[`2f70193`](https://gith
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/9190#issuecomment-150371978
I'm going to merge this. We should continue to do review post-hoc.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub a
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9127#issuecomment-150371690
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9127#issuecomment-150371691
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9127#issuecomment-150371612
**[Test build #44171 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44171/consoleFull)**
for PR 9127 at commit
[`46ad693`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8643#issuecomment-150371481
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/8643#issuecomment-150371516
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/9234#issuecomment-150371477
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/9234#issuecomment-150371495
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/9234#issuecomment-150370515
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/9234#issuecomment-150370532
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 kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/9220#discussion_r42813223
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -266,6 +266,11 @@ object SparkSubmit {
}
}
+
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42813128
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -930,3 +930,327 @@ object HyperLogLogPlusPlus
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42813053
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -930,3 +930,327 @@ object HyperLogLogPlusPlus
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42813076
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -857,3 +857,329 @@ object HyperLogLogPlusPlus
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42813023
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -930,3 +930,327 @@ object HyperLogLogPlusPlus
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42813001
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -930,3 +930,327 @@ object HyperLogLogPlusPlus
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42812941
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -930,3 +930,327 @@ object HyperLogLogPlusPlus
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42812969
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -930,3 +930,327 @@ object HyperLogLogPlusPlus
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42812933
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -930,3 +930,327 @@ object HyperLogLogPlusPlus
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42812911
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -930,3 +930,327 @@ object HyperLogLogPlusPlus
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42812923
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -930,3 +930,327 @@ object HyperLogLogPlusPlus
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9003#discussion_r42812895
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala
---
@@ -930,3 +930,327 @@ object HyperLogLogPlusPlus
201 - 300 of 763 matches
Mail list logo