[GitHub] spark pull request #20394: [SPARK-23214][SQL] cached data should not carry e...

2018-01-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/20394 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request #20394: [SPARK-23214][SQL] cached data should not carry e...

2018-01-26 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/20394#discussion_r164254321 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryRelation.scala --- @@ -73,11 +73,16 @@ case class InMemoryRelation(

[GitHub] spark pull request #20394: [SPARK-23214][SQL] cached data should not carry e...

2018-01-25 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/20394#discussion_r163943771 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryRelation.scala --- @@ -73,11 +73,16 @@ case class InMemoryRelation(

[GitHub] spark pull request #20394: [SPARK-23214][SQL] cached data should not carry e...

2018-01-25 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/20394#discussion_r163939019 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala --- @@ -169,14 +169,17 @@ class CacheManager extends Logging {

[GitHub] spark pull request #20394: [SPARK-23214][SQL] cached data should not carry e...

2018-01-25 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/20394#discussion_r163890891 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryRelation.scala --- @@ -62,7 +62,7 @@ case class InMemoryRelation(

[GitHub] spark pull request #20394: [SPARK-23214][SQL] cached data should not carry e...

2018-01-25 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/20394#discussion_r163799202 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/joins/BroadcastJoinSuite.scala --- @@ -70,8 +71,8 @@ class BroadcastJoinSuite extends

[GitHub] spark pull request #20394: [SPARK-23214][SQL] cached data should not carry e...

2018-01-25 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/20394#discussion_r163798968 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala --- @@ -169,14 +169,17 @@ class CacheManager extends Logging {

[GitHub] spark pull request #20394: [SPARK-23214][SQL] cached data should not carry e...

2018-01-25 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/20394#discussion_r163799084 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryRelation.scala --- @@ -62,7 +62,7 @@ case class InMemoryRelation(

[GitHub] spark pull request #20394: [SPARK-23214][SQL] cached data should not carry e...

2018-01-25 Thread cloud-fan
GitHub user cloud-fan opened a pull request: https://github.com/apache/spark/pull/20394 [SPARK-23214][SQL] cached data should not carry extra hint info ## What changes were proposed in this pull request? This is a regression introduced by