Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19229
**[Test build #82344 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82344/testReport)**
for PR 19229 at commit
[`21048a8`](https://github.com/apache/spark/commit/21
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/19393
LGTM but I wrote most of the code so perhaps we should find somebody else
to review.
---
-
To unsubscribe, e-mail: reviews-unsubsc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19287
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19389#discussion_r141999696
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -149,7 +149,7 @@ object TypeCoercion {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19287
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82343/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19287
**[Test build #82343 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82343/testReport)**
for PR 19287 at commit
[`5d09b3b`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19287
**[Test build #82343 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82343/testReport)**
for PR 19287 at commit
[`5d09b3b`](https://github.com/apache/spark/commit/5d
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19389#discussion_r141999620
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetPartitionDiscoverySuite.scala
---
@@ -249,6 +249,11 @@ clas
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19229#discussion_r141999421
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -2102,6 +2102,55 @@ class Dataset[T] private[sql](
}
/**
+
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/18732
Is this just a mapGroups function?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19327
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82342/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19327
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/19327
**[Test build #82342 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82342/testReport)**
for PR 19327 at commit
[`5cc7f0c`](https://github.com/apache/spark/commit/5
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19083
LGTM except one comment
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19083#discussion_r141998362
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -1074,13 +1078,30 @@ object CodeGenerator
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19290
Ah, thank you @shaneknapp. The command below:
```
Rscript -e "devtools::install_github('jimhester/lintr@5431140')"
```
should upgrades `lintr` to the desired version here
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19392#discussion_r141998045
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -510,11 +510,15 @@ rowFormat
;
tableIde
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/19344
ok, thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19344
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19344
These modified test cases are not following the standards. Impala added
extra (partition) predicates. The perf results are misleading.
---
--
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19344
---
-
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/19380
This itself is certainly not a bug. The type is on purpose and certainly
the answer is correct given the type. You are arguing for a new function called
something else but you can also do this with a
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19380
Currently, we are following Hive for these built-in functions. See
https://cwiki.apache.org/confluence/display/Hive/LanguageManual+UDF
Maybe we can wait and see whether more users have th
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/19083
@gatorsmile if you get time, could you check this? Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/19344
btw, could we also add add `tpcds-modifiedQueries` here?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For ad
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19344
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82341/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19344
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/19344
**[Test build #82341 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82341/testReport)**
for PR 19344 at commit
[`8d8a9ff`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19327
**[Test build #82342 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82342/testReport)**
for PR 19327 at commit
[`5cc7f0c`](https://github.com/apache/spark/commit/5c
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19396
IMO I think it might be better to throw an exception instead of shifting to
another shuffle. Since user want to use external shuffle explicitly, letting
user to know the issues and fix the issue w
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19399
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user caneGuy opened a pull request:
https://github.com/apache/spark/pull/19399
[SPARK-22175][WEB-UI] Add status column to history page
## What changes were proposed in this pull request?
Currently, the history page has no status column which represent the status
of a
Github user zuotingbing commented on the issue:
https://github.com/apache/spark/pull/19398
Got it, thanks all.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user zuotingbing closed the pull request at:
https://github.com/apache/spark/pull/19398
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.or
Github user zuotingbing commented on the issue:
https://github.com/apache/spark/pull/19398
Do you know what is the reason to choice to manually create? What about
changing it to create it automatically? Thanks.
---
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19398
See bug update. Please close this PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19344
LGTM pending Jenkins
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19117
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19117
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19370
what is this python console scripts idea, @jsnowacki ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19393
cc @rxin @JoshRosen @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19395#discussion_r141995486
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/CatalogSuite.scala
---
@@ -0,0 +1,37 @@
+/*
+ * Licensed to the Apa
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19395
The Hive might be too old
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19398
AFAIK, this is a by-design choice to manually create event log directory.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19398
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user zuotingbing opened a pull request:
https://github.com/apache/spark/pull/19398
[SPARK-22174][CORE]Support to automatically create the directory where the
event logs go (`spark.eventLog.dir`)
## What changes were proposed in this pull request?
`2017-09-30 09:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19117
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82337/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19117
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/19117
**[Test build #82337 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82337/testReport)**
for PR 19117 at commit
[`cfa0782`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19394
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/19394
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82338/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19394
**[Test build #82338 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82338/testReport)**
for PR 19394 at commit
[`d82ffca`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19397
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user guoxiaolongzte opened a pull request:
https://github.com/apache/spark/pull/19397
[SPARK-22173] Table CSS style needs to be adjusted in History Page and in
Executors Page.
## What changes were proposed in this pull request?
There is a problem with table CSS style
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/19395
Just curious about how would this happen? Is it due to a data corruption in
database?
---
-
To unsubscribe, e-mail: reviews-unsubs
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/19395#discussion_r141994085
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/CatalogSuite.scala
---
@@ -0,0 +1,37 @@
+/*
+ * Licensed to the Apache S
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19344
**[Test build #82341 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82341/testReport)**
for PR 19344 at commit
[`8d8a9ff`](https://github.com/apache/spark/commit/8d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19396
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user devaraj-kavali opened a pull request:
https://github.com/apache/spark/pull/19396
[SPARK-22172][CORE] Worker hangs when the external shuffle service port is
already in use
## What changes were proposed in this pull request?
Handling the NonFatal exceptions while
Github user httfighter commented on the issue:
https://github.com/apache/spark/pull/19380
I understand everyone's worries.But i hava few thoughts.
Firstly, the native unix_timestamp itself supports the "-MM-dd
HH:mm:ss.SSS" form of the date, but the result is lost in millisecon
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19383
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82333/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19383
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/19383
**[Test build #82333 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82333/testReport)**
for PR 19383 at commit
[`886903c`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19344
**[Test build #82340 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82340/testReport)**
for PR 19344 at commit
[`cbac959`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19344
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82340/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19344
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/19393
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82334/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19393
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/19393
**[Test build #82334 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82334/testReport)**
for PR 19393 at commit
[`fb97a73`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19395
**[Test build #82339 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82339/testReport)**
for PR 19395 at commit
[`9b2cc4c`](https://github.com/apache/spark/commit/9b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19344
**[Test build #82340 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82340/testReport)**
for PR 19344 at commit
[`cbac959`](https://github.com/apache/spark/commit/cb
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/19395
[SPARK-22171] [SQL] Describe Table Extended Failed when Table Owner is Empty
## What changes were proposed in this pull request?
Users could hit `java.lang.NullPointerException` when the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19395
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/19395
**[Test build #82339 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82339/testReport)**
for PR 19395 at commit
[`9b2cc4c`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19395
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82339/
Test FAILed.
---
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/19344#discussion_r141990530
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/TPCDSQueryBenchmark.scala
---
@@ -66,24 +64,11 @@ object TPCDSQueryBenchmark exte
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/19394
Ideally, this would also use a TaskMemoryManager so the driver can spill
results to disk instead of dying with an OOM. Is there any plan to add a memory
manager for the driver?
---
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19394
**[Test build #82338 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82338/testReport)**
for PR 19394 at commit
[`d82ffca`](https://github.com/apache/spark/commit/d8
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141989504
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamingSymmetricHashJoinHelper.scala
---
@@ -133,242 +134,6 @@ object StreamingSy
GitHub user rdblue opened a pull request:
https://github.com/apache/spark/pull/19394
SPARK-22170: Reduce memory consumption in broadcast joins.
This updates the broadcast join code path to lazily decompress pages and
iterate through UnsafeRows to prevent all rows from being held
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141987778
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamingSymmetricHashJoinExec.scala
---
@@ -207,31 +221,108 @@ case class Streamin
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141987664
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamingSymmetricHashJoinExec.scala
---
@@ -207,31 +221,108 @@ case class Streamin
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141986897
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamingSymmetricHashJoinExec.scala
---
@@ -207,31 +221,108 @@ case class Streamin
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141986865
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamingSymmetricHashJoinExec.scala
---
@@ -207,31 +221,108 @@ case class Streamin
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141986696
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationsSuite.scala
---
@@ -413,36 +414,103 @@ class UnsupportedOper
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141986708
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationsSuite.scala
---
@@ -413,36 +414,103 @@ class UnsupportedOper
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19309
Will review it this weekend.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mai
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141986634
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationsSuite.scala
---
@@ -413,36 +414,103 @@ class UnsupportedOper
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141986227
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationsSuite.scala
---
@@ -413,36 +414,103 @@ class UnsupportedOper
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141986168
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationsSuite.scala
---
@@ -413,36 +414,103 @@ class UnsupportedOper
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19117
**[Test build #82337 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82337/testReport)**
for PR 19117 at commit
[`cfa0782`](https://github.com/apache/spark/commit/cf
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141985815
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationsSuite.scala
---
@@ -413,36 +414,103 @@ class UnsupportedOper
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19388
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82331/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19388
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/19388
**[Test build #82331 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82331/testReport)**
for PR 19388 at commit
[`4c0985e`](https://github.com/apache/spark/commit/4
Github user joseph-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141984167
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationsSuite.scala
---
@@ -413,36 +414,103 @@ class Unsupp
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/19270
Thanks, I'll pull the latest changes and keep testing. And thanks for your
quick responses, I understand large changes like this take forever to review
and can get frustrating for the submitter.
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19327#discussion_r141982780
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/UnsupportedOperationChecker.scala
---
@@ -233,16 +234,54 @@ object UnsupportedOpe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18732
**[Test build #82336 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82336/testReport)**
for PR 18732 at commit
[`96ce587`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18732
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
1 - 100 of 361 matches
Mail list logo