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.

Sign up for GitHub

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

Jump to bottom

[SPARK-4959][SQL] Attributes are case sensitive when using a select query from a projection(Backport to Spark-1.2) #4013

Closed
wants to merge 1 commit into from
Closed

[SPARK-4959][SQL] Attributes are case sensitive when using a select query from a projection(Backport to Spark-1.2) #4013

wants to merge 1 commit into from

Conversation

chenghao-intel
Copy link
Contributor

This is a follow up of #3796 , which can not be merged back to Spark-1.2. Manually merge it.

@SparkQA
Copy link

SparkQA commented Jan 13, 2015

Test build #25439 has started for PR 4013 at commit 1f6c93d.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Jan 13, 2015

Test build #25439 has finished for PR 4013 at commit 1f6c93d.

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

@AmplabJenkins
Copy link

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

@chenghao-intel
Copy link
Contributor Author

cc @rxin @marmbrus

@rxin
Copy link
Contributor

rxin commented Jan 21, 2015

Thanks. Merging.

asfgit pushed a commit that referenced this pull request Jan 21, 2015
@chenghao-intel @rxin
[SPARK-4959][SQL] Attributes are case sensitive when using a select q…
92c238c 
…uery from a projection(Backport to Spark-1.2)

This is a follow up of #3796 , which can not be merged back to Spark-1.2. Manually merge it.

Author: Cheng Hao <hao.cheng@intel.com>

Closes #4013 from chenghao-intel/spark_4959_backport and squashes the following commits:

1f6c93d [Cheng Hao] backport to Spark-1.2
@chenghao-intel
Copy link
Contributor Author

Oh? Should I close this PR?

@yhuai
Copy link
Contributor

yhuai commented Jan 21, 2015

Yes, please.

@rxin
Copy link
Contributor

rxin commented Jan 21, 2015

Yes, please close. It's already been merged (but github doesn't close it automatically because it was not merged into master). Thanks!

@chenghao-intel chenghao-intel deleted the spark_4959_backport branch January 22, 2015 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@chenghao-intel @SparkQA @AmplabJenkins @rxin @yhuai