[GitHub] spark pull request: [SPARK-12183][ML][MLLIB] Remove mllib tree imp...

2016-03-22 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/11855#discussion_r57077278 --- Diff: mllib/src/test/scala/org/apache/spark/mllib/tree/DecisionTreeSuite.scala --- @@ -563,18 +136,10 @@ class DecisionTreeSuite extends SparkFunSuite

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

2016-03-22 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/11899#discussion_r57077049 --- Diff: core/src/test/java/org/apache/spark/util/collection/unsafe/sort/UnsafeExternalSorterSuite.java --- @@ -60,6 +60,8 @@ final

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

2016-03-22 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/11899#discussion_r57076828 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManagerManagedBuffer.scala --- @@ -29,19 +29,19 @@ import

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

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

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11899#issuecomment-200049631 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-14075] Refactor MemoryStore to be testa...

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

[GitHub] spark pull request: [SPARK-13041][Mesos]add driver sandbox uri to ...

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

[GitHub] spark pull request: [SPARK-13041][Mesos]add driver sandbox uri to ...

2016-03-22 Thread skonto
Github user skonto commented on the pull request: https://github.com/apache/spark/pull/11887#issuecomment-200048764 Jenkins, 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

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

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

[GitHub] spark pull request: [SPARK-13992][WIP] Add support for off-heap ca...

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

[GitHub] spark pull request: [SPARK-14080] [SQL] remove the unnecessary log...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11900#issuecomment-200047099 Can one of the admins verify this patch? --- 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-12183][ML][MLLIB] Remove mllib tree imp...

2016-03-22 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/11855#discussion_r57075903 --- Diff: mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala --- @@ -33,6 +39,406 @@ class RandomForestSuite extends SparkFunSuite

[GitHub] spark pull request: [SPARK-12183][ML][MLLIB] Remove mllib tree imp...

2016-03-22 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/11855#discussion_r57075802 --- Diff: mllib/src/test/scala/org/apache/spark/mllib/tree/DecisionTreeSuite.scala --- @@ -852,10 +399,7 @@ class DecisionTreeSuite extends SparkFunSuite

[GitHub] spark pull request: remove the unnecessary logical operation

2016-03-22 Thread bomeng
GitHub user bomeng opened a pull request: https://github.com/apache/spark/pull/11900 remove the unnecessary logical operation ## What changes were proposed in this pull request? This will remove the unnecessary logical operations in the codegen for Filter. ## How

[GitHub] spark pull request: [SPARK-12183][ML][MLLIB] Remove mllib tree imp...

2016-03-22 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/11855#discussion_r57075598 --- Diff: mllib/src/test/scala/org/apache/spark/mllib/tree/DecisionTreeSuite.scala --- @@ -539,18 +120,10 @@ class DecisionTreeSuite extends SparkFunSuite

[GitHub] spark pull request: [SPARK-12183][ML][MLLIB] Remove mllib tree imp...

2016-03-22 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/11855#discussion_r57075513 --- Diff: mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala --- @@ -33,6 +39,406 @@ class RandomForestSuite extends SparkFunSuite

[GitHub] spark pull request: [SPARK-12183][ML][MLLIB] Remove mllib tree imp...

2016-03-22 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/11855#discussion_r57075419 --- Diff: mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala --- @@ -33,6 +39,406 @@ class RandomForestSuite extends SparkFunSuite

[GitHub] spark pull request: [SPARK-14014] [SQL] Replace existing catalog w...

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

[GitHub] spark pull request: [SPARK-14050][ML] Add multiple languages suppo...

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

[GitHub] spark pull request: [SPARK-13579][build][wip] Stop building the ma...

2016-03-22 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/11796#issuecomment-200042764 jline error looks weirdly similar. Old version of jline (0.9) has class `jline.Terminal`, while newer versions have an interface. ``` 2016-03-22

[GitHub] spark pull request: [SPARK-13068][PYSPARK][ML] Type conversion for...

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

[GitHub] spark pull request: [SPARK-13887][PYTHON][TRIVIAL][BUILD] Make lin...

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

[GitHub] spark pull request: [SPARK-13887][PYTHON][TRIVIAL][BUILD] Make lin...

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

[GitHub] spark pull request: [SPARK-13068][PYSPARK][ML] Type conversion for...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11663#issuecomment-200041499 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

[GitHub] spark pull request: [SPARK-13887][PYTHON][TRIVIAL][BUILD] Make lin...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11898#issuecomment-200041452 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-13068][PYSPARK][ML] Type conversion for...

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

[GitHub] spark pull request: [SPARK-13809][SQL] State store for streaming a...

2016-03-22 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/11645#discussion_r57074381 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/HDFSBackedStateStoreProvider.scala --- @@ -0,0 +1,588 @@ +/* + *

[GitHub] spark pull request: [SPARK-13809][SQL] State store for streaming a...

2016-03-22 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/11645#discussion_r57074371 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/HDFSBackedStateStoreProvider.scala --- @@ -0,0 +1,588 @@ +/* + *

[GitHub] spark pull request: [SPARK-14052] [SQL] build a BytesToBytesMap di...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11870#issuecomment-200040986 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-14052] [SQL] build a BytesToBytesMap di...

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

[GitHub] spark pull request: [SPARK-14072][core] Show JVM/OS version inform...

2016-03-22 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11893#discussion_r57074280 --- Diff: core/src/main/scala/org/apache/spark/util/Benchmark.scala --- @@ -91,16 +92,31 @@ private[spark] object Benchmark { * This should return

[GitHub] spark pull request: [SPARK-14072][core] Show JVM/OS version inform...

2016-03-22 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11893#discussion_r57074170 --- Diff: core/src/main/scala/org/apache/spark/util/Benchmark.scala --- @@ -91,16 +92,31 @@ private[spark] object Benchmark { * This should return

[GitHub] spark pull request: [SPARK-14052] [SQL] build a BytesToBytesMap di...

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

[GitHub] spark pull request: [SPARK-13068][PYSPARK][ML] Type conversion for...

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

[GitHub] spark pull request: [SPARK-14072][core] Show JVM/OS version inform...

2016-03-22 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/11893#discussion_r57073725 --- Diff: core/src/main/scala/org/apache/spark/util/Benchmark.scala --- @@ -91,16 +92,31 @@ private[spark] object Benchmark { * This should return

[GitHub] spark pull request: [SPARK-12183][ML][MLLIB] Remove mllib tree imp...

2016-03-22 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/11855#discussion_r57073751 --- Diff: mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala --- @@ -33,6 +39,406 @@ class RandomForestSuite extends SparkFunSuite

[GitHub] spark pull request: [SPARK-13809][SQL] State store for streaming a...

2016-03-22 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/11645#discussion_r57073603 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/HDFSBackedStateStoreProvider.scala --- @@ -0,0 +1,588 @@ +/* + *

[GitHub] spark pull request: [SPARK-12183][ML][MLLIB] Remove mllib tree imp...

2016-03-22 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/11855#discussion_r57073360 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/tree/RandomForest.scala --- @@ -443,84 +272,4 @@ object RandomForest extends Serializable with

[GitHub] spark pull request: [SPARK-13817][SQL][MINOR] Renames Dataset.newD...

2016-03-22 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/11889#issuecomment-200039053 I think you meant to replace [this comment] with a link to something yes? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-14014] [SQL] Replace existing catalog w...

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

[GitHub] spark pull request: [SPARK-14056] Appends s3 specific configuratio...

2016-03-22 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/11876#discussion_r57072981 --- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala --- @@ -74,13 +74,12 @@ class SparkHadoopUtil extends Logging { }

[GitHub] spark pull request: [SPARK-13992][WIP] Add support for off-heap ca...

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

[GitHub] spark pull request: [SPARK-13992][WIP] Add support for off-heap ca...

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

[GitHub] spark pull request: [SPARK-13992][WIP] Add support for off-heap ca...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11805#issuecomment-200037299 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-14055] writeLocksByTask need to be upda...

2016-03-22 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/11875#issuecomment-200037047 Would it maybe be better to have some text explaining the change than just a JIRA link in the PR description? --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-13992][WIP] Add support for off-heap ca...

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

[GitHub] spark pull request: [SPARK-13713][SQL] Migrate parser from ANTLR3 ...

2016-03-22 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11557#discussion_r57071107 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/ng/AstBuilder.scala --- @@ -0,0 +1,1450 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-13891] [SQL] [TEST] Issue Exceptions wh...

2016-03-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11714#issuecomment-200034759 Can you bring this up to date? We can disable rules that make this fail. @marmbrus and I talked a bit and thought this is pretty critical to have. --- If your

[GitHub] spark pull request: [SPARK-13449] Naive Bayes wrapper in SparkR

2016-03-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11890 --- 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-13449] Naive Bayes wrapper in SparkR

2016-03-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11486 --- 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-13579][build][wip] Stop building the ma...

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

[GitHub] spark pull request: [SPARK-13449] Naive Bayes wrapper in SparkR

2016-03-22 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/11890#issuecomment-200033982 Thanks! Merged into 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

[GitHub] spark pull request: [SPARK-14073][Streaming]Move flume back to Spa...

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

[GitHub] spark pull request: [SPARK-13713][SQL] Migrate parser from ANTLR3 ...

2016-03-22 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11557#discussion_r57070615 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/dsl/package.scala --- @@ -161,6 +161,10 @@ package object dsl { def lower(e:

[GitHub] spark pull request: [SPARK-14073][Streaming]Move flume back to Spa...

2016-03-22 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/11895#issuecomment-200032183 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-14072][core] Show JVM/OS version inform...

2016-03-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11893#issuecomment-200030894 This is a bit ugly. Actually since it is obvious what the cpu/jvm/os are, why don't we just do ``` OpenJDK 64-Bit Server VM

[GitHub] spark pull request: [SPARK-13579][build][wip] Stop building the ma...

2016-03-22 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/11796#issuecomment-200030976 Weird jline error, trying to see if I can figure it out... but the tests I want to fail passed. retest this please. --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-14072][core] Show JVM/OS version inform...

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

[GitHub] spark pull request: [SPARK-14072][core] Show JVM/OS version inform...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11893#issuecomment-200029392 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-13041][Mesos]add driver sandbox uri to ...

2016-03-22 Thread skonto
Github user skonto commented on a diff in the pull request: https://github.com/apache/spark/pull/11887#discussion_r57069417 --- Diff: core/src/main/scala/org/apache/spark/deploy/mesos/ui/MesosClusterPage.scala --- @@ -68,6 +75,25 @@ private[mesos] class MesosClusterPage(parent:

[GitHub] spark pull request: [SPARK-13041][Mesos]add driver sandbox uri to ...

2016-03-22 Thread skonto
Github user skonto commented on a diff in the pull request: https://github.com/apache/spark/pull/11887#discussion_r57069354 --- Diff: core/src/main/scala/org/apache/spark/deploy/mesos/ui/MesosClusterPage.scala --- @@ -68,6 +75,25 @@ private[mesos] class MesosClusterPage(parent:

[GitHub] spark pull request: [SPARK-14072][core] Show JVM/OS version inform...

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

[GitHub] spark pull request: [SPARK-13068][PYSPARK][ML] Type conversion for...

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

[GitHub] spark pull request: [SPARK-13068][PYSPARK][ML] Type conversion for...

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

[GitHub] spark pull request: [SPARK-13068][PYSPARK][ML] Type conversion for...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11663#issuecomment-200027324 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

[GitHub] spark pull request: [SPARK-13041][Mesos]add driver sandbox uri to ...

2016-03-22 Thread skonto
Github user skonto commented on the pull request: https://github.com/apache/spark/pull/11887#issuecomment-200027126 Hive tests fail... --- 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-14075] Refactor MemoryStore to be testa...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11899#issuecomment-200024708 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-13068][PYSPARK][ML] Type conversion for...

2016-03-22 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/11663#discussion_r57068522 --- Diff: python/pyspark/ml/param/__init__.py --- @@ -275,23 +425,9 @@ def _set(self, **kwargs): """ for param, value in

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

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

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

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

[GitHub] spark pull request: [SPARK-13068][PYSPARK][ML] Type conversion for...

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

[GitHub] spark pull request: [SPARK-13068][PYSPARK][ML] Type conversion for...

2016-03-22 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/11663#discussion_r57068396 --- Diff: python/pyspark/ml/param/__init__.py --- @@ -65,6 +75,146 @@ def __eq__(self, other): return False +class

[GitHub] spark pull request: [SPARK-13068][PYSPARK][ML] Type conversion for...

2016-03-22 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/11663#discussion_r57068235 --- Diff: python/pyspark/ml/param/__init__.py --- @@ -65,6 +75,146 @@ def __eq__(self, other): return False +class

[GitHub] spark pull request: [SPARK-13068][PYSPARK][ML] Type conversion for...

2016-03-22 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/11663#discussion_r57068173 --- Diff: python/pyspark/ml/param/__init__.py --- @@ -17,28 +17,38 @@ from abc import ABCMeta import copy +import numpy as np +import

[GitHub] spark pull request: [SPARK-13449] Naive Bayes wrapper in SparkR

2016-03-22 Thread thunterdb
Github user thunterdb commented on the pull request: https://github.com/apache/spark/pull/11890#issuecomment-200023253 @mengxr it looks great, I have no comment --- 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-14070] [SQL] Use ORC data source for SQ...

2016-03-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11891#issuecomment-200022810 This is pretty cool -- the hive path is ridiculously slow. BTW I tried comparing Parquet vs ORC based on Spark master branch right now. I generated 100 million

[GitHub] spark pull request: [SPARK-13579][build][wip] Stop building the ma...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11796#issuecomment-200022624 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-13579][build][wip] Stop building the ma...

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

[GitHub] spark pull request: [SPARK-13579][build][wip] Stop building the ma...

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

[GitHub] spark pull request: [SPARK-13809][SQL] State store for streaming a...

2016-03-22 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/11645#discussion_r57067817 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStoreRDD.scala --- @@ -0,0 +1,70 @@ +/* + * Licensed to the

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

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

[GitHub] spark pull request: [SPARK-13887][PYTHON][TRIVIAL][BUILD] Make lin...

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

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

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

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

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

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11899#issuecomment-200021005 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-14075] Refactor MemoryStore to be testa...

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

[GitHub] spark pull request: [SPARK-13887][PYTHON][TRIVIAL][BUILD] Make lin...

2016-03-22 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/11898#issuecomment-200020557 Jenkins 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

[GitHub] spark pull request: [SPARK-13809][SQL] State store for streaming a...

2016-03-22 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/11645#discussion_r57066767 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStoreCoordinator.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

2016-03-22 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/11899#issuecomment-200020020 /cc @rxin @andrewor14 @sameeragarwal @nongli , this is blocking the new tests in https://github.com/apache/spark/pull/11791 --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-14075] Refactor MemoryStore to be testa...

2016-03-22 Thread JoshRosen
GitHub user JoshRosen opened a pull request: https://github.com/apache/spark/pull/11899 [SPARK-14075] Refactor MemoryStore to be testable independent of BlockManager This patch refactors the `MemoryStore` so that it can be tested without needing to construct / mock an entire

[GitHub] spark pull request: [SPARK-14060][SQL] Move StringToColumn implici...

2016-03-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11878 --- 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-13809][SQL] State store for streaming a...

2016-03-22 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/11645#discussion_r57066269 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/state/StateStoreCoordinator.scala --- @@ -0,0 +1,140 @@ +/* + * Licensed

[GitHub] spark pull request: [SPARK-13887][PYTHON][TRIVIAL][BUILD] Make lin...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11898#issuecomment-200018269 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-13887][PYTHON][TRIVIAL][BUILD] Make lin...

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

[GitHub] spark pull request: [SPARK-13887][PYTHON][TRIVIAL][BUILD] Make lin...

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

[GitHub] spark pull request: [SPARK-14060][SQL] Move StringToColumn implici...

2016-03-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11878#issuecomment-200017870 Thanks @cloud-fan. Merging in 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

[GitHub] spark pull request: [SPARK-13809][SQL] State store for streaming a...

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

[GitHub] spark pull request: [SPARK-13809][SQL] State store for streaming a...

2016-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11645#issuecomment-200017604 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-13981][SQL] Defer evaluating variables ...

2016-03-22 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/11792#discussion_r57065601 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala --- @@ -110,23 +114,28 @@ case class Filter(condition: Expression,

[GitHub] spark pull request: [SPARK-13809][SQL] State store for streaming a...

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

<    1   2   3   4   5   6   7   8   9   10   >