[GitHub] [spark] maropu commented on a change in pull request #28123: [SPARK-31350][SQL] Coalesce bucketed tables for sort merge join if applicable

2020-06-18 Thread GitBox
maropu commented on a change in pull request #28123: URL: https://github.com/apache/spark/pull/28123#discussion_r442637400 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/bucketing/CoalesceBucketsInSortMergeJoin.scala ## @@ -0,0 +1,132 @@ +/* + * Licensed

[GitHub] [spark] maropu commented on a change in pull request #28123: [SPARK-31350][SQL] Coalesce bucketed tables for sort merge join if applicable

2020-06-18 Thread GitBox
maropu commented on a change in pull request #28123: URL: https://github.com/apache/spark/pull/28123#discussion_r442637311 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/bucketing/CoalesceBucketsInSortMergeJoinSuite.scala ## @@ -0,0 +1,194 @@ +/* + * Lice

[GitHub] [spark] maropu commented on a change in pull request #28123: [SPARK-31350][SQL] Coalesce bucketed tables for sort merge join if applicable

2020-06-18 Thread GitBox
maropu commented on a change in pull request #28123: URL: https://github.com/apache/spark/pull/28123#discussion_r442596514 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/bucketing/CoalesceBucketsInSortMergeJoin.scala ## @@ -0,0 +1,132 @@ +/* + * Licensed

[GitHub] [spark] maropu commented on a change in pull request #28123: [SPARK-31350][SQL] Coalesce bucketed tables for sort merge join if applicable

2020-06-18 Thread GitBox
maropu commented on a change in pull request #28123: URL: https://github.com/apache/spark/pull/28123#discussion_r442596155 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/bucketing/CoalesceBucketsInSortMergeJoinSuite.scala ## @@ -0,0 +1,194 @@ +/* + * Lice

[GitHub] [spark] maropu commented on a change in pull request #28123: [SPARK-31350][SQL] Coalesce bucketed tables for sort merge join if applicable

2020-06-18 Thread GitBox
maropu commented on a change in pull request #28123: URL: https://github.com/apache/spark/pull/28123#discussion_r442028714 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/DataSourceScanExec.scala ## @@ -356,7 +360,8 @@ case class FileSourceScanExec(

[GitHub] [spark] maropu commented on a change in pull request #28123: [SPARK-31350][SQL] Coalesce bucketed tables for sort merge join if applicable

2020-06-18 Thread GitBox
maropu commented on a change in pull request #28123: URL: https://github.com/apache/spark/pull/28123#discussion_r442026312 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/DataSourceScanExec.scala ## @@ -544,8 +549,19 @@ case class FileSourceScanExec(

[GitHub] [spark] maropu commented on a change in pull request #28123: [SPARK-31350][SQL] Coalesce bucketed tables for sort merge join if applicable

2020-05-31 Thread GitBox
maropu commented on a change in pull request #28123: URL: https://github.com/apache/spark/pull/28123#discussion_r432942906 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/bucketing/CoalesceBucketsInSortMergeJoin.scala ## @@ -0,0 +1,112 @@ +/* + * Licensed