Skip to content

[SPARK-15915][SQL] Logical plans should use canonicalized plan when override sameResult. #13638

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

Closed
ueshin wants to merge 5 commits into from
Closed

[SPARK-15915][SQL] Logical plans should use canonicalized plan when override sameResult. #13638

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
9eb4c25
Add a test to check if DataFrame with plan overriding sameResult but …
ueshin Jun 13, 2016
52641f4
Modify CacheManager to use canonicalized plan for planToCache.
ueshin Jun 13, 2016
11dc433
Use canonicalized plan to compare.
ueshin Jun 13, 2016
3d27607
Revert "Modify CacheManager to use canonicalized plan for planToCache."
ueshin Jun 14, 2016
0c9ec86
Update a test name.
ueshin Jun 14, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Update a test name.
  • Loading branch information
@ueshin
ueshin committed Jun 14, 2016
commit 0c9ec86f7e2646466b91d75b26dc29de31b98d31
2 changes: 1 addition & 1 deletion sql/core/src/test/scala/org/apache/spark/sql/CachedTableSuite.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -553,7 +553,7 @@ class CachedTableSuite extends QueryTest with SQLTestUtils with SharedSQLContext
Seq(Row(1, "1")))
}

test("SPARK-15915 CacheManager should use canonicalized plan for planToCache") {
test("SPARK-15915 Logical plans should use canonicalized plan when override sameResult") {
val localRelation = Seq(1, 2, 3).toDF()
localRelation.createOrReplaceTempView("localRelation")

Expand Down