Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/17297#discussion_r106774285
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -193,13 +193,6 @@ private[spark] class TaskSchedulerImpl
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/16209
Sorry to cutting in though, IMHO we need to have general logic to inject
user-defined types via `UDTRegistration` into the DDL parser
(`CatalystSqlParser`). If we have the logic, we could use the
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/17312
@rxin Thanks a lot. I added a number after `Aggregated Metrics by Executor`
---
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
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/17312
![screenshot](https://cloud.githubusercontent.com/assets/4058918/24069386/0f556622-0be2-11e7-9f48-cc096cdd7d9b.png)
---
If your project is set up for it, you can reply to this email and have
Github user squito commented on the issue:
https://github.com/apache/spark/pull/17297
I'm a bit confused by the description:
> 1. When a fetch failure happens, the task set manager ask the dag
scheduler to abort all the non-running tasks. However, the running tasks in the
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106774155
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,347 @@ package
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106774124
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,347 @@ package
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/17088#discussion_r106774067
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerSuite.scala ---
@@ -394,6 +394,68 @@ class DAGSchedulerSuite extends SparkFunSuite
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/17088#discussion_r106774060
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -1390,7 +1401,34 @@ class DAGScheduler(
}
} else
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/17088#discussion_r106774040
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -1365,18 +1369,25 @@ class DAGScheduler(
*/
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/17088#discussion_r106774047
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -1390,7 +1401,34 @@ class DAGScheduler(
}
} else
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/17088#discussion_r106774045
--- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
---
@@ -1390,7 +1401,34 @@ class DAGScheduler(
}
} else
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106773943
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,347 @@ package
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106773716
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -83,9 +411,19 @@ object ReorderJoin extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106773686
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,347 @@ package
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106773624
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,347 @@ package
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106773584
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,347 @@ package
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/17312
Can you put a screenshot here? Might actually be useful to have.
---
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
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/17318
Can you put the after exception in the pr description as well?
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17337
---
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
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/17337
Merging in master/branch-2.1.
---
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17138#discussion_r106773079
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -0,0 +1,297 @@
+/*
+ * Licensed
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17138#discussion_r106773023
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -0,0 +1,297 @@
+/*
+ * Licensed
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17138#discussion_r106773013
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -0,0 +1,297 @@
+/*
+ * Licensed
Github user yaooqinn commented on the issue:
https://github.com/apache/spark/pull/17333
@jerryshao ð
---
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106772868
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,347 @@ package
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17240#discussion_r106772853
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
---
@@ -36,27 +36,24 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17338
**[Test build #74769 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74769/testReport)**
for PR 17338 at commit
GitHub user windpiger opened a pull request:
https://github.com/apache/spark/pull/17338
[SPARK-19990][SQL]create a temp file for file in test.jar's resource when
run test accross different modules
## What changes were proposed in this pull request?
After we have merged the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17112
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74765/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17112
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17112
**[Test build #74765 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74765/testReport)**
for PR 17112 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17191
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74764/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17191
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17191
**[Test build #74764 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74764/testReport)**
for PR 17191 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17219
**[Test build #74767 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74767/testReport)**
for PR 17219 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17088
**[Test build #74768 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74768/testReport)**
for PR 17088 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106772404
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,347 @@ package
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106772374
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,347 @@ package
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17219
**[Test build #74766 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74766/testReport)**
for PR 17219 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17192
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17192
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74763/
Test PASSed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106772124
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,347 @@ package
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17192
**[Test build #74763 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74763/testReport)**
for PR 17192 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106772029
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,340 @@ package
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106771968
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/SimpleCatalystConf.scala
---
@@ -40,6 +40,9 @@ case class SimpleCatalystConf(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17337
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17337
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74762/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17337
**[Test build #74762 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74762/testReport)**
for PR 17337 at commit
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/17216
Seems like an unrelated failure. Probably a flaky test.
On Mar 17, 2017 4:23 PM, "UCB AMPLab" wrote:
> Merged build finished. Test FAILed.
>
> â
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17164
@hvanhovell ping
---
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
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17318
@cloud-fan ping
---
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
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17191#discussion_r106770623
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -2598,4 +2598,26 @@ class SQLQuerySuite extends QueryTest with
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17191
@gatorsmile I checked and actually MySQL and PostgreSQL cannnot use numbers
as alias names;
```
// PostgreSQL
postgres=# SELECT gkey1 AS 1 FROM t2;
ERROR: syntax error at or
Github user yanji84 commented on the issue:
https://github.com/apache/spark/pull/17109
hello, any update on this?
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17112
**[Test build #74765 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74765/testReport)**
for PR 17112 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17191
**[Test build #74764 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74764/testReport)**
for PR 17191 at commit
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/16867
@squito
Sure. I did test for 100k tasks. The results are as below:
| | time cost |
| --| -- |
| insert | 135ms, 122ms, 119ms, 120ms, 163ms |
|
Github user ioana-delaney commented on the issue:
https://github.com/apache/spark/pull/17286
@wzhfy Some thoughts on how to solve the Cartesian problem as part of the
join enumeration algorithm is to apply a similar strategy to the one that we
discuss for star-plans. You keep track
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17191
aha, I'll check and just a sec.
---
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17192
(I just rebased to resolve the conflicts)
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17192
**[Test build #74763 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74763/testReport)**
for PR 17192 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17337
**[Test build #74762 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74762/testReport)**
for PR 17337 at commit
GitHub user jaceklaskowski opened a pull request:
https://github.com/apache/spark/pull/17337
[SQL][MINOR] Fix scaladoc for UDFRegistration
## What changes were proposed in this pull request?
Fix scaladoc for UDFRegistration
## How was this patch tested?
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17333
What a coincidence ð !
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16626
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16626
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74761/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16626
**[Test build #74761 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74761/testReport)**
for PR 16626 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r106768186
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/FailureSafeParser.scala
---
@@ -0,0 +1,68 @@
+/*
+ * Licensed to the
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17315#discussion_r106767486
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/FailureSafeParser.scala
---
@@ -0,0 +1,68 @@
+/*
+ * Licensed to the
Github user ioana-delaney commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106767465
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,347 @@ package
Github user lw-lin commented on the issue:
https://github.com/apache/spark/pull/17327
Thanks! Closed this.
---
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 lw-lin closed the pull request at:
https://github.com/apache/spark/pull/17327
---
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
Github user ron8hu commented on a diff in the pull request:
https://github.com/apache/spark/pull/15363#discussion_r106766281
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/joins.scala
---
@@ -20,19 +20,347 @@ package
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r106765877
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -296,6 +297,45 @@ class SessionCatalog(
}
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14615
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14615
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74759/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14615
**[Test build #74759 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74759/testReport)**
for PR 14615 at commit
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17326#discussion_r106765091
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/MultilayerPerceptronClassifierSuite.scala
---
@@ -74,6 +74,7 @@ class
Github user hhbyyh commented on the issue:
https://github.com/apache/spark/pull/17014
Hi @zhengruifeng , is there any update?
---
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17191
What is the behaviors of MySQL and Postgres when we use digits as alias?
```SQL
SELECT k1 AS `2`, k2 AS a, SUM(v) FROM t GROUP BY 2, k2
```
You might need to replace backticks
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/17179
LGTM.
---
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/17216
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17216
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74760/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17216
**[Test build #74760 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74760/testReport)**
for PR 17216 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17216
---
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
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/17216
LGTM. Merging it to master.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17216
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17216
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74758/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17216
**[Test build #74758 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74758/testReport)**
for PR 17216 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17216
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17216
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74757/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17216
**[Test build #74757 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74757/testReport)**
for PR 17216 at commit
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/17330#discussion_r106761010
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/subquery.scala
---
@@ -61,6 +63,36 @@ abstract class
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/17088
Ok that makes sense. I wanted to make sure that there wasn't some bug in
SlaveLost (which might lead to a simpler fix than this) but @squito's
description makes it clear that there are a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16626
**[Test build #74761 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74761/testReport)**
for PR 16626 at commit
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/17216#discussion_r106759443
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamExecution.scala
---
@@ -256,6 +259,15 @@ class StreamExecution(
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17330#discussion_r106758290
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/subquery.scala
---
@@ -61,6 +63,36 @@ abstract class SubqueryExpression(
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/17216#discussion_r106757213
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamExecution.scala
---
@@ -256,6 +259,15 @@ class StreamExecution(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17216
**[Test build #74760 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74760/testReport)**
for PR 17216 at commit
1 - 100 of 402 matches
Mail list logo