[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

2017-06-02 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/18189 cc @rxin @gatorsmile --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

2017-06-02 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18189 **[Test build #77689 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77689/testReport)** for PR 18189 at commit [`3dea8e0`](https://github.com/apache/spark/commit/3d

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

2017-06-02 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18189 **[Test build #77689 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77689/testReport)** for PR 18189 at commit [`3dea8e0`](https://github.com/apache/spark/commit/3

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

2017-06-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18189 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature e

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

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

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

2017-06-02 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/18189 cc @bogdanrdc --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

2017-06-03 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/18189 tbh the difference is so small that i don't think it is worth spending time here ... as pointed out it is not forceBroadcast either. --- If your project is set up for it, you can reply to this email a

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

2017-06-03 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/18189 I found this problem when I was playing with CBO stuff. This name(`isBroadcastable`) will be displayed with the query plan when users run `EXPLAIN COST`, that's why I think it matters. --- If yo

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

2017-06-04 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/18189 But isn't it in a hint? If you are worried about user, I'd just change it to "broadcast". --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

2017-06-06 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18189 **[Test build #0 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/0/testReport)** for PR 18189 at commit [`3706ab2`](https://github.com/apache/spark/commit/37

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

2017-06-06 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18189 **[Test build #0 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/0/testReport)** for PR 18189 at commit [`3706ab2`](https://github.com/apache/spark/commit/3

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

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

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

2017-06-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18189 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature e

[GitHub] spark issue #18189: [SPARK-20972][SQL] rename HintInfo.isBroadcastable to fo...

2017-06-06 Thread viirya
Github user viirya commented on the issue: https://github.com/apache/spark/pull/18189 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur