Skip to content

Conversation

@cloud-fan
Copy link
Contributor

No description provided.

@cloud-fan
Copy link
Contributor Author

cc @yhuai @liancheng

@SparkQA
Copy link

SparkQA commented Oct 25, 2015

Test build #44320 has finished for PR 9271 at commit d0d866b.

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

@yhuai
Copy link
Contributor

yhuai commented Oct 29, 2015

test this please

@yhuai
Copy link
Contributor

yhuai commented Oct 29, 2015

@cloud-fan Seems we also need to fix it in branch 1.5?

@SparkQA
Copy link

SparkQA commented Oct 29, 2015

Test build #44636 has finished for PR 9271 at commit d0d866b.

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

@yhuai
Copy link
Contributor

yhuai commented Oct 29, 2015

LGTM. Merging to master. Looks it will not apply cleanly with 1.5 branch. Can you have a separate pr for 1.5 branch?

@asfgit asfgit closed this in 96cf87f Oct 29, 2015
@cloud-fan
Copy link
Contributor Author

This part of code has been changed a lot by #8916 , should we backport that too?

@yhuai
Copy link
Contributor

yhuai commented Oct 30, 2015

@cloud-fan Seems not very hard to back port #8916 as well. How about we have a pr against 1.5 to include both #8916 and this one?

asfgit pushed a commit that referenced this pull request Oct 30, 2015
…d columns (1.5 backport)

[SPARK-10829](#8916) Filter combine partition key and attribute doesn't work in DataSource scan

[SPARK-11301](#9271) fix case sensitivity for filter on partitioned columns

Author: Wenchen Fan <[email protected]>

This patch had conflicts when merged, resolved by
Committer: Yin Huai <[email protected]>

Closes #9371 from cloud-fan/branch-1.5.
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.

3 participants