[GitHub] spark pull request: [SPARK-13257] [Improvement] Refine naive Bayes...

2016-02-22 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/11125#issuecomment-187591744 LGTM. Merged into master. Thanks! --- 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-13011] K-means wrapper in SparkR

2016-02-22 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/11124#issuecomment-187589974 @yinxusen Could you resolve conflicts? --- 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

[GitHub] spark pull request: [SPARK-13220][Core]deprecate yarn-client and y...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11229#issuecomment-187589887 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-13220][Core]deprecate yarn-client and y...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11229#issuecomment-187589891 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13355] [MLLIB] replace GraphImpl.fromEx...

2016-02-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11226 --- 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-13355] [MLLIB] replace GraphImpl.fromEx...

2016-02-22 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/11226#issuecomment-187589207 I've merged this into master, 1.6, 1.5, and 1.4. --- 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-13220][Core]deprecate yarn-client and y...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11229#issuecomment-187589261 **[Test build #51736 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51736/consoleFull)** for PR 11229 at commit

[GitHub] spark pull request: [SPARK-13037][ML][PySpark] PySpark ml.recommen...

2016-02-22 Thread vectorijk
Github user vectorijk commented on the pull request: https://github.com/apache/spark/pull/11044#issuecomment-187588613 _off-topic_ Hi Spark Devs! I was wondering if Spark community would be interested in mentoring students for Google Summer of Code(GSoC) under Apache

[GitHub] spark pull request: [SPARK-13385][MLlib] Enable AssociationRules t...

2016-02-22 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/11263#issuecomment-187587555 @zhengruifeng Please read https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark and follow the Spark code style defined there. There are some

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10705#issuecomment-187587361 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10705#issuecomment-187587358 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-8426] [scheduler] enhance blacklist mec...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8760#issuecomment-187586804 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [Spark-8426] [scheduler] enhance blacklist mec...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8760#issuecomment-187586803 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-8426] [scheduler] enhance blacklist mec...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8760#issuecomment-187586614 **[Test build #51737 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51737/consoleFull)** for PR 8760 at commit

[GitHub] spark pull request: [SPARK-12363][Mllib][Backport-1.4] Remove setR...

2016-02-22 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/11264#issuecomment-187586658 LGTM. Merged into branch-1.4. Could you please close this PR manually? Thanks! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-13266][PYSPARK] None read/writer option...

2016-02-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11305#issuecomment-187586646 OK maybe somebody else can take over with test coverage. Any takers? cc @viirya @gatorsmile --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187586197 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187586195 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-12757] Add block-level read/write locks...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10705#issuecomment-187586080 **[Test build #51748 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51748/consoleFull)** for PR 10705 at commit

[GitHub] spark pull request: [SPARK-12363][Mllib][Backport-1.3] Remove setR...

2016-02-22 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/11265#issuecomment-187586110 test 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-12746][ML] ArrayType(_, true) should al...

2016-02-22 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/11237#issuecomment-187585831 Merged into branch-1.6. Thanks! --- 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-13429] [MLlib] Unify Logistic Regressio...

2016-02-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11299 --- 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-7729][UI]Executor which has been killed...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10058#issuecomment-187585346 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-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187585367 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187585365 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-7729][UI]Executor which has been killed...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10058#issuecomment-187585349 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13429] [MLlib] Unify Logistic Regressio...

2016-02-22 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/11299#issuecomment-187585290 LGTM. Merged into master. Thanks! Btw, I created SPARK-13448 to track behavior changes in Spark 2.0. If you know other changes, please comment there. --- If your

[GitHub] spark pull request: [SPARK-7729][UI]Executor which has been killed...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10058#issuecomment-187584849 **[Test build #51734 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51734/consoleFull)** for PR 10058 at commit

[GitHub] spark pull request: [SPARK-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187583899 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-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187583900 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13220][Core]deprecate yarn-client and y...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11229#issuecomment-187583906 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13220][Core]deprecate yarn-client and y...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11229#issuecomment-187583905 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-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187583894 **[Test build #51743 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51743/consoleFull)** for PR 11260 at commit

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744637 --- Diff: core/src/main/scala/org/apache/spark/network/BlockDataManager.scala --- @@ -31,6 +31,14 @@ trait BlockDataManager { /** *

[GitHub] spark pull request: [SPARK-13220][Core]deprecate yarn-client and y...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11229#issuecomment-187583445 **[Test build #51733 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51733/consoleFull)** for PR 11229 at commit

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744607 --- Diff: core/src/main/scala/org/apache/spark/storage/MemoryStore.scala --- @@ -418,9 +423,11 @@ private[spark] class MemoryStore(blockManager:

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744601 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManagerManagedBuffer.scala --- @@ -0,0 +1,41 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744589 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala --- @@ -521,16 +509,17 @@ private[spark] class BlockManager(

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744546 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala --- @@ -318,7 +316,11 @@ private[spark] class BlockManager( * may not

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/6042 --- 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-13374][Streaming][wip] make it possible...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11249#issuecomment-187582252 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-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744476 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,357 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/6042#issuecomment-187582514 Merged into master. @thunterdb Please send another PR to address my comments. Thanks! --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744461 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [Spark-13374][Streaming][wip] make it possible...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11249#issuecomment-187582244 **[Test build #51746 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51746/consoleFull)** for PR 11249 at commit

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744479 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [Spark-13374][Streaming][wip] make it possible...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11249#issuecomment-187582255 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/6042#discussion_r53744424 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameStatSuite.scala --- @@ -269,3 +291,41 @@ class DataFrameStatSuite extends QueryTest with

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/6042#discussion_r53744427 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/stat/ApproxQuantileSuite.scala --- @@ -0,0 +1,129 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744409 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744430 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744423 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744386 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [Spark-13374][Streaming][wip] make it possible...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11249#issuecomment-187581494 **[Test build #51746 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51746/consoleFull)** for PR 11249 at commit

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744323 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744364 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744366 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744340 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744314 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,335 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744333 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/6042#discussion_r53744297 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/stat/StatFunctions.scala --- @@ -27,6 +30,312 @@ import

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53744319 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187579855 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-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/6042#issuecomment-187579843 @mengxr ok. thank you. --- 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-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187579858 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/6042#discussion_r53744061 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/stat/StatFunctions.scala --- @@ -27,6 +30,312 @@ import

[GitHub] spark pull request: [SPARK-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187579379 **[Test build #51738 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51738/consoleFull)** for PR 11260 at commit

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/6042#discussion_r53743924 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/stat/StatFunctions.scala --- @@ -27,6 +30,312 @@ import

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53743929 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/6042#discussion_r53743891 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/stat/StatFunctions.scala --- @@ -27,6 +30,312 @@ import

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/6042#discussion_r53743882 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameStatFunctions.scala --- @@ -37,6 +37,16 @@ import org.apache.spark.util.sketch.{BloomFilter,

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/6042#discussion_r53743888 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameStatFunctions.scala --- @@ -37,6 +37,16 @@ import org.apache.spark.util.sketch.{BloomFilter,

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/6042#discussion_r53743898 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/stat/StatFunctions.scala --- @@ -27,6 +30,312 @@ import

[GitHub] spark pull request: [SPARK-6761][SQL] Approximate quantile for Dat...

2016-02-22 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/6042#issuecomment-187578756 @viirya I had an offline discussion with @thunterdb today. I'm going to add some comments inline and then merge this PR. @thunterdb will send another PR to address my

[GitHub] spark pull request: [SPARK-12757] Add block-level read/write locks...

2016-02-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10705#discussion_r53743845 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockInfoManager.scala --- @@ -0,0 +1,356 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-13236] SQL Generation for Set Operation...

2016-02-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11195 --- 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-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187578291 **[Test build #51743 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51743/consoleFull)** for PR 11260 at commit

[GitHub] spark pull request: [SPARK-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187577629 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/11260#discussion_r53743303 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala --- @@ -137,6 +139,22 @@ class JoinSuite extends QueryTest with SharedSQLContext {

[GitHub] spark pull request: [SPARK-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187577622 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-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11260#discussion_r53743187 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala --- @@ -81,11 +81,13 @@ private[sql] abstract class SparkStrategies

[GitHub] spark pull request: [SPARK-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11260#discussion_r53743134 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala --- @@ -137,6 +139,22 @@ class JoinSuite extends QueryTest with SharedSQLContext {

[GitHub] spark pull request: [SPARK-13437][SQL] Introduce InternalColumn an...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11307#issuecomment-187575800 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13437][SQL] Introduce InternalColumn an...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11307#issuecomment-187575797 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-13437][SQL] Introduce InternalColumn an...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11307#issuecomment-187575556 **[Test build #51735 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51735/consoleFull)** for PR 11307 at commit

[GitHub] spark pull request: [SPARK-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187574944 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-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187574948 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13383][SQL] Keep broadcast hint after c...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11260#issuecomment-187574635 **[Test build #51731 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51731/consoleFull)** for PR 11260 at commit

[GitHub] spark pull request: [SPARK-13236] SQL Generation for Set Operation...

2016-02-22 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/11195#issuecomment-187574622 LGTM, merging to master. Thanks! --- 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-13374][Streaming][wip] make it possible...

2016-02-22 Thread mwws
Github user mwws commented on a diff in the pull request: https://github.com/apache/spark/pull/11249#discussion_r53742303 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/Checkpoint.scala --- @@ -41,6 +41,13 @@ class Checkpoint(ssc: StreamingContext, val

[GitHub] spark pull request: [SPARK-13440][SQL] ObjectType should accept an...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11316#issuecomment-187573679 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12941][SQL][MASTER] Spark-SQL JDBC Orac...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11306#issuecomment-187573369 **[Test build #51741 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51741/consoleFull)** for PR 11306 at commit

[GitHub] spark pull request: [SPARK-13440][SQL] ObjectType should accept an...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11316#issuecomment-187573676 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-13440][SQL] ObjectType should accept an...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11316#issuecomment-187573144 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-13440][SQL] ObjectType should accept an...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11316#issuecomment-187573148 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13440][SQL] ObjectType should accept an...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11316#issuecomment-187573032 **[Test build #51732 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51732/consoleFull)** for PR 11316 at commit

[GitHub] spark pull request: [SPARK-13440][SQL] ObjectType should accept an...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11316#issuecomment-187572881 **[Test build #51730 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51730/consoleFull)** for PR 11316 at commit

[GitHub] spark pull request: [SPARK-12583][Mesos] Mesos shuffle service: Do...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11272#issuecomment-187570175 **[Test build #51739 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51739/consoleFull)** for PR 11272 at commit

[GitHub] spark pull request: [SPARK-13435] [MLlib] Add Weighted Cohen's kap...

2016-02-22 Thread zhengruifeng
Github user zhengruifeng commented on a diff in the pull request: https://github.com/apache/spark/pull/11303#discussion_r53741198 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/evaluation/MulticlassMetrics.scala --- @@ -25,18 +25,19 @@ import org.apache.spark.rdd.RDD

[GitHub] spark pull request: [Spark-13374][Streaming][wip] make it possible...

2016-02-22 Thread mwws
Github user mwws commented on a diff in the pull request: https://github.com/apache/spark/pull/11249#discussion_r53741073 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/Checkpoint.scala --- @@ -41,6 +41,13 @@ class Checkpoint(ssc: StreamingContext, val

  1   2   3   4   5   6   7   8   >