[GitHub] spark pull request: [SPARK-5259][CORE]Make sure shuffle metadata a...

2015-07-03 Thread suyanNone
Github user suyanNone commented on a diff in the pull request: https://github.com/apache/spark/pull/4055#discussion_r33860928 --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala --- @@ -1193,8 +1193,10 @@ class DAGScheduler( // TODO: This will

[GitHub] spark pull request: [SPARK-6707] [CORE][MESOS]: Mesos Scheduler sh...

2015-07-03 Thread dragos
Github user dragos commented on the pull request: https://github.com/apache/spark/pull/5563#issuecomment-118372249 To me this looks good, I just didn't have the time to run it on our Mesos cluster again. I'll try to do so ASAP, but in the meantime maybe @tnachen or @deanwampler give

[GitHub] spark pull request: [SPARK-8796][SQL] mark child as transient in I...

2015-07-03 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/7192#discussion_r33860021 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala --- @@ -43,8 +42,8 @@ import

[GitHub] spark pull request: [SPARK-7785] [MLlib] [PySpark] Add __str__ and...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6342#issuecomment-118338986 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-7871][SQL]Improve the outputPartitionin...

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

[GitHub] spark pull request: [SPARK-8759][SQL] add default eval to binary a...

2015-07-03 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/7157#discussion_r33864982 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala --- @@ -167,6 +167,25 @@ abstract class BinaryExpression

[GitHub] spark pull request: [SPARK-8238][SPARK-8239][SPARK-8242][SPARK-824...

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

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

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

[GitHub] spark pull request: [SPARK-8796][SQL] make sure SparkPlan is only ...

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

[GitHub] spark pull request: [SPARK-5259][CORE]Make sure shuffle metadata a...

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4055#issuecomment-118371224 [Test build #36498 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36498/console) for PR 4055 at commit

[GitHub] spark pull request: [SPARK-5259][CORE]Make sure shuffle metadata a...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4055#issuecomment-118371294 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-8800][SQL] Fix inaccurate precision/sca...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7212#issuecomment-118371416 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8238][SPARK-8239][SPARK-8242][SPARK-824...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6843#issuecomment-118372945 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-8759][SQL] add default eval to binary a...

2015-07-03 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/7157#discussion_r33860587 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala --- @@ -167,6 +167,25 @@ abstract class

[GitHub] spark pull request: [SPARK-8238][SPARK-8239][SPARK-8242][SPARK-824...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6843#issuecomment-118355955 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-6602][Core]Replace Akka Serialization w...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7159#issuecomment-118361204 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-118364368 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-7871][SQL]Improve the outputPartitionin...

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6413#issuecomment-118368276 [Test build #36499 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36499/console) for PR 6413 at commit

[GitHub] spark pull request: [SPARK-7871][SQL]Improve the outputPartitionin...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6413#issuecomment-118368479 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-8800][SQL] Fix inaccurate precision/sca...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7212#issuecomment-118371432 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-5259][CORE]Make sure shuffle metadata a...

2015-07-03 Thread suyanNone
Github user suyanNone commented on a diff in the pull request: https://github.com/apache/spark/pull/4055#discussion_r33861063 --- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala --- @@ -305,7 +305,7 @@ private[spark] class MapOutputTrackerMaster(conf: SparkConf)

[GitHub] spark pull request: [SPARK-5259][CORE]Make sure shuffle metadata a...

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

[GitHub] spark pull request: [SPARK-8800][SQL] Fix inaccurate precision/sca...

2015-07-03 Thread viirya
GitHub user viirya opened a pull request: https://github.com/apache/spark/pull/7212 [SPARK-8800][SQL] Fix inaccurate precision/scale of Decimal division operation JIRA: https://issues.apache.org/jira/browse/SPARK-8800 Previously, we turn to Java BigDecimal's divide with

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-118370909 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-8238][SPARK-8239][SPARK-8242][SPARK-824...

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6843#issuecomment-118372243 [Test build #36500 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36500/console) for PR 6843 at commit

[GitHub] spark pull request: [SPARK-8238][SPARK-8239][SPARK-8242][SPARK-824...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6843#issuecomment-118372312 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-8238][SPARK-8239][SPARK-8242][SPARK-824...

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6843#issuecomment-118372843 [Test build #36501 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36501/console) for PR 6843 at commit

[GitHub] spark pull request: [SPARK-6287][MESOS] Add dynamic allocation to ...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4984#issuecomment-118338860 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-6287][MESOS] Add dynamic allocation to ...

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4984#issuecomment-118338785 [Test build #36496 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36496/console) for PR 4984 at commit

[GitHub] spark pull request: [SPARK-7785] [MLlib] [PySpark] Add __str__ and...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6342#issuecomment-118345320 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-8238][SPARK-8239][SPARK-8242][SPARK-824...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6843#issuecomment-118356600 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8238][SPARK-8239][SPARK-8242][SPARK-824...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6843#issuecomment-118356616 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8765] [MLlib] Fix PySpark PowerIteratio...

2015-07-03 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/7177#issuecomment-118370370 @mengxr @jkbradley Yes, the cluster assignments is deterministic subject to numerical difference. The current tests is deterministic right now just like the test

[GitHub] spark pull request: [SPARK-8796][SQL] make sure SparkPlan is only ...

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7192#issuecomment-118373279 [Test build #36504 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36504/console) for PR 7192 at commit

[GitHub] spark pull request: [SPARK-8796][SQL] make sure SparkPlan is only ...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7192#issuecomment-118373292 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-7785] [MLlib] [PySpark] Add __str__ and...

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6342#issuecomment-118345239 [Test build #36497 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36497/console) for PR 6342 at commit

[GitHub] spark pull request: [SPARK-8572] [SQL] Type coercion for ScalaUDFs

2015-07-03 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/7203#issuecomment-118356340 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

[GitHub] spark pull request: [SPARK-8759][SQL] add default eval to binary a...

2015-07-03 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/7157#issuecomment-118359398 hi @chenghao-intel , thanks for looking into this! I agree with your opinions, but have some other thoughts: * In this PR I added a new `nullSafeEval` for binary

[GitHub] spark pull request: [SPARK-6602][Core]Replace Akka Serialization w...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7159#issuecomment-118361215 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-6602][Core]Replace Akka Serialization w...

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

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-03 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7184#discussion_r33868622 --- Diff: mllib/src/test/java/org/apache/spark/ml/feature/JavaPCASuite.java --- @@ -0,0 +1,114 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-8796][SQL] make sure SparkPlan is only ...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7192#issuecomment-118369743 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8389][Streaming][PySpark] Expose KafkaR...

2015-07-03 Thread jerryshao
Github user jerryshao commented on the pull request: https://github.com/apache/spark/pull/7185#issuecomment-118369856 Still has serialization bug in the code, will fix it soon. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-8800][SQL] Fix inaccurate precision/sca...

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

[GitHub] spark pull request: [SPARK-8733][MLLIB] ML RDD.unpersist calls sho...

2015-07-03 Thread ilganeli
Github user ilganeli closed the pull request at: https://github.com/apache/spark/pull/7160 --- 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-7785] [MLlib] [PySpark] Add __str__ and...

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

[GitHub] spark pull request: [SPARK-5259][CORE]Make sure shuffle metadata a...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4055#issuecomment-118341006 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8796][SQL] mark child as transient in I...

2015-07-03 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/7192#discussion_r33864096 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala --- @@ -100,8 +100,8 @@ abstract class SparkPlan extends

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-118364340 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8796][SQL] make sure SparkPlan is only ...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7192#issuecomment-118369720 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8788] [ML] Add Java unit test for PCA t...

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7184#issuecomment-118370711 [Test build #36503 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36503/console) for PR 7184 at commit

[GitHub] spark pull request: [SPARK-8796][SQL] mark child as transient in I...

2015-07-03 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/7192#discussion_r33860383 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala --- @@ -100,8 +100,8 @@ abstract class SparkPlan extends

[GitHub] spark pull request: [SPARK-5259][CORE]Make sure shuffle metadata a...

2015-07-03 Thread suyanNone
Github user suyanNone commented on a diff in the pull request: https://github.com/apache/spark/pull/4055#discussion_r33861293 --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala --- @@ -1193,8 +1193,10 @@ class DAGScheduler( // TODO: This will

[GitHub] spark pull request: [SPARK-8759][SQL] add default eval to binary a...

2015-07-03 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/7157#issuecomment-118340829 Thank you for working on this @cloud-fan.This PR definitely will save the code, but I have some concerns on the interface updating, not about what you did in

[GitHub] spark pull request: [SPARK-5259][CORE]Make sure shuffle metadata a...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4055#issuecomment-118341078 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-7871][SQL]Improve the outputPartitionin...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6413#issuecomment-118347540 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-7871][SQL]Improve the outputPartitionin...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6413#issuecomment-118347630 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8796][SQL] mark child as transient in I...

2015-07-03 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/7192#issuecomment-118355222 hi @chenghao-intel , IMO `SparkPlan` should only be executed at driver side and no need to serialize and pass it to executor side. So in this PR, I'm trying to remove

[GitHub] spark pull request: [SPARK-8238][SPARK-8239][SPARK-8242][SPARK-824...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6843#issuecomment-118355942 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8238][SPARK-8239][SPARK-8242][SPARK-824...

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

[GitHub] spark pull request: [SPARK-6305] add log4j2 profile and prevent lo...

2015-07-03 Thread ejsarge-gr
Github user ejsarge-gr commented on the pull request: https://github.com/apache/spark/pull/4998#issuecomment-118378860 Would we be able to document this better or re-examine it a little? The problem is that log4j 1.2.17 is shaded into the assembly jar. Thus, to override it we

[GitHub] spark pull request: [SPARK-8501] [SQL] Avoids reading schema from ...

2015-07-03 Thread sarutak
Github user sarutak commented on the pull request: https://github.com/apache/spark/pull/7200#issuecomment-118382044 This PR has already merged right? It's funny that this PR is still open. @liancheng Mind manually closing? --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-8582][Core] Add CheckpointingIterator t...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7021#issuecomment-118386386 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-8582][Core] Add CheckpointingIterator t...

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

[GitHub] spark pull request: [SPARK-6980] [CORE] Akka timeout exceptions in...

2015-07-03 Thread tedyu
Github user tedyu commented on a diff in the pull request: https://github.com/apache/spark/pull/6205#discussion_r33873268 --- Diff: core/src/main/scala/org/apache/spark/rpc/RpcEnv.scala --- @@ -182,3 +184,109 @@ private[spark] object RpcAddress { RpcAddress(host, port)

[GitHub] spark pull request: [SPARK-8279][SQL]Add math function round

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

[GitHub] spark pull request: [SPARK-8269][SQL]string function: initcap

2015-07-03 Thread tarekauel
Github user tarekauel commented on the pull request: https://github.com/apache/spark/pull/7208#issuecomment-118405356 Can you update the title and the soundex Jira ticket name. I guess you have mixed up the branches. This pr contains code of #7115 --- If your project is set up for

[GitHub] spark pull request: [SPARK-5707][SQL] Avoid codegen for HashedRela...

2015-07-03 Thread marmbrus
GitHub user marmbrus opened a pull request: https://github.com/apache/spark/pull/7213 [SPARK-5707][SQL] Avoid codegen for HashedRelation keys Using codegen causes serialized code generated row types to leak out of the JVM when doing broadcast joins. This PR changes it so we always

[GitHub] spark pull request: [SPARK-8279][SQL]Add math function round

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

[GitHub] spark pull request: [SPARK-8582][Core] Add CheckpointingIterator t...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7021#issuecomment-118387170 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8582][Core] Add CheckpointingIterator t...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7021#issuecomment-118387151 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8279][SQL]Add math function round

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6938#issuecomment-118388416 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-6980] [CORE] Akka timeout exceptions in...

2015-07-03 Thread BryanCutler
Github user BryanCutler commented on the pull request: https://github.com/apache/spark/pull/6205#issuecomment-118401399 Awesome, thanks for all your help @squito! That was a heck of a first Jira, but it was a great experience and I definitely learned a lot. --- If your project is

[GitHub] spark pull request: [SPARK-8810] [SQL] Added several UDF unit test...

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

[GitHub] spark pull request: [SPARK-8279][SQL]Add math function round

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6938#issuecomment-118404825 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-8582][Core] Add CheckpointingIterator t...

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

[GitHub] spark pull request: [SPARK-8279][SQL]Add math function round

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6938#issuecomment-118389639 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-8279][SQL]Add math function round

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6938#issuecomment-118389624 [Test build #36510 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36510/console) for PR 6938 at commit

[GitHub] spark pull request: [SPARK-8656][WebUI] Fix the webUI and JSON API...

2015-07-03 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/7038#discussion_r33873825 --- Diff: core/src/main/scala/org/apache/spark/deploy/master/WorkerInfo.scala --- @@ -107,4 +107,6 @@ private[spark] class WorkerInfo( def

[GitHub] spark pull request: [SPARK-8656][WebUI] Fix the webUI and JSON API...

2015-07-03 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/7038#discussion_r33873807 --- Diff: core/src/main/scala/org/apache/spark/deploy/JsonProtocol.scala --- @@ -76,12 +76,13 @@ private[deploy] object JsonProtocol { }

[GitHub] spark pull request: [SPARK-8810] [SQL] Added several UDF unit test...

2015-07-03 Thread spirom
Github user spirom commented on a diff in the pull request: https://github.com/apache/spark/pull/7207#discussion_r33876368 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/UDFSuite.scala --- @@ -82,6 +83,48 @@ class UDFSuite extends QueryTest { assert(ctx.sql(SELECT

[GitHub] spark pull request: [SPARK-8810] [SQL] Added several UDF unit test...

2015-07-03 Thread spirom
Github user spirom commented on a diff in the pull request: https://github.com/apache/spark/pull/7207#discussion_r33876378 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/UDFSuite.scala --- @@ -82,6 +83,48 @@ class UDFSuite extends QueryTest { assert(ctx.sql(SELECT

[GitHub] spark pull request: [SPARK-8269][SQL]string function: initcap

2015-07-03 Thread tarekauel
Github user tarekauel commented on a diff in the pull request: https://github.com/apache/spark/pull/7208#discussion_r33876817 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringOperations.scala --- @@ -298,3 +299,56 @@ case class

[GitHub] spark pull request: [SPARK-8279][SQL]Add math function round

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6938#issuecomment-118405114 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 project does

[GitHub] spark pull request: [SPARK-8549][SparkR] Fix the line length of Sp...

2015-07-03 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/7204#issuecomment-118380990 Thanks @yu-iskw -- Just for my information can you paste a link to output of `dev/lint-r` after this change ? --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-8279][SQL]Add math function round

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6938#issuecomment-118385017 Build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-8796][SQL] make sure SparkPlan is only ...

2015-07-03 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/7192#issuecomment-118388532 I am sorry if I missed anything. What is the problem? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-8279][SQL]Add math function round

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6938#issuecomment-118388395 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8776] Increase the default MaxPermSize

2015-07-03 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/7196#discussion_r33873758 --- Diff: launcher/src/test/java/org/apache/spark/launcher/SparkSubmitCommandBuilderSuite.java --- @@ -194,7 +194,7 @@ private void

[GitHub] spark pull request: [SPARK-8796][SQL] make sure SparkPlan is only ...

2015-07-03 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/7192#issuecomment-118390526 @cloud-fan not sure if I follow. The `$out` reference is generally something pulled into the closure by the compiler, not something we can avoid. In fact, closure

[GitHub] spark pull request: [SPARK-8279][SQL]Add math function round

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6938#issuecomment-118391418 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8279][SQL]Add math function round

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6938#issuecomment-118391433 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-6602][Core]Replace Akka Serialization w...

2015-07-03 Thread adamv
Github user adamv commented on a diff in the pull request: https://github.com/apache/spark/pull/7159#discussion_r33875006 --- Diff: core/pom.xml --- @@ -373,6 +373,10 @@ scopetest/scope /dependency dependency + groupIdorg.apache.curator/groupId

[GitHub] spark pull request: [SPARK-8582][Core] Add CheckpointingIterator t...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7021#issuecomment-118406411 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-8582][Core] Add CheckpointingIterator t...

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7021#issuecomment-118406123 [Test build #36509 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36509/console) for PR 7021 at commit

[GitHub] spark pull request: [SPARK-5707][SQL] Avoid codegen for HashedRela...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7213#issuecomment-118409672 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-5707][SQL] Avoid codegen for HashedRela...

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7213#issuecomment-118409666 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8279][SQL]Add math function round

2015-07-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6938#issuecomment-118385031 Build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-8582][Core] Add CheckpointingIterator t...

2015-07-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7021#issuecomment-118386384 [Test build #36508 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36508/console) for PR 7021 at commit

[GitHub] spark pull request: [SPARK-8279][SQL]Add math function round

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

  1   2   3   4   >