[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

2017-02-04 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/16786 Thank you @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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

2017-02-03 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/16786 Thanks, merging to master! --- 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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

2017-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16786 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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

2017-02-03 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/16786 LGTM pending test --- 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 #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

2017-02-03 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/16786 retest this please --- 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 #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

2017-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16786 Merged build finished. Test FAILed. --- 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 issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

2017-02-03 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/16786 Thanks, I just updated and rebased. --- 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 issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

2017-02-03 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/16786 Also clean up the comments? --- 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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

2017-02-03 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/16786 Could you rename `findTightestCommonTypeOfTwo` to `findTightestCommonType`? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

2017-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16786 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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

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

[GitHub] spark issue #16786: [SPARK-19446][SQL] Remove unused findTightestCommonType ...

2017-02-03 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/16786 cc @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 so,