Github user mallman commented on the issue:
https://github.com/apache/spark/pull/22880
Can someone with Jenkins retest privileges please kick off a retest?
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98558/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22944
**[Test build #98558 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98558/testReport)**
for PR 22944 at commit
Github user mallman commented on the issue:
https://github.com/apache/spark/pull/22880
Jenkins retest please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22951
**[Test build #98567 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98567/testReport)**
for PR 22951 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22960
**[Test build #98566 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98566/testReport)**
for PR 22960 at commit
Github user vincentye38 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8355#discussion_r231648528
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2.scala
---
@@ -67,6 +68,46 @@ object
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22951#discussion_r231646713
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CsvExpressionsSuite.scala
---
@@ -209,4 +210,20 @@ class
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22951#discussion_r231641960
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/JsonFunctionsSuite.scala ---
@@ -578,4 +581,20 @@ class JsonFunctionsSuite extends QueryTest
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22951#discussion_r231640870
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/CsvFunctionsSuite.scala ---
@@ -117,4 +120,20 @@ class CsvFunctionsSuite extends QueryTest
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22938
**[Test build #98565 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98565/testReport)**
for PR 22938 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/22957
i didn't look at your new code, but is your old code safe? e.g. a project
that depends on the new alias.
---
-
To unsubscribe,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98557/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98556/
Test FAILed.
---
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22952#discussion_r231634109
--- Diff: docs/structured-streaming-programming-guide.md ---
@@ -530,6 +530,8 @@ Here are the details of all the sources in Spark.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22944
**[Test build #98557 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98557/testReport)**
for PR 22944 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22944
**[Test build #98556 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98556/testReport)**
for PR 22944 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22938
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98555/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22938
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22938
**[Test build #98555 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98555/testReport)**
for PR 22938 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22932
@felixcheung . If the question is about writer versions, Spark/Hive works
on top of ORC/Parquet library. ORC/Parquet library already writes its specific
version for that purpose. For me, it
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22965
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22965
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98553/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22965
**[Test build #98553 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98553/testReport)**
for PR 22965 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22504
**[Test build #98564 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98564/testReport)**
for PR 22504 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22957
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98552/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22957
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22957
**[Test build #98552 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98552/testReport)**
for PR 22957 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22921
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22921
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98551/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22921
**[Test build #98551 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98551/testReport)**
for PR 22921 at commit
Github user rvesse commented on the issue:
https://github.com/apache/spark/pull/22959
First glance this looks like a lot of nice simplification, will take a
proper look over this tomorrow
---
-
To unsubscribe,
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22965#discussion_r231610295
--- Diff: sql/core/benchmarks/DataSourceReadBenchmark-results.txt ---
@@ -2,268 +2,268 @@
SQL Single Numeric Column Scan
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22965#discussion_r231608634
--- Diff: sql/core/benchmarks/DataSourceReadBenchmark-results.txt ---
@@ -2,268 +2,268 @@
SQL Single Numeric Column Scan
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22932
Thank you for review, @felixcheung . Could you elaborate a little bit more?
Here, three writers are used: new native ORC writer, old Hive ORC writer, and
native Parquet writer.
> a prop
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21688
**[Test build #98563 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98563/testReport)**
for PR 21688 at commit
Github user pgandhi999 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21688#discussion_r231605935
--- Diff: core/src/main/resources/org/apache/spark/ui/static/stagepage.js
---
@@ -0,0 +1,965 @@
+/*
+ * Licensed to the Apache Software
Github user pgandhi999 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21688#discussion_r231605818
--- Diff: core/src/main/resources/org/apache/spark/ui/static/stagepage.js
---
@@ -0,0 +1,965 @@
+/*
+ * Licensed to the Apache Software
Github user pgandhi999 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21688#discussion_r231605894
--- Diff: core/src/main/resources/org/apache/spark/ui/static/stagepage.js
---
@@ -0,0 +1,965 @@
+/*
+ * Licensed to the Apache Software
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22921
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22921
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22504
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98562/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22947
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22947
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22504
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22504
**[Test build #98562 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98562/testReport)**
for PR 22504 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22909
Kubernetes integration test status failure
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/4821/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22909
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22909
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22947#discussion_r231600766
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/misc.scala
---
@@ -66,6 +66,8 @@ case class AssertTrue(child: Expression)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22921
**[Test build #98561 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98561/testReport)**
for PR 22921 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22909
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/4821/
---
Github user uzadude commented on the issue:
https://github.com/apache/spark/pull/22964
this is the original query. we can see the explode followed by the shuffle:
```
import org.apache.spark.sql.functions._
import org.apache.spark.sql.expressions._
val N = 1
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/22948#discussion_r231598045
--- Diff: dev/appveyor-install-dependencies.ps1 ---
@@ -115,7 +115,7 @@ $env:Path += ";$env:HADOOP_HOME\bin"
Pop-Location
#
Github user mpmolek commented on the issue:
https://github.com/apache/spark/pull/22937
@felixcheung SPARK_HOME has been getting filtered out since #10329
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22947
**[Test build #98560 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98560/testReport)**
for PR 22947 at commit
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/22921#discussion_r231596680
--- Diff: R/pkg/R/functions.R ---
@@ -1663,9 +1692,24 @@ setMethod("toDegrees",
#' @aliases toRadians toRadians,Column-method
#' @note
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22909
**[Test build #98559 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98559/testReport)**
for PR 22909 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22909
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r231591033
--- Diff:
core/src/main/scala/org/apache/spark/deploy/security/KafkaDelegationTokenProvider.scala
---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22944
**[Test build #98558 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98558/testReport)**
for PR 22944 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22944
@cloud-fan I have a simpler fix for this issue. Can you take another look?
Thanks.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22944
**[Test build #98557 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98557/testReport)**
for PR 22944 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22944
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22944
**[Test build #98556 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98556/testReport)**
for PR 22944 at commit
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/22938
I made a fix for broken array and map in `JsonToStructs` because inside of
`FailureSafeParser` is not clear from where the call came. I am still not sure
that wrapping actual type by `StructType`
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22087
I'd escalate to dev@ for more visibility. It has some longer-term
consequences, and I'd like to hear current thinking on how much these APIs
should change, etc.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22963
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22938
**[Test build #98555 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98555/testReport)**
for PR 22938 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22963
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98549/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22963
**[Test build #98549 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98549/testReport)**
for PR 22963 at commit
Github user patrickbrownsync commented on a diff in the pull request:
https://github.com/apache/spark/pull/22855#discussion_r231558355
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -214,8 +230,14 @@ class KryoSerializer(conf: SparkConf)
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22087
@srowen I think it would not be a big deal if we would change the return
type of these to Double, since they are now private. If that is the only change
needed we may be able to reuse the
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22893#discussion_r23159
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -521,19 +521,21 @@ object MLUtils extends Logging {
* The bound
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22818
since this is a bug fix, shall we also backport it?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user patrickbrownsync commented on a diff in the pull request:
https://github.com/apache/spark/pull/22855#discussion_r231554649
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -92,6 +94,20 @@ class KryoSerializer(conf: SparkConf)
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22921#discussion_r231554470
--- Diff: R/pkg/R/functions.R ---
@@ -319,6 +319,27 @@ setMethod("acos",
column(jc)
})
+#' @details
+#'
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22966
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98554/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22966
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22966
**[Test build #98554 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98554/testReport)**
for PR 22966 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22087
Aha. On the one hand, I suppose they can't extend that class because of the
signature difference, and indeed it says it's the superclass of regression and
classification models. I can imagine that
Github user uzadude commented on the issue:
https://github.com/apache/spark/pull/22964
The whole idea is that we'll get one shuffle and it will be before the
explode as the window's partition is contained in the repartition.
I'll show the physical plan.
---
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/22965
@dongjoon-hyun @yucai
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/22966
@dongjoon-hyun
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/22965#discussion_r231549251
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/orc/OrcReadBenchmark.scala ---
@@ -32,9 +32,11 @@ import org.apache.spark.sql.types._
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22966
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22966
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22087
@erikerlandson @srowen actually we already have a `PredictionModel` for
this. am not sure why clustering algorithms are not extending it though, but
in that class the method returns a `Double`,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22966
**[Test build #98554 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98554/testReport)**
for PR 22966 at commit
GitHub user gengliangwang opened a pull request:
https://github.com/apache/spark/pull/22966
[PARK-25965][SQL] Add avro read benchmark
## What changes were proposed in this pull request?
Add read benchmark for Avro, which is missing for a period.
The benchmark is similar
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/22964
@uzadude where is this relevant? You will end up with two shuffles if you
do this.
---
-
To unsubscribe, e-mail:
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22955
shall we also remove the end-to-end tests which are now not needed anymore?
---
-
To unsubscribe, e-mail:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22894
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22087
Yeah it's a good point and I wonder if @jkbradley or @mengxr or @MLnick
want to weigh in. If this superclass method existed, I think it would be
`predict(Vector):Int` anyway, so seems pretty
201 - 300 of 400 matches
Mail list logo