[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176608171 Yea the pull request description gets merged as commit description, and is the only thing that sticks around forever. --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176607697 Sorry I thought the JIRA description tells it, will add it to PR description next time. --- If your project is set up for it, you can reply to this email and have yo

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176605624 OK got it. @cloud-fan Please add that to the pull request description next time. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176605185 Existing tests were failing locally, but not on Jenkins. We are not sure why. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10970 --- 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 ena

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176603047 @cloud-fan is this fixing a bug? If yes, please add a test somewhere. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176602785 Tested it locally, test passed. I'm merging this into master, thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176579246 "。" replace with "." ? --- 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-13067][SQL] workaround for a weird scal...

2016-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176560874 **[Test build #2472 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2472/consoleFull)** for PR 10970 at commit [`7e66869`](https://

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176531526 **[Test build #2472 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2472/consoleFull)** for PR 10970 at commit [`7e66869`](https://g

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176506985 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176506889 **[Test build #50320 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50320/consoleFull)** for PR 10970 at commit [`7e66869`](https://g

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176506982 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 projec

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176499567 **[Test build #50320 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50320/consoleFull)** for PR 10970 at commit [`7e66869`](https://gi

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176497748 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

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176495845 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176495842 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 projec

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176495803 **[Test build #50316 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50316/consoleFull)** for PR 10970 at commit [`7e66869`](https://g

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176489619 **[Test build #50316 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50316/consoleFull)** for PR 10970 at commit [`7e66869`](https://gi

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176487968 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

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176480160 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 projec

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176480162 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/10970#issuecomment-176475787 CC @davies @yhuai @marmbrus @davies can you try it out locally? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-13067][SQL] workaround for a weird scal...

2016-01-28 Thread cloud-fan
GitHub user cloud-fan opened a pull request: https://github.com/apache/spark/pull/10970 [SPARK-13067][SQL] workaround for a weird scala reflection problem A simple workaround to avoid getting parameter types when convert a logical plan to json。 You can merge this pull request int