Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/17059
@datumbox you mention there is GC & performance overhead which makes some
sense. Have you run into problems with very large scale (like millions users &
items & ratings)? I did regression tests
[her
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17064#discussion_r103393968
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala ---
@@ -168,15 +168,16 @@ class CacheManager extends Logging {
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17015
LGTM except a few comments.
---
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
Github user datumbox commented on the issue:
https://github.com/apache/spark/pull/17059
@imatiach-msft This comparison is intentional and checks 2 things: That the
number is within integer range and that the Id does not have any non-zero
digits after the decimal point. If the number i
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17015#discussion_r103393810
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/MetastoreRelation.scala ---
@@ -1,179 +0,0 @@
-/*
- * Licensed to the Apache Software
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16867
**[Test build #73575 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73575/testReport)**
for PR 16867 at commit
[`9778b67`](https://github.com/apache/spark/commit/97
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17093
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73568/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17093
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17093
**[Test build #73568 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73568/testReport)**
for PR 17093 at commit
[`a3ac29b`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17015
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73566/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17015
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17015
**[Test build #73566 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73566/testReport)**
for PR 17015 at commit
[`d10bfbc`](https://github.com/apache/spark/commit/d
Github user datumbox commented on a diff in the pull request:
https://github.com/apache/spark/pull/17059#discussion_r103392500
--- Diff: mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala
---
@@ -82,12 +82,20 @@ private[recommendation] trait ALSModelParams extends
P
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17015#discussion_r103391249
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveTableScanExec.scala
---
@@ -138,27 +153,35 @@ case class HiveTableScanExec(
Github user jinxing64 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16867#discussion_r103391138
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala ---
@@ -911,14 +916,14 @@ private[spark] class TaskSetManager(
logD
Github user kayousterhout commented on a diff in the pull request:
https://github.com/apache/spark/pull/16867#discussion_r103390199
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala ---
@@ -911,14 +916,14 @@ private[spark] class TaskSetManager(
Github user hhbyyh commented on the issue:
https://github.com/apache/spark/pull/17090
the same as https://github.com/apache/spark/pull/12574 ?
---
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17064#discussion_r103390013
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala ---
@@ -168,15 +168,16 @@ class CacheManager extends Logging {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16867
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73563/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16867
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16867
**[Test build #73563 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73563/testReport)**
for PR 16867 at commit
[`cd16008`](https://github.com/apache/spark/commit/c
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/13036
I am happy to do so. I assume that It seems already almost done except for
https://github.com/apache/spark/pull/13036#discussion_r84476560?
---
If your project is set up for it, you can reply t
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16867
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 does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16867
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73562/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16867
**[Test build #73562 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73562/testReport)**
for PR 16867 at commit
[`f28d900`](https://github.com/apache/spark/commit/f
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17015#discussion_r103387597
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeColumnCommand.scala
---
@@ -40,38 +38,24 @@ case class AnalyzeColumnComm
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17015#discussion_r103387529
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeColumnCommand.scala
---
@@ -90,10 +74,10 @@ object AnalyzeColumnCommand
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16929
**[Test build #73574 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73574/testReport)**
for PR 16929 at commit
[`0c088bf`](https://github.com/apache/spark/commit/0c
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16929
@brkyvz, @marmbrus - I think it is ready for another look. Could you see if
I understood your comments correctly?
---
If your project is set up for it, you can reply to this email and have your
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16929#discussion_r103386481
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/jsonExpressions.scala
---
@@ -480,36 +480,79 @@ case class JsonTuple(ch
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16929
**[Test build #73573 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73573/testReport)**
for PR 16929 at commit
[`9f1e966`](https://github.com/apache/spark/commit/9f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17095
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 does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17095
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73565/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17095
**[Test build #73565 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73565/testReport)**
for PR 17095 at commit
[`55c525e`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17089
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 does not have this feature
e
Github user uncleGen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17052#discussion_r103384682
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSourceSuite.scala
---
@@ -989,7 +989,8 @@ class FileStreamSourceSuite extends
Github user uncleGen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17052#discussion_r103385145
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -231,8 +231,9 @@ abstract class SparkStrategies extends
Quer
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16929
**[Test build #73572 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73572/testReport)**
for PR 16929 at commit
[`54e60bb`](https://github.com/apache/spark/commit/54
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17089
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73564/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17089
**[Test build #73564 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73564/testReport)**
for PR 17089 at commit
[`8bca8d3`](https://github.com/apache/spark/commit/8
Github user uncleGen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17052#discussion_r103385054
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1119,11 +1119,16 @@ case class DecimalAggregates(con
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/17056#discussion_r103384950
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/HashExpressionsSuite.scala
---
@@ -371,6 +370,48 @@ class HashExpressio
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/17056#discussion_r103384029
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/HashExpressionsSuite.scala
---
@@ -371,6 +370,48 @@ class HashExpressio
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17052
**[Test build #73571 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73571/testReport)**
for PR 17052 at commit
[`9ffbad2`](https://github.com/apache/spark/commit/9f
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/16954#discussion_r103384302
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1398,42 +1399,46 @@ class Analyzer(
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16929
**[Test build #73570 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73570/testReport)**
for PR 16929 at commit
[`72d6410`](https://github.com/apache/spark/commit/72
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/16938#discussion_r103384037
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala
---
@@ -140,8 +140,8 @@ case class CreateDataSource
Github user windpiger commented on the issue:
https://github.com/apache/spark/pull/17055
ping @gatorsmile
---
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,
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17015#discussion_r103383919
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveStrategies.scala ---
@@ -91,18 +97,58 @@ class ResolveHiveSerdeTable(session: SparkSessio
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10307
**[Test build #73569 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73569/testReport)**
for PR 10307 at commit
[`e425438`](https://github.com/apache/spark/commit/e4
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17015#discussion_r103383279
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -349,36 +350,41 @@ object CatalogTypes {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17079
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 does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17079
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73560/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17079
**[Test build #73560 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73560/testReport)**
for PR 17079 at commit
[`94879a8`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17052
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 does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17052
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73558/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17093
**[Test build #73568 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73568/testReport)**
for PR 17093 at commit
[`a3ac29b`](https://github.com/apache/spark/commit/a3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17052
**[Test build #73558 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73558/testReport)**
for PR 17052 at commit
[`c87651a`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10307
**[Test build #73567 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73567/testReport)**
for PR 10307 at commit
[`401f682`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10307
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73567/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10307
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10307
**[Test build #73567 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73567/testReport)**
for PR 10307 at commit
[`401f682`](https://github.com/apache/spark/commit/40
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17067
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 does not have this feature
e
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/17090
cc @MLnick
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17067
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73561/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17067
**[Test build #73561 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73561/testReport)**
for PR 17067 at commit
[`e4f347e`](https://github.com/apache/spark/commit/e
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/13036
Ok, lets see if maybe @zero323 or @HyukjinKwon are interested in taking
this over. Otherwise I'll add this to my backlog.
---
If your project is set up for it, you can reply to this email and have
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/13036
@holdenk please feel free to take this over. Can't find time to work on it
---
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 proj
Github user sethah closed the pull request at:
https://github.com/apache/spark/pull/13036
---
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 ena
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17047
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 does not have this feature
e
Github user skambha commented on a diff in the pull request:
https://github.com/apache/spark/pull/17067#discussion_r103379909
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ColumnResolutionSuite.scala
---
@@ -0,0 +1,173 @@
+/*
+ * Licensed to the Apa
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17015
**[Test build #73566 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73566/testReport)**
for PR 17015 at commit
[`d10bfbc`](https://github.com/apache/spark/commit/d1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17047
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73554/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17047
**[Test build #73554 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73554/testReport)**
for PR 17047 at commit
[`7753998`](https://github.com/apache/spark/commit/7
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17015
retest this please
---
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
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17067#discussion_r103378842
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/SQLViewSuite.scala ---
@@ -52,6 +52,19 @@ abstract class SQLViewSuite extends QueryTest
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17067#discussion_r103378807
--- Diff: sql/core/src/test/resources/sql-tests/inputs/columnresolution.sql
---
@@ -0,0 +1,82 @@
+-- Scenario: column resolution scenarios with datas
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17067#discussion_r103378740
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ColumnResolutionSuite.scala
---
@@ -0,0 +1,173 @@
+/*
+ * Licensed to the
Github user imatiach-msft commented on the issue:
https://github.com/apache/spark/pull/17059
@datumbox I like the changes, I just had a minor concern about the code
where we call v.intValue and then compare this to v.doubleValue -- due to
precision issues, I'm not sure if this is desi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17052
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 does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17052
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73559/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17052
**[Test build #73559 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73559/testReport)**
for PR 17052 at commit
[`59f4272`](https://github.com/apache/spark/commit/5
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/17089#discussion_r103378494
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InsertIntoHadoopFsRelationCommand.scala
---
@@ -148,6 +148,9 @@ case class I
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17095
**[Test build #73565 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73565/testReport)**
for PR 17095 at commit
[`55c525e`](https://github.com/apache/spark/commit/55
Github user skambha commented on a diff in the pull request:
https://github.com/apache/spark/pull/17067#discussion_r103378392
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/SQLViewSuite.scala ---
@@ -52,6 +52,19 @@ abstract class SQLViewSuite extends QueryTest wit
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/17059#discussion_r103378267
--- Diff: mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala
---
@@ -82,12 +82,20 @@ private[recommendation] trait ALSModelParams exten
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17067#discussion_r103378199
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/SQLViewSuite.scala ---
@@ -52,6 +52,19 @@ abstract class SQLViewSuite extends QueryTest
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17089
**[Test build #73564 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73564/testReport)**
for PR 17089 at commit
[`8bca8d3`](https://github.com/apache/spark/commit/8b
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17089
retest this please
---
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
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17089#discussion_r103377788
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InsertIntoHadoopFsRelationCommand.scala
---
@@ -148,6 +148,9 @@ case class
Github user skambha commented on the issue:
https://github.com/apache/spark/pull/17067
Thanks much Xiao for the review and comments.
I have made the following changes:
- Separated out the -ve cases from the +ve cases.
- Moved positive tests and also the cases that sh
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/17093#discussion_r103377566
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileIndexSuite.scala
---
@@ -178,6 +179,12 @@ class FileIndexSuite extends S
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17095
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73556/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17095
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17095
**[Test build #73556 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73556/testReport)**
for PR 17095 at commit
[`570ce24`](https://github.com/apache/spark/commit/5
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17093#discussion_r103377147
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileIndexSuite.scala
---
@@ -178,6 +179,12 @@ class FileIndexSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17094
**[Test build #73557 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73557/testReport)**
for PR 17094 at commit
[`9a04d0b`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17094
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73557/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17094
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 does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16867
**[Test build #73563 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73563/testReport)**
for PR 16867 at commit
[`cd16008`](https://github.com/apache/spark/commit/cd
1 - 100 of 613 matches
Mail list logo
