[SQL]redundant methods for broadcast

[scwf]

redundant methods for broadcast in TableReader

[AmplabJenkins]

Can one of the admins verify this patch?

[JoshRosen]

Jenkins, this is ok to test.

[SparkQA]

QA tests have started for PR 2862 at commit 414cc24.

• This patch merges cleanly.

[SparkQA]

QA tests have finished for PR 2862 at commit 414cc24.

• This patch fails to build.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • case class ReconnectWorker(masterUrl: String) extends DeployMessage

[AmplabJenkins]

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

[scwf]

@JoshRosen test failed in streaming, compile error? it's very confusing, can you retest this?

[JoshRosen]

Jenkins, retest this please.

[SparkQA]

QA tests have started for PR 2862 at commit 414cc24.

• This patch merges cleanly.

[SparkQA]

QA tests have finished for PR 2862 at commit 414cc24.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

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

[AmplabJenkins]

Can one of the admins verify this patch?

[JoshRosen]

LGTM; I thought I had cleaned this up, but I guess the change got lost in the course of the JobConf cloning patch. I'll merge this into master.