[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-149457452 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-149457470 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-149458541 **[Test build #43968 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43968/consoleFull)** for PR 8621 at commit

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-20 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-149451946 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

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8621 --- 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 feature is

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-19 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-149397394 If I understand correct, the `PromotePrecision` only will be used for type coercion, which is in the analysis phase, and remove it in optimize phase should be

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-148502689 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-148502712 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-15 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-148561118 cc @yhuai --- 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

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-15 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/8621#discussion_r42198441 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -732,11 +732,12 @@ object SimplifyCasts extends

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-148504866 [Test build #43804 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43804/consoleFull) for PR 8621 at commit

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-148537980 [Test build #43804 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43804/console) for PR 8621 at commit

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-148538141 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

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-10-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-148538143 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-09-15 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-140354172 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

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-09-14 Thread adrian-wang
Github user adrian-wang commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-140244187 @chenghao-intel --- 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

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-09-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-138124030 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

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-09-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-138124031 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-09-06 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-138123997 [Test build #42070 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42070/console) for PR 8621 at commit

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-09-06 Thread adrian-wang
GitHub user adrian-wang opened a pull request: https://github.com/apache/spark/pull/8621 [SPARK-10463] [SQL] remove PromotePrecision during optimization PromotePrecision is not necessary after HiveTypeCoercion done. Jira:

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-09-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-138051536 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-09-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-138108136 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-10463] [SQL] remove PromotePrecision du...

2015-09-06 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8621#issuecomment-138109217 [Test build #42070 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42070/consoleFull) for PR 8621 at commit