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

2015-06-29 Thread ankurcha
Github user ankurcha commented on the pull request: https://github.com/apache/spark/pull/5563#issuecomment-116781619 took care of those @squito --- 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-8704] [ML] [PySpark] Add additional met...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7086#issuecomment-116783301 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-8611] [WebUI] Fix wrong REST UI Protoco...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/7006#issuecomment-116783952 Hi sorry for slipping I'll look at this today. --- 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-8048] Partitionning of an RDD with 0 pa...

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

[GitHub] spark pull request: [SPARK-8048] Partitionning of an RDD with 0 pa...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7020#issuecomment-116784048 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-8437] [DOCS] Using directory path witho...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/7036#issuecomment-116785704 but yes, 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

[GitHub] spark pull request: [SPARK-8437] [DOCS] Using directory path witho...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/7036#issuecomment-116785573 Is having `*` always strictly faster? If so I wonder if we should just always append it if it's missing. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-8410][SPARK-8475] remove previous ivy r...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7089#issuecomment-116787969 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-5161] [WIP] Parallelize Python test exe...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7031#issuecomment-116787900 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-1537 [WiP] Application Timeline Server i...

2015-06-29 Thread steveloughran
Github user steveloughran commented on a diff in the pull request: https://github.com/apache/spark/pull/5423#discussion_r33496190 --- Diff: docs/monitoring.md --- @@ -256,6 +256,157 @@ still required, though there is only one application available. Eg. to see the running

[GitHub] spark pull request: [SPARK-5161] [WIP] Parallelize Python test exe...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7031#issuecomment-116787968 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-8410][SPARK-8475] remove previous ivy r...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7089#issuecomment-116787888 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-8437] [DOCS] Using directory path witho...

2015-06-29 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/7036#issuecomment-116786118 I think the problem is that it's not certain that the FS supports *, although the usual ones do, it seems. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-5161] [WIP] Parallelize Python test exe...

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

[GitHub] spark pull request: [SPARK-8692][SQL] re-order the case statements...

2015-06-29 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/7073#discussion_r33496993 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/columnar/ColumnStatsSuite.scala --- @@ -22,19 +22,19 @@ import

[GitHub] spark pull request: [SPARK-6910] [SQL] Support for pushing predica...

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

[GitHub] spark pull request: [SPARK-8657] [YARN] Fail to upload conf archiv...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/7055#issuecomment-116792972 @litao-buptsse would you mind opening this against the master branch so future Spark releases also benefit from the fix? --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-8681] fixed wrong ordering of columns i...

2015-06-29 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/7060#discussion_r33497360 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameStatSuite.scala --- @@ -65,22 +67,22 @@ class DataFrameStatSuite extends SparkFunSuite {

[GitHub] spark pull request: [SPARK-8378][Streaming]Add the Python API for ...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6830#issuecomment-116793699 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-8378][Streaming]Add the Python API for ...

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

[GitHub] spark pull request: [SPARK-8590][SQL] add code gen for ExtractValu...

2015-06-29 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/6982#issuecomment-116794010 @cloud-fan This PR looks pretty good to me. Since we're replying on `StructField.nullable` to skip nullablity check in codegen, that could be kind of risk, because we

[GitHub] spark pull request: [SPARK-8709] Exclude hadoop-client's mockito-a...

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

[GitHub] spark pull request: [SPARK-8688][YARN]Bug fix: disable the cache f...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/7069#discussion_r33498497 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/AMDelegationTokenRenewer.scala --- @@ -65,6 +65,8 @@ private[yarn] class

[GitHub] spark pull request: [SPARK-5161] [WIP] Parallelize Python test exe...

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

[GitHub] spark pull request: SPARK-7889 Jobs progress of apps on complete p...

2015-06-29 Thread steveloughran
Github user steveloughran commented on the pull request: https://github.com/apache/spark/pull/6935#issuecomment-116795511 (BTW, I'm not going to be looking at this for a couple of weeks; focusing on a (big) patch --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-8688][YARN]Bug fix: disable the cache f...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/7069#discussion_r33498576 --- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala --- @@ -334,6 +334,17 @@ class SparkHadoopUtil extends Logging { *

[GitHub] spark pull request: [SPARK-8066, SPARK-8067] [hive] Add support fo...

2015-06-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7026 --- 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-8710] [SQL] Change ScalaReflection.mirr...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7094#issuecomment-116814815 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-7862] [SQL] Disable the error message r...

2015-06-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/6882 --- 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-8700][ML] Disable feature scaling in Lo...

2015-06-29 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/7080#discussion_r33505545 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala --- @@ -534,27 +554,39 @@ private class LogisticCostFun(

[GitHub] spark pull request: [SPARK-8700][ML] Disable feature scaling in Lo...

2015-06-29 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/7080#discussion_r33505552 --- Diff: mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala --- @@ -208,8 +208,11 @@ class LogisticRegressionSuite

[GitHub] spark pull request: [SPARK-8700][ML] Disable feature scaling in Lo...

2015-06-29 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/7080#discussion_r33505540 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala --- @@ -149,15 +158,26 @@ class LogisticRegression(override val

[GitHub] spark pull request: [SPARK-8675] Executors created by LocalBackend...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7091#issuecomment-116828194 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-8579] [SQL] support arbitrary object in...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6959#issuecomment-116828193 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-8579] [SQL] support arbitrary object in...

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

[GitHub] spark pull request: [SPARK-8592] [CORE] CoarseGrainedExecutorBacke...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7077#issuecomment-116832856 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-8656][WebUI] Fix the webUI and JSON API...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7038#issuecomment-116832820 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-8669] [SQL] Fix crash with BINARY (ENUM...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7048#issuecomment-116832803 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-8669] [SQL] Fix crash with BINARY (ENUM...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7048#issuecomment-116832857 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-8660

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7096#issuecomment-116832823 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-8592] [CORE] CoarseGrainedExecutorBacke...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7077#issuecomment-116832797 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-7171: Added a method to retrieve metrics...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5805#issuecomment-116832906 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-7171: Added a method to retrieve metrics...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5805#issuecomment-116832855 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-8669] [SQL] Fix crash with BINARY (ENUM...

2015-06-29 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/7048#issuecomment-116831914 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 feature

[GitHub] spark pull request: [SPARK-8579] [SQL] support arbitrary object in...

2015-06-29 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/6959#discussion_r33510353 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/UnsafeFixedWidthAggregationMapSuite.scala --- @@ -33,10 +34,10 @@ class

[GitHub] spark pull request: [SPARK-8538][SPARK-8539][ML] Linear Regression...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7099#issuecomment-116839436 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-8538][SPARK-8539][ML] Linear Regression...

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

[GitHub] spark pull request: [SPARK-8393] [Streaming] Handle InterruptedExc...

2015-06-29 Thread rekhajoshm
Github user rekhajoshm commented on the pull request: https://github.com/apache/spark/pull/6969#issuecomment-116844189 ack @andrewor14 --- 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-5681][Streaming] Use a lock to eliminat...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/6294#issuecomment-116846507 @zsxwing @tdas the hanging issue seems like one that would be good to fix. Any updates on this patch? --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-8410][SPARK-8475] remove previous ivy r...

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

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4399#issuecomment-116846496 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-8709] Exclude hadoop-client's mockito-a...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7090#issuecomment-116846495 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-8199][SPARK-8184][SPARK-8183][SPARK-818...

2015-06-29 Thread tarekauel
Github user tarekauel commented on a diff in the pull request: https://github.com/apache/spark/pull/6981#discussion_r33513954 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetime.scala --- @@ -0,0 +1,255 @@ +/* + * Licensed to the

[GitHub] spark pull request: [SPARK-7810] [pyspark] solve python rdd socket...

2015-06-29 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/6338#issuecomment-116853270 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-8691][WebUI]Enable GZip by default for ...

2015-06-29 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/7072#discussion_r33516520 --- Diff: core/src/main/scala/org/apache/spark/ui/JettyUtils.scala --- @@ -166,8 +167,26 @@ private[spark] object JettyUtils extends Logging {

[GitHub] spark pull request: [SPARK-7810] [pyspark] solve python rdd socket...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6338#issuecomment-116853428 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-7810] [pyspark] solve python rdd socket...

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

[GitHub] spark pull request: [SPARK-8660][ML] Convert JavaDoc style comment...

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

[GitHub] spark pull request: [SPARK-6910] [SQL] Support for pushing predica...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6921#issuecomment-116857401 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-8478][SQL] Harmonize UDF-related code t...

2015-06-29 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/6920#issuecomment-116864319 Thanks, merged to 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 have

[GitHub] spark pull request: [SPARK-8669] [SQL] Fix crash with BINARY (ENUM...

2015-06-29 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/7048#issuecomment-116874802 Hm, not knowing why @SparkQA said the build failed, it was actually a success. Anyway, I'm merging this to master. Thanks again for working on this! --- If your

[GitHub] spark pull request: [SPARK-8661][ML] for LinearRegressionSuite.sca...

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

[GitHub] spark pull request: [SPARK-8661][ML] for LinearRegressionSuite.sca...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7098#issuecomment-116874930 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-8660

2015-06-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7096#issuecomment-116830752 Jenkins, test this please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: Spark-8660

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7096#issuecomment-116830453 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-8710] [SQL] Change ScalaReflection.mirr...

2015-06-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7094#discussion_r33508154 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala --- @@ -28,7 +28,8 @@ import org.apache.spark.sql.types._ */

[GitHub] spark pull request: [SPARK-6910] [SQL] Support for pushing predica...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6921#issuecomment-116830518 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-6910] [SQL] Support for pushing predica...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6921#issuecomment-116830478 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-8619][Streaming]Don't recover keytab an...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/7008#discussion_r33509760 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/Checkpoint.scala --- @@ -44,11 +44,19 @@ class Checkpoint(@transient ssc: StreamingContext,

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

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

[GitHub] spark pull request: [SPARK-8619][Streaming]Don't recover keytab an...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/7008#discussion_r33509669 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/Checkpoint.scala --- @@ -44,11 +44,19 @@ class Checkpoint(@transient ssc: StreamingContext,

[GitHub] spark pull request: [SPARK-8634][Streaming][Tests]Fix flaky test S...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/7017#issuecomment-116840393 Merging into master 1.4... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-8579] [SQL] support arbitrary object in...

2015-06-29 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/6959#discussion_r33510593 --- Diff: sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeFixedWidthAggregationMap.java --- @@ -70,67 +92,38 @@ private

[GitHub] spark pull request: [SPARK-8434][SQL]Add a pretty parameter to t...

2015-06-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/6877#issuecomment-116845403 maybe pretty - truncate? --- 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-8323][CORE]Remove mapOutputTracker fiel...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/6778#issuecomment-116845367 @yufan-liu mind closing this 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-8660][ML] Convert JavaDoc style comment...

2015-06-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7096#issuecomment-116845502 Thanks - LGTM. We can merge this once Jenkins comes back happy. --- 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-8709] Exclude hadoop-client's mockito-a...

2015-06-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7090 --- 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-8538][SPARK-8539][ML] Linear Regression...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7099#issuecomment-116848948 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-8538][SPARK-8539][ML] Linear Regression...

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

[GitHub] spark pull request: [SPARK-8103][core] DAGScheduler should not sub...

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

[GitHub] spark pull request: [SPARK-6602][Core] Update Master, Worker, Clie...

2015-06-29 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5392#discussion_r33515404 --- Diff: core/src/main/scala/org/apache/spark/deploy/client/AppClient.scala --- @@ -40,98 +37,139 @@ import org.apache.spark.util.{ActorLogReceive,

[GitHub] spark pull request: [SPARK-8478][SQL] Harmonize UDF-related code t...

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

[GitHub] spark pull request: [SPARK-8691][WebUI]Enable GZip by default for ...

2015-06-29 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/7072#issuecomment-116853701 I would still argue for this change because in remote environments having a page with a 10MB payload is pretty bad. It's just good form to compress output for very

[GitHub] spark pull request: [SPARK-8661][ML] for LinearRegressionSuite.sca...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7098#issuecomment-116856965 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-8661][ML] for LinearRegressionSuite.sca...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7098#issuecomment-116856937 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-8579] [SQL] support arbitrary object in...

2015-06-29 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/6959#issuecomment-116859002 LGTM overall. --- 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-6602][Core] Update Master, Worker, Clie...

2015-06-29 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5392#discussion_r33518468 --- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala --- @@ -235,21 +250,47 @@ private[worker] class Worker( * still not

[GitHub] spark pull request: [SPARK-6602][Core] Update Master, Worker, Clie...

2015-06-29 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5392#discussion_r33519252 --- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala --- @@ -302,27 +367,27 @@ private[worker] class Worker( // the

[GitHub] spark pull request: [SPARK-6602][Core] Update Master, Worker, Clie...

2015-06-29 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5392#discussion_r33519250 --- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala --- @@ -302,27 +367,27 @@ private[worker] class Worker( // the

[GitHub] spark pull request: [SPARK-6910] [SQL] Support for pushing predica...

2015-06-29 Thread piaozhexiu
Github user piaozhexiu commented on the pull request: https://github.com/apache/spark/pull/6921#issuecomment-116860213 This is weird. A couple of different tests randomly fail at each run. I am wondering whether this is because derby db intermittently fails during unit tests.

[GitHub] spark pull request: [SPARK-8567] [SQL] Add logs to record the prog...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7092#issuecomment-116860415 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-6602][Core] Update Master, Worker, Clie...

2015-06-29 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5392#discussion_r33519340 --- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala --- @@ -466,23 +531,13 @@ private[worker] class Worker( case _ =

[GitHub] spark pull request: [SPARK-8029][core] shuffleoutput per attempt

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6648#issuecomment-116873768 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-8713] Make codegen thread safe

2015-06-29 Thread davies
GitHub user davies opened a pull request: https://github.com/apache/spark/pull/7101 [SPARK-8713] Make codegen thread safe Codegen takes three steps: 1. Take a list of expressions, convert them into Java source code and a list of expressions that don't not support codegen

[GitHub] spark pull request: [SPARK-8357] [SQL] Memory leakage on unsafe ag...

2015-06-29 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/6810#discussion_r33523568 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/AggregateSuite.scala --- @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-8690][SQL] Add a setting to disable Spa...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7070#issuecomment-116875210 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-8690][SQL] Add a setting to disable Spa...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7070#issuecomment-116875229 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-8130] - Made spark.files.useFetchCache ...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/7051#issuecomment-116875310 @JoshRosen and I discussed this offline. So apparently this feature has been broken for a long time. Since recently, each Spark executor actually gets its own unique

[GitHub] spark pull request: [SPARK-8715] ArrayOutOfBoundsException fixed f...

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

[GitHub] spark pull request: [HOTFIX] Fix whitespace style error

2015-06-29 Thread marmbrus
Github user marmbrus closed the pull request at: https://github.com/apache/spark/pull/7102 --- 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

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