Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/2782#discussion_r18783262
--- Diff: core/src/main/java/org/apache/spark/TaskContext.java ---
@@ -37,7 +37,7 @@
* Contextual information about a task which can be read or mutated during
execution.
*/
@DeveloperApi
-public class TaskContext implements Serializable {
+public abstract class TaskContext implements Serializable {
--- End diff --
I'd prefer this class not have a constructor (the whole idea here is that
we don't want user to instantiate it). Basically we want to move all of the
logic here to `TaskContextImpl` and only define the interface here in terms of
abstract methods.
---
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 enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
