Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22614
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97014/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20761
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97025/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20761
**[Test build #97025 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97025/testReport)**
for PR 20761 at commit
[`707eb18`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22614
**[Test build #97014 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97014/testReport)**
for PR 22614 at commit
[`f42bbec`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22498
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22498
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97013/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22498
**[Test build #97013 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97013/testReport)**
for PR 22498 at commit
[`70472a2`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20761
**[Test build #97025 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97025/testReport)**
for PR 20761 at commit
[`707eb18`](https://github.com/apache/spark/commit/70
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22647
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22647
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3728/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22647
**[Test build #97024 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97024/testReport)**
for PR 22647 at commit
[`adb63e4`](https://github.com/apache/spark/commit/ad
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22647
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22060
**[Test build #97023 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97023/testReport)**
for PR 22060 at commit
[`7fc1d11`](https://github.com/apache/spark/commit/7f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22060
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3727/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22060
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22632
@gatorsmile I have some confusion. Is this
https://github.com/apache/spark/blob/58c55cb4a6d72d72df908e37aa63f617b3cc5587/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/Date
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22647
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97011/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22647
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22647
**[Test build #97011 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97011/testReport)**
for PR 22647 at commit
[`adb63e4`](https://github.com/apache/spark/commit/a
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22636
The time reduction is not obvious. Let us keep this unchanged?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.
Github user huaxingao commented on a diff in the pull request:
https://github.com/apache/spark/pull/22295#discussion_r223165392
--- Diff: python/pyspark/sql/session.py ---
@@ -252,6 +255,20 @@ def newSession(self):
"""
return self.__class__(self._sc, self._
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22638
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22047#discussion_r223164281
--- Diff: python/pyspark/sql/functions.py ---
@@ -403,6 +403,28 @@ def countDistinct(col, *cols):
return Column(jc)
+def every(co
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20832
@efimpoberezkin Could you please close this PR?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additio
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20832#discussion_r223164144
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Column.scala ---
@@ -1208,85 +1208,172 @@ class ColumnName(name: String) extends
Column(name) {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12922
**[Test build #97022 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97022/consoleFull)**
for PR 12922 at commit
[`3f91492`](https://github.com/apache/spark/commit/3
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22622
Thank you, @gatorsmile, @HyukjinKwon , @viirya , @dilipbiswal !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22237
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97016/
Test FAILed.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22642
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22237
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22237
**[Test build #97016 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97016/testReport)**
for PR 22237 at commit
[`69bbfef`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21809
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97008/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21809
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21809
**[Test build #97008 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97008/testReport)**
for PR 21809 at commit
[`0be099a`](https://github.com/apache/spark/commit/0
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22642
LGTM
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additiona
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22642#discussion_r223163190
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSparkSubmitSuite.scala ---
@@ -33,11 +33,13 @@ import org.apache.spark.sql.execution.comm
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97021 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97021/testReport)**
for PR 22623 at commit
[`a82e75f`](https://github.com/apache/spark/commit/a8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22503
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21688
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21688
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97007/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21688
**[Test build #97007 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97007/testReport)**
for PR 21688 at commit
[`a4ed19d`](https://github.com/apache/spark/commit/a
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22622
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22648
**[Test build #97020 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97020/testReport)**
for PR 22648 at commit
[`09b70cb`](https://github.com/apache/spark/commit/09
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22648
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22648
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3726/
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22622
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/22648
[MINOR] Clean up the joinCriteria in SQL parser
## What changes were proposed in this pull request?
Clean up the joinCriteria parsing in the parser by directly using
identifierList
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20761
**[Test build #97019 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97019/testReport)**
for PR 20761 at commit
[`a1c8fba`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20761
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97019/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20761
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20761
**[Test build #97019 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97019/testReport)**
for PR 20761 at commit
[`a1c8fba`](https://github.com/apache/spark/commit/a1
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r223159523
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorSuite.scala
---
@@ -134,6 +166,42 @@ class YarnAllocatorSuite exte
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22622
@gatorsmile . Could you review this again? For your comment, I added
[SPARK-25656 Add an example section about how to use Parquet/ORC library
options](https://issues.apache.org/jira/browse/SPA
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22622#discussion_r223158108
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/orc/OrcSourceSuite.scala
---
@@ -115,6 +116,69 @@ abstract class OrcSuit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22612
**[Test build #97018 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97018/testReport)**
for PR 22612 at commit
[`a11e3a2`](https://github.com/apache/spark/commit/a1
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22647
nit. Could you update the title to (1) or (2), @gatorsmile ?
```
1. [SPARK-25655] [BUILD] Add -Pspark-ganglia-lgpl to the scala style check
2. [SPARK-25655] [BUILD] Add profile 'spar
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22060
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97009/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22060
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22060
**[Test build #97009 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97009/testReport)**
for PR 22060 at commit
[`d595a0c`](https://github.com/apache/spark/commit/d
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22643#discussion_r223155383
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/sources/HadoopFsRelationTest.scala
---
@@ -760,23 +775,27 @@ abstract class HadoopFsRelationTe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22379
**[Test build #97017 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97017/testReport)**
for PR 22379 at commit
[`b318239`](https://github.com/apache/spark/commit/b3
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22379#discussion_r223152506
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CsvExpressionsSuite.scala
---
@@ -0,0 +1,160 @@
+/*
+ * Licensed to
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22642#discussion_r223150948
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSparkSubmitSuite.scala ---
@@ -33,11 +33,13 @@ import org.apache.spark.sql.execution.command.
Github user rezasafi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22612#discussion_r223150611
--- Diff:
core/src/main/scala/org/apache/spark/executor/ProcfsBasedSystems.scala ---
@@ -0,0 +1,274 @@
+/*
+ * Licensed to the Apache Software Foun
Github user rezasafi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22612#discussion_r223150568
--- Diff:
core/src/main/scala/org/apache/spark/executor/ProcfsBasedSystems.scala ---
@@ -0,0 +1,266 @@
+/*
+ * Licensed to the Apache Software Foun
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22612
**[Test build #97015 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97015/testReport)**
for PR 22612 at commit
[`a9f924c`](https://github.com/apache/spark/commit/a9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22237
**[Test build #97016 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97016/testReport)**
for PR 22237 at commit
[`69bbfef`](https://github.com/apache/spark/commit/69
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r223150095
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/TestYarnResourceRequestHelper.scala
---
@@ -0,0 +1,92 @@
+/*
+ * Licens
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22614
**[Test build #97014 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97014/testReport)**
for PR 22614 at commit
[`f42bbec`](https://github.com/apache/spark/commit/f4
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22642#discussion_r223148975
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSparkSubmitSuite.scala ---
@@ -33,11 +33,13 @@ import org.apache.spark.sql.execution.comm
Github user kmanamcheri commented on a diff in the pull request:
https://github.com/apache/spark/pull/22614#discussion_r223148534
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/HiveClientSuite.scala
---
@@ -87,6 +90,18 @@ class HiveClientSuite(version: String)
Github user kmanamcheri commented on a diff in the pull request:
https://github.com/apache/spark/pull/22614#discussion_r223148506
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -544,6 +544,15 @@ object SQLConf {
.booleanConf
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22640
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97004/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22640
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22640
**[Test build #97004 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97004/testReport)**
for PR 22640 at commit
[`1c5bfef`](https://github.com/apache/spark/commit/1
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r223147289
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ResourceRequestValidatorSuite.scala
---
@@ -0,0 +1,132 @@
+/*
+ * Licen
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22643#discussion_r223146731
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/sources/HadoopFsRelationTest.scala
---
@@ -760,23 +775,27 @@ abstract class HadoopFsRelationTest
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22644
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22644
Thank you very much @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22237#discussion_r223144685
--- Diff: docs/sql-programming-guide.md ---
@@ -1879,6 +1879,10 @@ working with timestamps in `pandas_udf`s to get the
best performance, see
# Mi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20761
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20761
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97012/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20761
**[Test build #97012 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97012/testReport)**
for PR 20761 at commit
[`76f7e62`](https://github.com/apache/spark/commit/7
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22532
Thanks for submitting the PR! I quickly scan the code changes. It sounds
like the general direction is right but the quality is not ready.
I would suggest to write the test plan before d
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r223142574
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/TestYarnResourceTypeHelper.scala
---
@@ -0,0 +1,93 @@
+/*
+ * Licensed
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r223142490
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ResourceTypeValidatorSuite.scala
---
@@ -0,0 +1,150 @@
+/*
+ * Licensed
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97003/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97003 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97003/testReport)**
for PR 22623 at commit
[`81e1730`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22498
**[Test build #97013 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97013/testReport)**
for PR 22498 at commit
[`70472a2`](https://github.com/apache/spark/commit/70
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22614#discussion_r223139984
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -544,6 +544,15 @@ object SQLConf {
.booleanConf
Github user kmanamcheri commented on a diff in the pull request:
https://github.com/apache/spark/pull/22614#discussion_r223139874
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/HiveClientSuite.scala
---
@@ -87,6 +90,18 @@ class HiveClientSuite(version: String)
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r223139838
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ResourceRequestTestHelper.scala
---
@@ -0,0 +1,95 @@
+/*
+ * Licensed t
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r223139747
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnAllocatorSuite.scala
---
@@ -63,6 +63,10 @@ class YarnAllocatorSuite extend
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r223139279
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,128 @@
+/*
+ * Licensed to the
Github user szyszy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r223139202
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ClientSuite.scala
---
@@ -199,6 +200,92 @@ class ClientSuite extends SparkFunSu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20761
**[Test build #97012 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97012/testReport)**
for PR 20761 at commit
[`76f7e62`](https://github.com/apache/spark/commit/76
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22614#discussion_r223137994
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/client/HiveClientSuite.scala
---
@@ -87,6 +90,18 @@ class HiveClientSuite(version: String)
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/22647
LGTM pending tests
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22644
cc @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
101 - 200 of 561 matches
Mail list logo