[GitHub] spark pull request: [SPARK-12138] [SQL] Escape \u in the generated...

2015-12-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10155 --- 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-12138] [SQL] Escape \u in the generated...

2015-12-06 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/10155#issuecomment-162337908 Will also merge it to branch 1.5. --- 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

[GitHub] spark pull request: [SPARK-12138] [SQL] Escape \u in the generated...

2015-12-06 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/10155#issuecomment-162337974 I have also merged it to branch 1.5. --- 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

[GitHub] spark pull request: [SPARK-12138] [SQL] Escape \u in the generated...

2015-12-06 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/10155#issuecomment-162337764 Thanks. LGTM. Merging to master and branch 1.6. --- 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-12138] [SQL] Escape \u in the generated...

2015-12-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10155#issuecomment-162139446 **[Test build #47213 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47213/consoleFull)** for PR 10155 at commit

[GitHub] spark pull request: [SPARK-12138] [SQL] Escape \u in the generated...

2015-12-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10155#issuecomment-162139502 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

[GitHub] spark pull request: [SPARK-12138] [SQL] Escape \u in the generated...

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

[GitHub] spark pull request: [SPARK-12138] [SQL] Escape \u in the generated...

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

[GitHub] spark pull request: [SPARK-12138] [SQL] Escape \u in the generated...

2015-12-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10155#issuecomment-162155462 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-12138] [SQL] Escape \u in the generated...

2015-12-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10155#issuecomment-162155220 **[Test build #47217 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47217/consoleFull)** for PR 10155 at commit

[GitHub] spark pull request: [SPARK-12138] [SQL] Escape \u in the generated...

2015-12-04 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/10155#issuecomment-162140125 Weird... My code changes are not related to the failed test case in SparkR. ``` count(sampled3) < 3 isn't true ``` --- If your project is set

[GitHub] spark pull request: [SPARK-12138] [SQL] Escape \u in the generated...

2015-12-04 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/10155#issuecomment-162140292 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

[GitHub] spark pull request: [SPARK-12138] [SQL] Escape \u in the generated...

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

[GitHub] spark pull request: [SPARK-12138] [SQL] Escape \u in the generated...

2015-12-04 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/10155#issuecomment-162148385 Found a bug in the function `sample` of R. Will submit a PR later. Thanks! --- If your project is set up for it, you can reply to this email and have your reply