Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-3559][SQL] Remove unnecessary columns from List of needed Column Ids in Hive Conf #2713

Closed
wants to merge 2 commits into from

Conversation

gvramana
Copy link
Contributor

@gvramana gvramana commented Oct 8, 2014

Author: Venkata Ramana G ramana.gollamudi@huawei.com

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@marmbrus
Copy link
Contributor

marmbrus commented Oct 8, 2014

ok to test

@marmbrus
Copy link
Contributor

marmbrus commented Oct 8, 2014

Is there a way to test this?

@AmplabJenkins
Copy link

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

@gvramana
Copy link
Contributor Author

gvramana commented Oct 9, 2014

I have verified that only required columns are being passed by adding log. This cannot be verified by result test, as TableReader will pick required columns only when converting result to row.

@gvramana
Copy link
Contributor Author

gvramana commented Oct 9, 2014

@marmbrus, Jenkins Test Failure is due to git fetch failure. Can you please trigger the test again.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@SparkQA
Copy link

SparkQA commented Oct 10, 2014

QA tests have started for PR 2713 at commit b7ba768.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 10, 2014

QA tests have finished for PR 2713 at commit b7ba768.

  • 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 e10d71e Oct 13, 2014
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