Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15984
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 vanzin commented on the issue:
https://github.com/apache/spark/pull/15974
This will never be merged into 1.6, so you'll have to send a new PR against
master. Please close this one.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15986#discussion_r89256608
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -339,7 +341,7 @@ private[spark] class TaskSchedulerImpl(
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/15904
@ajbozarth Thank you for reminding me, i will take a look at it later.
---
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 uncleGen commented on the issue:
https://github.com/apache/spark/pull/15974
I think there is no need to open another reduplicate PR. Do your mind
closing this PR, and let work on #15904 ?
---
If your project is set up for it, you can reply to this email and have your
Github user ChorPangChan commented on the issue:
https://github.com/apache/spark/pull/15974
I stay adding a new package to streaming is a better structure then modify
the spark-core.
can we make the decision for which implementation to use first
---
If your project is set up for
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15959#discussion_r89258367
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/Statistics.scala
---
@@ -58,60 +61,170 @@ case class Statistics(
}
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15959#discussion_r89258401
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/Statistics.scala
---
@@ -58,60 +61,170 @@ case class Statistics(
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15959
**[Test build #69052 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69052/consoleFull)**
for PR 15959 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15983
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69045/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15983
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/15877
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69047/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15877
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15694
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15986
cc @tejasapatil
---
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
Github user lw-lin commented on the issue:
https://github.com/apache/spark/pull/15949
Also cc @marmbrus @tdas @zsxwing; please be aware of
[SPARK-18552](https://issues.apache.org/jira/browse/SPARK-18552), thanks.
---
If your project is set up for it, you can reply to this email and
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15930
---
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 JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15986#discussion_r89252629
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -89,9 +89,11 @@ private[spark] class TaskSchedulerImpl(
val
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15877
**[Test build #69047 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69047/consoleFull)**
for PR 15877 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15987
Hi, @hvanhovell .
Could you review this PR?
This is the first attempt to use `UnresolvedAttribute` and Analyzer rule.
There are two debatable issues.
- Catalyst Analyzer
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15983#discussion_r89256222
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -373,8 +373,19 @@ final class DataFrameWriter[T] private[sql](ds:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15959#discussion_r89257647
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/Statistics.scala
---
@@ -58,60 +61,170 @@ case class Statistics(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15877
**[Test build #69049 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69049/consoleFull)**
for PR 15877 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15959#discussion_r89258891
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/Statistics.scala
---
@@ -58,60 +61,176 @@ case class Statistics(
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15959
LGTM except one minor comment
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15877
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/15877
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69049/
Test FAILed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15959#discussion_r89265835
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala ---
@@ -310,6 +270,110 @@ class StatisticsSuite extends QueryTest with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15986
**[Test build #69053 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69053/consoleFull)**
for PR 15986 at commit
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/15986
I managed to come up with a standalone end-to-end reproduction of the
shuffle file leak, allowing me to validate this patch's fix.
Run
```
./bin/spark-shell
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15986
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69053/
Test PASSed.
---
GitHub user JoshRosen opened a pull request:
https://github.com/apache/spark/pull/15986
[SPARK-18553][CORE][branch-2.0] Fix leak of TaskSetManager following
executor loss
## What changes were proposed in this pull request?
This patch fixes a critical resource leak in the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15982
**[Test build #69040 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69040/consoleFull)**
for PR 15982 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15987
**[Test build #69050 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69050/consoleFull)**
for PR 15987 at commit
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15986#discussion_r89256275
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -89,9 +89,11 @@ private[spark] class TaskSchedulerImpl(
val
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15959#discussion_r89257869
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/Statistics.scala
---
@@ -58,60 +61,170 @@ case class Statistics(
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15986#discussion_r89257822
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -339,7 +341,7 @@ private[spark] class TaskSchedulerImpl(
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15987
Why does it cast it to double? The fix looks pretty weird.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15959
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69051/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15959
**[Test build #69051 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69051/consoleFull)**
for PR 15959 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15959
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/15986
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/15877
**[Test build #69049 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69049/consoleFull)**
for PR 15877 at commit
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/15877
cc @rxin @hvanhovell
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15985
**[Test build #69048 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69048/consoleFull)**
for PR 15985 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15984
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69041/
Test PASSed.
---
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15986#discussion_r89253541
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -525,7 +525,12 @@ private[spark] class TaskSchedulerImpl(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15984
**[Test build #69041 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69041/consoleFull)**
for PR 15984 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15959#discussion_r89258070
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/Statistics.scala
---
@@ -58,60 +61,170 @@ case class Statistics(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15986
**[Test build #69053 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69053/consoleFull)**
for PR 15986 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15987
**[Test build #69050 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69050/consoleFull)**
for PR 15987 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15988
**[Test build #69056 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69056/consoleFull)**
for PR 15988 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15959
**[Test build #69057 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69057/consoleFull)**
for PR 15959 at commit
GitHub user ericl opened a pull request:
https://github.com/apache/spark/pull/15985
[SPARK-18545] Verify number of hive client RPCs in
PartitionedTablePerfStatsSuite
## What changes were proposed in this pull request?
This would help catch accidental O(n) calls to the hive
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15959
cc @cloud-fan
---
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
Github user Yunni commented on the issue:
https://github.com/apache/spark/pull/15874
Thanks @sethah ! Your comment was very helpful and detailed :-)
---
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/11888
Why did we merge this when the description says "N/A"?
---
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 lw-lin commented on the issue:
https://github.com/apache/spark/pull/15480
@cloud-fan @hvanhovell would you take a look at this? Seems like it's
targeted for 2.1. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user lw-lin commented on the issue:
https://github.com/apache/spark/pull/15480
Jenkins 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
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/15930
Merged into master and branch-2.1. Thanks for reviewing.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15983
**[Test build #69045 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69045/consoleFull)**
for PR 15983 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15971
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/15971
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69042/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15971
**[Test build #69042 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69042/consoleFull)**
for PR 15971 at commit
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15986#discussion_r89255724
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -339,7 +341,7 @@ private[spark] class TaskSchedulerImpl(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15986
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69046/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15986
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/15986
**[Test build #69046 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69046/consoleFull)**
for PR 15986 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15480
**[Test build #69044 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69044/consoleFull)**
for PR 15480 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15480
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/15480
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69044/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15987
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/15987
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69050/
Test FAILed.
---
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15959#discussion_r89264465
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala ---
@@ -310,6 +270,110 @@ class StatisticsSuite extends QueryTest with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15985
**[Test build #69043 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69043/consoleFull)**
for PR 15985 at commit
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15959#discussion_r89258106
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/Statistics.scala
---
@@ -58,60 +61,170 @@ case class Statistics(
}
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15959#discussion_r89258088
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala ---
@@ -319,7 +319,8 @@ class StatisticsSuite extends QueryTest with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15959
**[Test build #69054 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69054/consoleFull)**
for PR 15959 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15694
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15959
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/15983
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/15949
**[Test build #69038 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69038/consoleFull)**
for PR 15949 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15983
**[Test build #69039 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69039/consoleFull)**
for PR 15983 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15949
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69038/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15983
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69039/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15949
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 witgo commented on a diff in the pull request:
https://github.com/apache/spark/pull/15505#discussion_r89249175
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskDescription.scala ---
@@ -17,27 +17,178 @@
package org.apache.spark.scheduler
Github user witgo commented on a diff in the pull request:
https://github.com/apache/spark/pull/15505#discussion_r89249162
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala ---
@@ -592,47 +579,6 @@ class TaskSetManagerSuite extends SparkFunSuite
Github user weiqingy commented on the issue:
https://github.com/apache/spark/pull/15960
@srowen Ok. Thanks for the reply.
---
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/15986
**[Test build #69046 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69046/consoleFull)**
for PR 15986 at commit
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15986#discussion_r89252657
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -89,9 +89,11 @@ private[spark] class TaskSchedulerImpl(
val
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15982
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69040/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15982
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 dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/15987
[SPARK-18515][SQL] AlterTableDropPartitions fails for non-string columns
## What changes were proposed in this pull request?
While
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15986#discussion_r89256137
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -339,7 +341,7 @@ private[spark] class TaskSchedulerImpl(
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15986#discussion_r89256962
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/TaskSchedulerImplSuite.scala ---
@@ -274,4 +274,30 @@ class TaskSchedulerImplSuite extends
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/15974
I think the base solutions are same, expect some other information which I
am working to add.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15959#discussion_r89257721
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/Statistics.scala
---
@@ -58,60 +61,170 @@ case class Statistics(
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15956
Yea, I'll backport
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15959
**[Test build #69055 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69055/consoleFull)**
for PR 15959 at commit
1 - 100 of 638 matches
Mail list logo