GitHub user jiangxb1987 opened a pull request:
https://github.com/apache/spark/pull/21972
[SPARK-24795][CORE][FOLLOWUP] Combine BarrierTaskContext with
BarrierTaskContextImpl
## What changes were proposed in this pull request?
According to
https://github.com/apache/spark/pull/21758#discussion_r206746905 , current
declaration of `BarrierTaskContext` didn't extend methods from `TaskContext`.
Since `TaskContext` is an abstract class and we don't want to change it to a
trait, we have to define class `BarrierTaskContext` directly.
## How was this patch tested?
Existing tests.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jiangxb1987/spark BarrierTaskContext
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/21972.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #21972
----
commit e5987cf281136528ec0d23f82fe1505abd6545b3
Author: Xingbo Jiang <xingbo.jiang@...>
Date: 2018-08-02T15:09:37Z
combine BarrierTaskContext with BarrierTaskContextImpl.
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
