Skip to content

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

Closed
gvramana wants to merge 2 commits into
apache:masterfrom
gvramana:remove_unnecessary_columns
Closed

gvramana wants to merge 2 commits into
apache:masterfrom
gvramana:remove_unnecessary_columns

Conversation

@gvramana

@gvramana gvramana commented Oct 8, 2014

Copy link
Copy Markdown
Contributor

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

@AmplabJenkins

Copy link
Copy Markdown

Can one of the admins verify this patch?

@marmbrus

marmbrus commented Oct 8, 2014

Copy link
Copy Markdown
Contributor

ok to test

@marmbrus

marmbrus commented Oct 8, 2014

Copy link
Copy Markdown
Contributor

Is there a way to test this?

@AmplabJenkins

Copy link
Copy Markdown

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

gvramana commented Oct 9, 2014

Copy link
Copy Markdown
Contributor Author

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

gvramana commented Oct 9, 2014

Copy link
Copy Markdown
Contributor Author

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

@AmplabJenkins

Copy link
Copy Markdown

Can one of the admins verify this patch?

@SparkQA

SparkQA commented Oct 10, 2014

Copy link
Copy Markdown

QA tests have started for PR 2713 at commit b7ba768.

  • This patch merges cleanly.

@SparkQA

SparkQA commented Oct 10, 2014

Copy link
Copy Markdown

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
Copy Markdown
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