[SPARK-6754] Remove unnecessary TaskContextHelper

[kayousterhout]

The TaskContextHelper was originally necessary because TaskContext was written in Java, which does
not have a way to specify that classes are package-private, so TaskContextHelper existed to work
around this. Now that TaskContext has been re-written in Scala, this class is no longer necessary.

@rxin can you look at this? It looks like you missed this bit of cleanup when you moved TaskContext from Java to Scala in #4324

cc @ScrapCodes and @pwendell who added this originally.

[rxin]

LGTM

[SparkQA]

Test build #29811 has finished for PR 5402 at commit f089800.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.
• This patch does not change any dependencies.

[pwendell]

LGTM

On Tue, Apr 7, 2015 at 6:26 PM, UCB AMPLab [email protected] wrote:

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29811/
Test PASSed.

—
Reply to this email directly or view it on GitHub
#5402 (comment).

[ScrapCodes]

(Though it is redundant to say. :) ) LGTM too.

[rxin]

I'm merging this in master. Thanks.