Github user chetkhatri commented on a diff in the pull request:
https://github.com/apache/spark/pull/20018#discussion_r158368554
--- Diff:
examples/src/main/scala/org/apache/spark/examples/sql/hive/SparkHiveExample.scala
---
@@ -102,8 +101,63 @@ object SparkHiveExample {
Github user chetkhatri commented on a diff in the pull request:
https://github.com/apache/spark/pull/20018#discussion_r158366994
--- Diff:
examples/src/main/scala/org/apache/spark/examples/sql/hive/SparkHiveExample.scala
---
@@ -102,8 +101,63 @@ object SparkHiveExample {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20048
**[Test build #85282 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85282/testReport)**
for PR 20048 at commit
GitHub user marmbrus opened a pull request:
https://github.com/apache/spark/pull/20048
[SPARK-22862] Docs on lazy elimination of columns missing from an encoder
This behavior has confused some users, so lets clarify it.
You can merge this pull request into a Git repository by
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19715
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85275/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19715
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/19993
**[Test build #85281 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85281/testReport)**
for PR 19993 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19715
**[Test build #85275 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85275/testReport)**
for PR 19715 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19993
thanks @hhbyyh, I removed the 2 too strict checks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20043#discussion_r158361417
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -56,7 +56,36 @@ import
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20043#discussion_r158361160
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/nullExpressions.scala
---
@@ -321,7 +322,12 @@ case class IsNull(child:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20047
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/20047
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85271/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20047
**[Test build #85271 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85271/testReport)**
for PR 20047 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20047
**[Test build #85280 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85280/testReport)**
for PR 20047 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19591
**[Test build #85279 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85279/testReport)**
for PR 19591 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19946
**[Test build #85278 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85278/testReport)**
for PR 19946 at commit
Github user foxish commented on a diff in the pull request:
https://github.com/apache/spark/pull/19946#discussion_r158357822
--- Diff: docs/running-on-yarn.md ---
@@ -18,7 +18,8 @@ Spark application's configuration (driver, executors, and
the AM when running in
There
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19591
> you're talking about removing the System.exit() in SparkSubmit?
Yes, and also potentially changing error messages currently printed to the
terminal into exceptions when running through the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20045
**[Test build #85277 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85277/testReport)**
for PR 20045 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20046
**[Test build #85276 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85276/testReport)**
for PR 20046 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19591#discussion_r158355602
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/LauncherServer.java ---
@@ -210,8 +209,13 @@ int getPort() {
* Removes the client handle
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20020
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/20020
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85269/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20020
**[Test build #85269 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85269/testReport)**
for PR 20020 at commit
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20046
Jenkins, ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20045
Jenkins, ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/19993
Yes featurehasher doesnât need inputCol and inputCols - since it is a new
multi column transformer. We may see this more in future - as I think new
transformers should be able to work on
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19591#discussion_r158351004
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/LauncherServer.java ---
@@ -260,24 +264,30 @@ private long getConnectionTimeout() {
}
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19591#discussion_r158349922
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/InProcessAppHandle.java ---
@@ -0,0 +1,74 @@
+/*
+ * Licensed to the Apache Software
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20020
How about the command `SaveIntoDataSourceCommand`?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19715
**[Test build #85275 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85275/testReport)**
for PR 19715 at commit
Github user huaxingao commented on a diff in the pull request:
https://github.com/apache/spark/pull/19715#discussion_r158347717
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/QuantileDiscretizerSuite.scala
---
@@ -386,19 +382,16 @@ class QuantileDiscretizerSuite
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20034
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85270/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20034
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/20034
**[Test build #85270 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85270/testReport)**
for PR 20034 at commit
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/19993#discussion_r158344862
--- Diff: mllib/src/main/scala/org/apache/spark/ml/param/params.scala ---
@@ -249,6 +250,31 @@ object ParamValidators {
def
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20020#discussion_r158344919
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveStrategies.scala ---
@@ -148,7 +148,8 @@ object HiveAnalysis extends Rule[LogicalPlan] {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20020#discussion_r158344489
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -479,14 +468,15 @@ case class DataSource(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20020#discussion_r158341954
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/DataWritingCommand.scala
---
@@ -20,30 +20,32 @@ package
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20020#discussion_r158341832
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -27,7 +27,7 @@ import
Github user rxin closed the pull request at:
https://github.com/apache/spark/pull/19973
---
-
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/20047
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/20047
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85274/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20047
**[Test build #85274 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85274/testReport)**
for PR 20047 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19977
LGTM except the comments left above by @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user squito commented on the issue:
https://github.com/apache/spark/pull/17619
Sorry I am only just looking at this now --
I am not so sure this is doing what you think. I think the notion of
"task" in MesosCoarseGrainedSchedulerBackend might be something different,
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19977#discussion_r158339454
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -48,17 +48,37 @@ import
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20047
**[Test build #85274 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85274/testReport)**
for PR 20047 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19977#discussion_r158337643
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -73,17 +93,27 @@ case class
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19940#discussion_r158336014
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -1193,7 +1196,8 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19940
**[Test build #85273 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85273/testReport)**
for PR 19940 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20020
**[Test build #85272 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85272/testReport)**
for PR 20020 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20047
Looks good but you have to fix conflicts now.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20039#discussion_r158333948
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/LiveListenerBus.scala ---
@@ -124,13 +127,19 @@ private[spark] class LiveListenerBus(conf:
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/20020#discussion_r158333591
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/commands.scala
---
@@ -87,6 +87,44 @@ case class ExecutedCommandExec(cmd:
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20034#discussion_r158333472
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -186,7 +186,7 @@ private[hive] class HiveClientImpl(
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20041
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20034#discussion_r158332740
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -186,7 +186,7 @@ private[hive] class HiveClientImpl(
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20008
---
-
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/20041
LGTM
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20008
The test failure is not related to this PR.
LGTM
Thanks! Merged to master.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20019
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85267/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20019
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/20019
**[Test build #85267 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85267/testReport)**
for PR 20019 at commit
Github user dongjoon-hyun closed the pull request at:
https://github.com/apache/spark/pull/17980
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17980
Almost codes inside this PR are merged now except #19943 .
---
-
To unsubscribe, e-mail:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19981
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19981
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20043#discussion_r158329715
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -56,7 +56,36 @@ import
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20020#discussion_r158329686
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/commands.scala
---
@@ -87,6 +87,44 @@ case class ExecutedCommandExec(cmd:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20020
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85266/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20020
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/20020
**[Test build #85266 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85266/testReport)**
for PR 20020 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20043#discussion_r158328435
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala
---
@@ -118,10 +118,10 @@ abstract class Expression
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19977#discussion_r158327829
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -658,6 +660,30 @@ object TypeCoercion {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20047
**[Test build #85271 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85271/testReport)**
for PR 20047 at commit
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20047
@vanzin @carsonwang
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user squito opened a pull request:
https://github.com/apache/spark/pull/20047
[SPARK-22861][SQL] SQLAppStatusListener handles multi-job executions.
When one execution has multiple jobs, we need to append to the set of
stages, not replace them on every job.
Added
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19977#discussion_r158327166
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -73,17 +93,27 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20046
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19977#discussion_r158326164
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -48,17 +48,37 @@ import
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/19681#discussion_r158326122
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/SQLAppStatusListener.scala
---
@@ -0,0 +1,366 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19981
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user smurakozi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20046#discussion_r158326073
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameWindowFunctionsSuite.scala
---
@@ -154,6 +154,217 @@ class DataFrameWindowFunctionsSuite
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20041
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/20041
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85265/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19981
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85263/
Test PASSed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19977#discussion_r158325851
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -48,17 +48,37 @@ import
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20041
**[Test build #85265 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85265/testReport)**
for PR 20041 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19981
**[Test build #85263 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85263/testReport)**
for PR 19981 at commit
GitHub user attilapiros opened a pull request:
https://github.com/apache/spark/pull/20046
Adding unit tests for missing aggregate functions
## What changes were proposed in this pull request?
Improving the test coverage of window functions focusing on missing test
for
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19940
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19940#discussion_r158324941
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -1193,7 +1196,8 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20043
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85264/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20043
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/20043
**[Test build #85264 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85264/testReport)**
for PR 20043 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19940#discussion_r158323966
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -195,6 +195,14 @@ class
Github user wesm commented on the issue:
https://github.com/apache/spark/pull/19884
@BryanCutler can you give me a minimal repro for the timestamp issue you
cited above?
---
-
To unsubscribe, e-mail:
Github user Jiri-Kremser commented on the issue:
https://github.com/apache/spark/pull/19802
I've also added tests and removed "WIP" from the label? Do you guys want
the change or would you like to have something more elaborated, like an initial
handshake?
One note about the
201 - 300 of 500 matches
Mail list logo
