[GitHub] spark pull request: [SPARK-10890][SQL] don't log expected JdbcSQLE...

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

[GitHub] spark pull request: [SPARK-10342] [SPARK-10309] [SPARK-10474] [SPA...

2015-10-29 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/9241#discussion_r43464654 --- Diff: core/src/main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java --- @@ -219,61 +230,158 @@ public BytesToBytesMap( */ public

[GitHub] spark pull request: [SPARK-10342] [SPARK-10309] [SPARK-10474] [SPA...

2015-10-29 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/9241#discussion_r43464618 --- Diff: core/src/main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java --- @@ -219,61 +230,158 @@ public BytesToBytesMap( */ public

[GitHub] spark pull request: [SPARK-10342] [SPARK-10309] [SPARK-10474] [SPA...

2015-10-29 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/9241#discussion_r43464779 --- Diff: core/src/main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java --- @@ -227,62 +238,147 @@ public BytesToBytesMap( */ public

[GitHub] spark pull request: [SPARK-11383][Docs] Replaced example code in m...

2015-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9353#issuecomment-152378606 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-11383][Docs] Replaced example code in m...

2015-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9353#issuecomment-152378558 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-10342] [SPARK-10309] [SPARK-10474] [SPA...

2015-10-29 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/9241#discussion_r43464962 --- Diff: core/src/main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java --- @@ -678,20 +747,31 @@ private void allocate(int capacity) { }

[GitHub] spark pull request: [SPARK-11383][Docs] Replaced example code in m...

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

[GitHub] spark pull request: [SPARK-10342] [SPARK-10309] [SPARK-10474] [SPA...

2015-10-29 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9241#issuecomment-152379629 LGTM overall right now (pending Jenkins for commit 4ee1f42). We can address any minor issues in followups. I'll take one final glance at one of the new unit

[GitHub] spark pull request: [SPARK-11195][CORE] Use correct classloader fo...

2015-10-29 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9367#issuecomment-152379692 @brkyvz might know more about testing Spark Submit. --- 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 pull request: [SPARK-2827][GraphX] Add collectDegreeDist to ...

2015-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4399#issuecomment-152379639 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-2827][GraphX] Add collectDegreeDist to ...

2015-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4399#issuecomment-152379647 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-11195][CORE] Use correct classloader fo...

2015-10-29 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9367#issuecomment-152379661 Jenkins, this is ok to 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

[GitHub] spark pull request: [SPARK-11195][CORE] Use correct classloader fo...

2015-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9367#issuecomment-152380950 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-2827][GraphX] Add collectDegreeDist to ...

2015-10-29 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/4399#issuecomment-152381853 @ankurdave @andrewor14 Fixed and could you merge this? --- 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 pull request: [SPARK-2827][GraphX] Add collectDegreeDist to ...

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

[GitHub] spark pull request: [SPARK-10342] [SPARK-10309] [SPARK-10474] [SPA...

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

[GitHub] spark pull request: [SPARK-10342] [SPARK-10309] [SPARK-10474] [SPA...

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

[GitHub] spark pull request: [SPARK-10342] [SPARK-10309] [SPARK-10474] [SPA...

2015-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9241#issuecomment-152385979 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-11195][CORE] Use correct classloader fo...

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

[GitHub] spark pull request: [SPARK-11057] [SQL] Add correlation and covari...

2015-10-29 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/9366#issuecomment-152387852 cc @mengxr --- 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-11383][Docs] Replaced example code in m...

2015-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9353#issuecomment-152388583 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-11383][Docs] Replaced example code in m...

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

[GitHub] spark pull request: [SPARK-11383][Docs] Replaced example code in m...

2015-10-29 Thread rishabhbhardwaj
Github user rishabhbhardwaj commented on the pull request: https://github.com/apache/spark/pull/9353#issuecomment-152390706 @yinxusen I have removed the empty lines.Thanks for the comments. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-11273][SQL] Move ArrayData/MapData/Data...

2015-10-29 Thread cequencer
Github user cequencer commented on the pull request: https://github.com/apache/spark/pull/9239#issuecomment-152391757 I am trying to build the latest master branch, but I am having compile errors that seemed to be coming from this commit: cdea0174e32a5f4c28fd59899b2e9774994303d5

[GitHub] spark pull request: [SPARK-11383][Docs] Replaced example code in m...

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

[GitHub] spark pull request: [SPARK-11024][SQL] Optimize NULL in

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

[GitHub] spark pull request: [SPARK-11024][SQL] Optimize NULL in

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

[GitHub] spark pull request: [SPARK-11024][SQL] Optimize NULL in

2015-10-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9348#issuecomment-152392941 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-11410] [SQL] Add APIs to provide functi...

2015-10-29 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/9364#discussion_r43466845 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala --- @@ -987,4 +992,116 @@ class DataFrameSuite extends QueryTest with

<    5   6   7   8   9   10