[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-08-02 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21668 @cloud-fan ping --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@s

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-07-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21668 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93382/ Test PASSed. ---

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-07-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21668 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-07-21 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21668 **[Test build #93382 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93382/testReport)** for PR 21668 at commit [`0b1f751`](https://github.com/apache/spark/commit/0

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-07-21 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21668 **[Test build #93382 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93382/testReport)** for PR 21668 at commit [`0b1f751`](https://github.com/apache/spark/commit/0b

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-07-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21668 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1199/

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-07-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21668 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-07-21 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21668 ping --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-07-11 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21668 @cloud-fan If no problem, could you check #20345 and merge it first? Based on that, I'd like to start refactoring for the approach. ---

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-07-04 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/21668 sounds reasonable, also cc @wzhfy @maryannxue --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additiona

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-07-03 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21668 One of refactoring ideas is to inject the functionality of `ReorderJoin`(=`StarSchemaDetection`) into `CostBasedJoinReorder`; In [the batch rule `Join Reorder` (`Once` strategy)](https:/

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-07-02 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21668 yea, ok. I'll reconsider this again. Thanks! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional com

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-07-02 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/21668 yea this is a real problem, but I feel a better solution is to integrate the StarSchemaDetection into CBO. How hard will it be? --- --

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21668 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-06-29 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21668 **[Test build #92460 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92460/testReport)** for PR 21668 at commit [`f0db73b`](https://github.com/apache/spark/commit/f

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21668 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92460/ Test PASSed. ---

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-06-29 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21668 **[Test build #92460 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92460/testReport)** for PR 21668 at commit [`f0db73b`](https://github.com/apache/spark/commit/f0

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21668 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21668 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/572/

[GitHub] spark issue #21668: [SPARK-24690][SQL] Add a new config to control plan stat...

2018-06-28 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21668 This comes from #20345. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: review