Skip to content

Conversation

@scwf
Copy link
Contributor

@scwf scwf commented Oct 21, 2014

Append columns ids and names before broadcast hiveExtraConf in HadoopTableReader.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

1 similar comment
@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@scwf
Copy link
Contributor Author

scwf commented Oct 24, 2014

In now version appended columns ids and names will not broadcast because we append them after creating table reader. This leads to the config broadcasted to executor side dose not contain the configs of appended columns and names.

@SparkQA
Copy link

SparkQA commented Oct 25, 2014

Test build #435 has started for PR 2885 at commit a8c498c.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 25, 2014

Test build #435 has finished for PR 2885 at commit a8c498c.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@scwf
Copy link
Contributor Author

scwf commented Oct 25, 2014

Test failed in core, can you retest this?

@SparkQA
Copy link

SparkQA commented Oct 26, 2014

Test build #456 has started for PR 2885 at commit a8c498c.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 26, 2014

Test build #456 has finished for PR 2885 at commit a8c498c.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Oct 26, 2014

Test build #462 has started for PR 2885 at commit a8c498c.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 26, 2014

Test build #462 has finished for PR 2885 at commit a8c498c.

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

@marmbrus
Copy link
Contributor

Thanks! Merged to master.

@asfgit asfgit closed this in f4e8c28 Oct 26, 2014
@scwf scwf deleted the HadoopTableReader branch October 27, 2014 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants