[GitHub] spark pull request: [SPARK-11714][Mesos] Make Spark on Mesos honor...

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

[GitHub] spark pull request: [SPARK-11714][Mesos] Make Spark on Mesos honor...

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

[GitHub] spark pull request: [SPARK-11714][Mesos] Make Spark on Mesos honor...

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

[GitHub] spark pull request: [SPARK-11714][Mesos] Make Spark on Mesos honor...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11157#issuecomment-195631588 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-11714][Mesos] Make Spark on Mesos honor...

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

[GitHub] spark pull request: [SPARK-11714][Mesos] Make Spark on Mesos honor...

2016-03-11 Thread skonto
Github user skonto commented on the pull request: https://github.com/apache/spark/pull/11157#issuecomment-195631531 updated the PR @tnachen @mgummelt, ready for review --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-13777] [ML] Remove constant features fr...

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

[GitHub] spark pull request: [SPARK-11714][Mesos] Make Spark on Mesos honor...

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

[GitHub] spark pull request: [SPARK-13139][SQL] Follow-ups to #11573

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11667#issuecomment-195630704 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-13671] [SPARK-13311] [SQL] Use differen...

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

[GitHub] spark pull request: [SPARK-13671] [SPARK-13311] [SQL] Use differen...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11514#issuecomment-195630758 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-13139][SQL] Follow-ups to #11573

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

[GitHub] spark pull request: [SPARK-13671] [SPARK-13311] [SQL] Use differen...

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

[GitHub] spark pull request: [SPARK-13139][SQL] Follow-ups to #11573

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

[GitHub] spark pull request: [SPARK-13658][SQL] BooleanSimplification rule ...

2016-03-11 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/11647#discussion_r55912815 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Canonicalize.scala --- @@ -36,15 +36,16 @@ import

[GitHub] spark pull request: [SPARK-13810] [CORE] Add Port Configuration Su...

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

[GitHub] spark pull request: [SPARK-13834] Update sbt and MiMa plugin for 2...

2016-03-11 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11669#issuecomment-195626487 Hi, @JoshRosen . Now, this PR includes SBT/MiMa updates for SPARK-13834. Build and MiMa is tested on both JDK8 and JDK7. I hope it will pass at this

[GitHub] spark pull request: [SPARK-13834] Update sbt and MiMa plugin for 2...

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

[GitHub] spark pull request: [SPARK-13834] Update sbt for 2.x.

2016-03-11 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11661#issuecomment-195625234 Oh, @JoshRosen . I'll merge both PR. It passed now based on #11669 . Sorry for making you confused. --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-XXXX][SQL] Support for inferring filter...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11665#issuecomment-195625087 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-XXXX][SQL] Support for inferring filter...

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

[GitHub] spark pull request: [SPARK-XXXX][SQL] Support for inferring filter...

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

[GitHub] spark pull request: [WIP]Remove streaming-flume, streaming-mqtt, s...

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

[GitHub] spark pull request: [WIP]Remove streaming-flume, streaming-mqtt, s...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11672#issuecomment-195624381 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: [WIP]Remove streaming-flume, streaming-mqtt, s...

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

[GitHub] spark pull request: [WIP]Remove streaming-flume, streaming-mqtt, s...

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

[GitHub] spark pull request: [minor][build][test-maven] Remove invocation o...

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

[GitHub] spark pull request: [minor][build][test-maven] Remove invocation o...

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

[GitHub] spark pull request: [minor][build][test-maven] Remove invocation o...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11671#issuecomment-195624114 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: [WIP]Remove streaming-flume, streaming-mqtt, s...

2016-03-11 Thread zsxwing
GitHub user zsxwing opened a pull request: https://github.com/apache/spark/pull/11672 [WIP]Remove streaming-flume, streaming-mqtt, streaming-zeromq, streaming-akka, streaming-twitter to Spark packages ## What changes were proposed in this pull request? Just test if can

[GitHub] spark pull request: [SPARK-13810] [CORE] Add Port Configuration Su...

2016-03-11 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/11644#discussion_r55911638 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -2014,8 +2014,10 @@ private[spark] object Utils extends Logging { } catch {

[GitHub] spark pull request: [SPARK-10907][SPARK-6157] Remove pendingUnroll...

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

[GitHub] spark pull request: [minor][build][test-maven] Remove invocation o...

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

[GitHub] spark pull request: [SPARK-10907][SPARK-6157] Remove pendingUnroll...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11613#issuecomment-195623515 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: [minor][build][test-maven] Remove invocation o...

2016-03-11 Thread vanzin
GitHub user vanzin opened a pull request: https://github.com/apache/spark/pull/11671 [minor][build][test-maven] Remove invocation of dependency plugin. Removing the execution id prevents the goal from running unless it's explicitly invoked from the command line. You can merge

[GitHub] spark pull request: [SPARK-10907][SPARK-6157] Remove pendingUnroll...

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

[GitHub] spark pull request: Branch 1.6

2016-03-11 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/11668#issuecomment-195622769 Could you close this PR 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-13810] [CORE] Add Port Configuration Su...

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

[GitHub] spark pull request: [SPARK-XX] [SQL] avoid the copy in HashedRelat...

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

[GitHub] spark pull request: [SPARK-XX] [SQL] avoid the copy in HashedRelat...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11666#issuecomment-195621420 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-XX] [SQL] avoid the copy in HashedRelat...

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

[GitHub] spark pull request: [SPARK-13810] [CORE] Add Port Configuration Su...

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

[GitHub] spark pull request: [SPARK-13810] [CORE] Add Port Configuration Su...

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

[GitHub] spark pull request: [SPARK-13810] [CORE] Add Port Configuration Su...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11644#issuecomment-195620582 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-XXXX][SQL] Support for inferring filter...

2016-03-11 Thread sameeragarwal
Github user sameeragarwal commented on the pull request: https://github.com/apache/spark/pull/11665#issuecomment-195620519 cc @nongli @yhuai @gatorsmile --- 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-13834] Update sbt for 2.x.

2016-03-11 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11661#issuecomment-195620503 Hi, @JoshRosen . I made a PR for MiMa first: #11669 . --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-XXX][BUILD] Update MiMa plugin to 0.1.9...

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

[GitHub] spark pull request: [SPARK-XXXX][SQL] Remove redundant conditions ...

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

[GitHub] spark pull request: [SPARK-13810] [CORE] Add Port Configuration Su...

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

[GitHub] spark pull request: Branch 1.6

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11668#issuecomment-195620383 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-XXXX][SQL] Remove redundant conditions ...

2016-03-11 Thread sameeragarwal
Github user sameeragarwal commented on the pull request: https://github.com/apache/spark/pull/11670#issuecomment-195620188 cc @yhuai --- 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-XXXX][SQL] Remove redundant conditions ...

2016-03-11 Thread sameeragarwal
GitHub user sameeragarwal opened a pull request: https://github.com/apache/spark/pull/11670 [SPARK-][SQL] Remove redundant conditions while combining filters ## What changes were proposed in this pull request? **[I'll link it to the JIRA once ASF JIRA is back online]**

[GitHub] spark pull request: [SPARK-XXX][BUILD] Update MiMa plugin to 0.1.9...

2016-03-11 Thread dongjoon-hyun
GitHub user dongjoon-hyun opened a pull request: https://github.com/apache/spark/pull/11669 [SPARK-XXX][BUILD] Update MiMa plugin to 0.1.9. ## What changes were proposed in this pull request? NOTE: Apache JIRA is down now. I'll update the title as soon as possible.

[GitHub] spark pull request: Branch 1.6

2016-03-11 Thread pradeepsavadi
GitHub user pradeepsavadi opened a pull request: https://github.com/apache/spark/pull/11668 Branch 1.6 ## What changes were proposed in this pull request? (Please fill in changes proposed in this fix) ## How was this patch tested? (Please explain how

[GitHub] spark pull request: [SPARK-10907][SPARK-6157] Remove pendingUnroll...

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

[GitHub] spark pull request: [SPARK-13833] Guard against race condition whe...

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

[GitHub] spark pull request: [SPARK-13833] Guard against race condition whe...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11660#issuecomment-195619419 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-13833] Guard against race condition whe...

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

[GitHub] spark pull request: [SPARK-13834] Update sbt for 2.x.

2016-03-11 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11661#issuecomment-195617712 Really, it's interesting issue. --- 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-13834] Update sbt for 2.x.

2016-03-11 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11661#issuecomment-195617676 Thank you, @JoshRosen . I tried both together, too. But, it still has a problem. By the way, during that procedure, I found that new MiMa plugin reports

[GitHub] spark pull request: [SPARK-13834] Update sbt for 2.x.

2016-03-11 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/11661#issuecomment-195616909 Hmm, interesting. Note that there's now an updated version of MiMa, so we might also consider bumping that (and other SBT plugins) as well. --- If your project is

[GitHub] spark pull request: [SPARK-13777] [ML] Remove constant features fr...

2016-03-11 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/11610#discussion_r55909205 --- Diff: mllib/src/main/scala/org/apache/spark/ml/optim/WeightedLeastSquares.scala --- @@ -108,6 +101,53 @@ private[ml] class WeightedLeastSquares(

[GitHub] spark pull request: [SPARK-10907][SPARK-6157] Remove pendingUnroll...

2016-03-11 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/11613#issuecomment-195616282 I ended up having to roll back that size estimation improvement because it broke some tests which relied on the old behavior. I'd still like to make that change but

[GitHub] spark pull request: [SPARK-13777] [ML] Remove constant features fr...

2016-03-11 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/11610#discussion_r55909088 --- Diff: mllib/src/main/scala/org/apache/spark/ml/optim/WeightedLeastSquares.scala --- @@ -108,6 +101,53 @@ private[ml] class WeightedLeastSquares(

[GitHub] spark pull request: [SPARK-10907][SPARK-6157] Remove pendingUnroll...

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

[GitHub] spark pull request: [SPARK-10907][SPARK-6157] Remove pendingUnroll...

2016-03-11 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/11613#discussion_r55908919 --- Diff: core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala --- @@ -100,48 +92,136 @@ private[spark] class MemoryStore( */

[GitHub] spark pull request: [SPARK-13811][SPARK-13836] [SQL] Removed IsNot...

2016-03-11 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/11649#issuecomment-195615310 I guess I would probably propagate them during planning (i.e. when you are creating the physical `Filter` operator) since its not really an optimization to add them

[GitHub] spark pull request: [SPARK-13777] [ML] Remove constant features fr...

2016-03-11 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/11610#discussion_r55908792 --- Diff: mllib/src/main/scala/org/apache/spark/ml/optim/WeightedLeastSquares.scala --- @@ -120,34 +160,47 @@ private[ml] class WeightedLeastSquares(

[GitHub] spark pull request: [SPARK-13396] Stop using our internal deprecat...

2016-03-11 Thread jodersky
Github user jodersky commented on a diff in the pull request: https://github.com/apache/spark/pull/11544#discussion_r55908688 --- Diff: core/src/main/scala/org/apache/spark/ui/jobs/JobProgressListener.scala --- @@ -374,28 +374,30 @@ class JobProgressListener(conf: SparkConf)

[GitHub] spark pull request: [SPARK-13811][SPARK-13836] [SQL] Removed IsNot...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11649#issuecomment-195614379 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-13811][SPARK-13836] [SQL] Removed IsNot...

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

[GitHub] spark pull request: [SPARK-13811][SPARK-13836] [SQL] Removed IsNot...

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

[GitHub] spark pull request: [SPARK-13810] [CORE] Add Port Configuration Su...

2016-03-11 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/11644#discussion_r55908574 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -2014,8 +2014,10 @@ private[spark] object Utils extends Logging { } catch {

[GitHub] spark pull request: [SPARK-XX] [SQL] fast serialization for collec...

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

[GitHub] spark pull request: [SPARK-XX] [SQL] fast serialization for collec...

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

[GitHub] spark pull request: [SPARK-XX] [SQL] fast serialization for collec...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11664#issuecomment-195614080 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-13777] [ML] Remove constant features fr...

2016-03-11 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/11610#discussion_r55908431 --- Diff: mllib/src/main/scala/org/apache/spark/ml/optim/WeightedLeastSquares.scala --- @@ -108,6 +101,53 @@ private[ml] class WeightedLeastSquares(

[GitHub] spark pull request: [SPARK-13777] [ML] Remove constant features fr...

2016-03-11 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/11610#discussion_r55908419 --- Diff: mllib/src/main/scala/org/apache/spark/ml/optim/WeightedLeastSquares.scala --- @@ -108,6 +101,53 @@ private[ml] class WeightedLeastSquares(

[GitHub] spark pull request: [SPARK-13810] [CORE] Add Port Configuration Su...

2016-03-11 Thread bjornjon
Github user bjornjon commented on a diff in the pull request: https://github.com/apache/spark/pull/11644#discussion_r55908378 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -2014,8 +2014,10 @@ private[spark] object Utils extends Logging { } catch

[GitHub] spark pull request: [SPARK-13777] [ML] Remove constant features fr...

2016-03-11 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/11610#discussion_r55908346 --- Diff: mllib/src/main/scala/org/apache/spark/ml/optim/WeightedLeastSquares.scala --- @@ -108,6 +101,53 @@ private[ml] class WeightedLeastSquares(

[GitHub] spark pull request: [SPARK-13823] [CORE] [STREAMING] [SQL] Always ...

2016-03-11 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/11657#issuecomment-195613637 LGTM. The original idea was to import `c.g.c.b.Charsets.UTF_8` so that later you could just change the import to `java.nio.charset.StandardCharsets.UTF_8` without

[GitHub] spark pull request: [SPARK-13396] Stop using our internal deprecat...

2016-03-11 Thread GayathriMurali
Github user GayathriMurali commented on a diff in the pull request: https://github.com/apache/spark/pull/11544#discussion_r55908239 --- Diff: core/src/main/scala/org/apache/spark/ui/jobs/JobProgressListener.scala --- @@ -374,28 +374,30 @@ class JobProgressListener(conf: SparkConf)

[GitHub] spark pull request: [SPARK-13777] [ML] Remove constant features fr...

2016-03-11 Thread sethah
Github user sethah commented on a diff in the pull request: https://github.com/apache/spark/pull/11610#discussion_r55908232 --- Diff: mllib/src/main/scala/org/apache/spark/ml/optim/WeightedLeastSquares.scala --- @@ -108,6 +101,53 @@ private[ml] class WeightedLeastSquares(

[GitHub] spark pull request: SPARK-13779: Avoid cancelling non-local contai...

2016-03-11 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/11612#issuecomment-195612369 LGTM. --- 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

[GitHub] spark pull request: [SPARK-13626] [core] Avoid duplicate config de...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11510#issuecomment-195612256 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-13578] [core] Modify launch scripts to ...

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

[GitHub] spark pull request: [SPARK-13626] [core] Avoid duplicate config de...

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

[GitHub] spark pull request: SPARK-13779: Avoid cancelling non-local contai...

2016-03-11 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/11612#discussion_r55907969 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala --- @@ -265,25 +265,52 @@ private[yarn] class YarnAllocator( // For

[GitHub] spark pull request: [SPARK-13626] [core] Avoid duplicate config de...

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

[GitHub] spark pull request: [SPARK-13746][Tests]stop using deprecated Sync...

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

[GitHub] spark pull request: [SPARK-13576] [build] [test-maven] Don't creat...

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

[GitHub] spark pull request: [SPARK-13811][SPARK-13836] [SQL] Removed IsNot...

2016-03-11 Thread sameeragarwal
Github user sameeragarwal commented on the pull request: https://github.com/apache/spark/pull/11649#issuecomment-195611317 @marmbrus the reason for adding these inferred filters in the logical plan is because we currently only propagate constraints in the logical plan (and they are

[GitHub] spark pull request: [SPARK-13746][Tests]stop using deprecated Sync...

2016-03-11 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/11580#issuecomment-195611297 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 have this

[GitHub] spark pull request: [SPARK-13139][SQL] Follow-ups to #11573

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

[GitHub] spark pull request: [SPARK-13576] [build] [test-maven] Don't creat...

2016-03-11 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/11452#issuecomment-195611100 SQL test failure, I assume unrelated? 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.

[GitHub] spark pull request: [SPARK-13576] [build] [test-maven] Don't creat...

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

[GitHub] spark pull request: [SPARK-13576] [build] [test-maven] Don't creat...

2016-03-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11452#issuecomment-195610678 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-13576] [build] [test-maven] Don't creat...

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

[GitHub] spark pull request: [SPARK-13834] Update sbt for 2.x.

2016-03-11 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/11661#issuecomment-195610240 Actually, it seems that MIMA test passes on JDK8 and fails on JDK7. I didn't check that before PR. I'm investigating the reason now. Although I closed

[GitHub] spark pull request: [SPARK-13139][SQL] Follow-ups to #11573

2016-03-11 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/11667#issuecomment-195609699 @yhuai --- 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

<    1   2   3   4   5   6   7   >