[GitHub] spark pull request: [SPARK-4148][PySpark] fix seed distribution an...

2014-11-03 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/3010#issuecomment-61448170 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-3586][streaming]Support nested director...

2014-11-03 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/2765#discussion_r19721473 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/StreamingContext.scala --- @@ -351,8 +351,8 @@ class StreamingContext private[streaming] (

[GitHub] spark pull request: [SPARK-3586][streaming]Support nested director...

2014-11-03 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/2765#discussion_r19721494 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/dstream/FileInputDStream.scala --- @@ -104,7 +107,25 @@ class FileInputDStream[K: ClassTag,

[GitHub] spark pull request: [Minor] Minor bug fixes in bin/run-example

2014-11-03 Thread witgo
Github user witgo commented on the pull request: https://github.com/apache/spark/pull/3069#issuecomment-61449081 `mvn package` generated file is like this `spark-examples-1.2.0-SNAPSHOT-*` . `./sbt/sbt clean assembly` generated file is like this

[GitHub] spark pull request: [Minor] Minor bug fixes in bin/run-example

2014-11-03 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/3069#issuecomment-61449297 Ah I see. But shouldn't the naming just be made consistent between SBT and Maven then? --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [Minor] Minor bug fixes in bin/run-example

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

[GitHub] spark pull request: [Minor] Minor bug fixes in bin/run-example

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

[GitHub] spark pull request: The `hive-0.13.1` maven profile now sets `hive...

2014-11-03 Thread coderfi
GitHub user coderfi opened a pull request: https://github.com/apache/spark/pull/3072 The `hive-0.13.1` maven profile now sets `hive.version=0.13.1a` instead of `hive.version=0.13.1`. e.g. mvn -Phive -Phive=0.13.1 Note: `hive.version=0.13.1a` is the default property

[GitHub] spark pull request: The `hive-0.13.1` maven profile now sets `hive...

2014-11-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3072#issuecomment-61449561 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-3826][SQL]enable hive-thriftserver to s...

2014-11-03 Thread coderfi
Github user coderfi commented on the pull request: https://github.com/apache/spark/pull/2685#issuecomment-61449633 @liancheng PR #3072 created (all of one line! :) ). --- 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: The `hive-0.13.1` maven profile now sets `hive...

2014-11-03 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/3072#issuecomment-61449876 test this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

2014-11-03 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/3066#discussion_r19722086 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/package.scala --- @@ -451,4 +451,33 @@ package object sql { * Builder for [[Metadata]]. If

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

2014-11-03 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/3066#discussion_r19722088 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/package.scala --- @@ -451,4 +451,33 @@ package object sql { * Builder for [[Metadata]]. If

[GitHub] spark pull request: The `hive-0.13.1` maven profile now sets `hive...

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

[GitHub] spark pull request: [WIP] [SPARK-4198] [Core] Refactor some BlockM...

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

[GitHub] spark pull request: [WIP] [SPARK-4198] [Core] Refactor some BlockM...

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

[GitHub] spark pull request: Move SparkContext accumulator methods to Accum...

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

[GitHub] spark pull request: Move SparkContext accumulator methods to Accum...

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

[GitHub] spark pull request: [MLLIB] [spark-2352] Implementation of an 1-hi...

2014-11-03 Thread bgreeven
Github user bgreeven commented on a diff in the pull request: https://github.com/apache/spark/pull/1290#discussion_r19722711 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/ann/ArtificialNeuralNetwork.scala --- @@ -0,0 +1,528 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [MLLIB] [spark-2352] Implementation of an 1-hi...

2014-11-03 Thread bgreeven
Github user bgreeven commented on a diff in the pull request: https://github.com/apache/spark/pull/1290#discussion_r19723239 --- Diff: docs/mllib-ann.md --- @@ -0,0 +1,223 @@ +--- +layout: global +title: Artificial Neural Networks - MLlib +displayTitle: a

[GitHub] spark pull request: [SPARK-3573][MLLIB] Make MLlib's Vector compat...

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

[GitHub] spark pull request: [SPARK-3573][MLLIB] Make MLlib's Vector compat...

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

[GitHub] spark pull request: The `hive-0.13.1` maven profile now sets `hive...

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

[GitHub] spark pull request: The `hive-0.13.1` maven profile now sets `hive...

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

[GitHub] spark pull request: The `hive-0.13.1` maven profile now sets `hive...

2014-11-03 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/3072#issuecomment-61457803 @coderfi Would you please file a JIRA ticket for this and then rename the title of this PR to something like [SPARK-][Build] Fixes hive.version in Maven profile

[GitHub] spark pull request: The `hive-0.13.1` maven profile now sets `hive...

2014-11-03 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/3072#issuecomment-61458285 This LGTM besides PR title and the missing JIRA ticket. Thanks! --- 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-3891][SQL] Add array support to percent...

2014-11-03 Thread gvramana
Github user gvramana commented on the pull request: https://github.com/apache/spark/pull/2802#issuecomment-61459117 Reworked on this over the changes of #2762. Updated the pull request comment. Please test the same --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [Minor] Minor bug fixes in bin/run-example

2014-11-03 Thread witgo
Github user witgo commented on the pull request: https://github.com/apache/spark/pull/3069#issuecomment-61463811 The current solution is simple to implement, and other source also used this solution. eg:

[GitHub] spark pull request: [SPARK-3891][SQL] Add array support to percent...

2014-11-03 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/2802#issuecomment-61464552 test this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-3891][SQL] Add array support to percent...

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

[GitHub] spark pull request: [SPARK-3891][SQL] Add array support to percent...

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

[GitHub] spark pull request: [SPARK-3891][SQL] Add array support to percent...

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

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

2014-11-03 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/3066#issuecomment-61468129 test this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

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

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

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

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

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

[GitHub] spark pull request: [SPARK-3965] Ensure that Spark assembly for ha...

2014-11-03 Thread dajac
Github user dajac commented on the pull request: https://github.com/apache/spark/pull/2822#issuecomment-61469324 @pwendell any news on 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 does not

[GitHub] spark pull request: [SPARK-4204][Core][WebUI] Change Utils.excepti...

2014-11-03 Thread zsxwing
GitHub user zsxwing opened a pull request: https://github.com/apache/spark/pull/3073 [SPARK-4204][Core][WebUI] Change Utils.exceptionString to contain the inner exceptions and make the error information in Web UI more friendly This PR fixed `Utils.exceptionString` to output the

[GitHub] spark pull request: [SPARK-4204][Core][WebUI] Change Utils.excepti...

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

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

2014-11-03 Thread culler
Github user culler commented on a diff in the pull request: https://github.com/apache/spark/pull/3066#discussion_r19730958 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/dsl/package.scala --- @@ -146,6 +146,31 @@ package object dsl { def upper(e:

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

2014-11-03 Thread culler
Github user culler commented on a diff in the pull request: https://github.com/apache/spark/pull/3066#discussion_r19731906 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Projection.scala --- @@ -139,6 +140,12 @@ class JoinedRow extends Row {

[GitHub] spark pull request: [SPARK-732][SPARK-3628][CORE][RESUBMIT] elimin...

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

[GitHub] spark pull request: [SPARK-4012] call tryOrExit instead of logUnca...

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

[GitHub] spark pull request: [SPARK-3967] Ensure that files are fetched ato...

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

[GitHub] spark pull request: [SPARK-1545] [mllib] Add Random Forests

2014-11-03 Thread 0asa
Github user 0asa commented on a diff in the pull request: https://github.com/apache/spark/pull/2435#discussion_r19732330 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/tree/impl/DecisionTreeMetadata.scala --- @@ -128,13 +139,34 @@ private[tree] object DecisionTreeMetadata

[GitHub] spark pull request: [SPARK-3967] Ensure that files are fetched ato...

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

[GitHub] spark pull request: [SPARK-3967] Ensure that files are fetched ato...

2014-11-03 Thread preaudc
Github user preaudc commented on the pull request: https://github.com/apache/spark/pull/2855#issuecomment-61479433 I've just resync my local fork so that the patch merges cleanly (i.e. there are no other modifications compared to the initial patch) --- If your project is set up for

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

2014-11-03 Thread culler
Github user culler commented on a diff in the pull request: https://github.com/apache/spark/pull/3066#discussion_r19733162 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Row.scala --- @@ -42,6 +44,31 @@ object Row { * This method can be

[GitHub] spark pull request: [MLLIB] [spark-2352] Implementation of an 1-hi...

2014-11-03 Thread bgreeven
Github user bgreeven commented on a diff in the pull request: https://github.com/apache/spark/pull/1290#discussion_r19733377 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/ann/ArtificialNeuralNetwork.scala --- @@ -0,0 +1,528 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [MLLIB] [spark-2352] Implementation of an 1-hi...

2014-11-03 Thread bgreeven
Github user bgreeven commented on a diff in the pull request: https://github.com/apache/spark/pull/1290#discussion_r19733442 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/ann/ArtificialNeuralNetwork.scala --- @@ -0,0 +1,528 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

2014-11-03 Thread culler
Github user culler commented on a diff in the pull request: https://github.com/apache/spark/pull/3066#discussion_r19733534 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/types/dataTypes.scala --- @@ -187,7 +187,8 @@ case object NullType extends DataType

[GitHub] spark pull request: [MLLIB] [spark-2352] Implementation of an 1-hi...

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

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

2014-11-03 Thread culler
Github user culler commented on a diff in the pull request: https://github.com/apache/spark/pull/3066#discussion_r19733740 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/types/timetypes.scala --- @@ -0,0 +1,97 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-732][SPARK-3628][CORE][RESUBMIT] elimin...

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

[GitHub] spark pull request: [SPARK-732][SPARK-3628][CORE][RESUBMIT] elimin...

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

[GitHub] spark pull request: [MLLIB] [spark-2352] Implementation of an 1-hi...

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

[GitHub] spark pull request: [SPARK-4204][Core][WebUI] Change Utils.excepti...

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

[GitHub] spark pull request: [SPARK-4204][Core][WebUI] Change Utils.excepti...

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

[GitHub] spark pull request: [SPARK-1545] [mllib] Add Random Forests

2014-11-03 Thread 0asa
Github user 0asa commented on the pull request: https://github.com/apache/spark/pull/2435#issuecomment-61482956 Any ideas or plans to support more types of ensemble of decision tree (such as Extra-Trees, Bagging)? I'd be more than happy to help with those... where should I start?

[GitHub] spark pull request: [SPARK-732][SPARK-3628][CORE][RESUBMIT] elimin...

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

[GitHub] spark pull request: [SPARK-3957]: show broadcast variable resource...

2014-11-03 Thread CodingCat
Github user CodingCat commented on the pull request: https://github.com/apache/spark/pull/2851#issuecomment-61484584 so, this one is ready for the further review @shivaram @rxin @andrewor14 --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-3778] newAPIHadoopRDD doesn't properly ...

2014-11-03 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/2676#issuecomment-61484885 ping @pwendell @mateiz --- 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: Timestamp and Date classes which work in the c...

2014-11-03 Thread culler
Github user culler commented on a diff in the pull request: https://github.com/apache/spark/pull/3066#discussion_r19735638 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/package.scala --- @@ -451,4 +451,33 @@ package object sql { * Builder for [[Metadata]]. If there

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

2014-11-03 Thread culler
Github user culler commented on a diff in the pull request: https://github.com/apache/spark/pull/3066#discussion_r19736442 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/package.scala --- @@ -451,4 +451,33 @@ package object sql { * Builder for [[Metadata]]. If there

[GitHub] spark pull request: [SPARK-4012] call tryOrExit instead of logUnca...

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

[GitHub] spark pull request: [SPARK-4012] call tryOrExit instead of logUnca...

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

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

2014-11-03 Thread culler
Github user culler commented on the pull request: https://github.com/apache/spark/pull/3066#issuecomment-61488635 I have corrected the issues pointed out in the comments. Please retest. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-4012] call tryOrExit instead of logUnca...

2014-11-03 Thread CodingCat
Github user CodingCat commented on the pull request: https://github.com/apache/spark/pull/2864#issuecomment-61489536 Hey, @andrewor14 , Any further thoughts about this PR? --- 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-3967] Ensure that files are fetched ato...

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

[GitHub] spark pull request: [SPARK-3967] Ensure that files are fetched ato...

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

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

2014-11-03 Thread culler
Github user culler commented on a diff in the pull request: https://github.com/apache/spark/pull/3066#discussion_r19737251 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/columnar/ColumnType.scala --- @@ -372,6 +372,11 @@ private[sql] object TIMESTAMP extends

[GitHub] spark pull request: [SPARK-3967] Ensure that files are fetched ato...

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

[GitHub] spark pull request: [SPARK-3967] Ensure that files are fetched ato...

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

[GitHub] spark pull request: [SPARK-732][SPARK-3628][CORE][RESUBMIT] elimin...

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

[GitHub] spark pull request: [SPARK-1545] [mllib] Add Random Forests

2014-11-03 Thread manishamde
Github user manishamde commented on the pull request: https://github.com/apache/spark/pull/2435#issuecomment-61494531 @0asa Yes. PRs for these will be great. Could you check if there are already existing JIRA for these -- if not, you could create a JIRA tickets. Also, please

[GitHub] spark pull request: [SPARK-732][SPARK-3628][CORE][RESUBMIT] elimin...

2014-11-03 Thread CodingCat
Github user CodingCat commented on the pull request: https://github.com/apache/spark/pull/2524#issuecomment-61494925 still flaky test...@mateiz, shall we get this merged before 1.2 release? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-732][SPARK-3628][CORE][RESUBMIT] elimin...

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

[GitHub] spark pull request: Timestamp and Date classes which work in the c...

2014-11-03 Thread culler
Github user culler commented on the pull request: https://github.com/apache/spark/pull/3066#issuecomment-61498140 Hmmm. The GitHub docs seem to suggest that I am supposed to mention @liancheng and @rxin in my comments. So I am doing that. My apologies if this is superfluous or

[GitHub] spark pull request: [MLLIB] [spark-2352] Implementation of an 1-hi...

2014-11-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1290#issuecomment-61499579 **[Test build #22818 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22818/consoleFull)** for PR 1290 at commit

[GitHub] spark pull request: [MLLIB] [spark-2352] Implementation of an 1-hi...

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

[GitHub] spark pull request: Support for mesos DockerInfo in fine-grained m...

2014-11-03 Thread hellertime
GitHub user hellertime opened a pull request: https://github.com/apache/spark/pull/3074 Support for mesos DockerInfo in fine-grained mode. This patch adds partial support for running spark on mesos inside of a docker container. Only fine-grained mode is presently supported, and

[GitHub] spark pull request: Support for mesos DockerInfo in fine-grained m...

2014-11-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3074#issuecomment-61501664 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: Support for mesos DockerInfo in fine-grained m...

2014-11-03 Thread hellertime
Github user hellertime commented on the pull request: https://github.com/apache/spark/pull/3074#issuecomment-61502880 This patch is in reference to https://issues.apache.org/jira/browse/SPARK-2691 --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-4207][SQL] Query which has syntax like ...

2014-11-03 Thread ravipesala
GitHub user ravipesala opened a pull request: https://github.com/apache/spark/pull/3075 [SPARK-4207][SQL] Query which has syntax like 'not like' is not working in Spark SQL Queries which has 'not like' is not working spark sql. sql(SELECT * FROM records where value not

[GitHub] spark pull request: [SPARK-4207][SQL] Query which has syntax like ...

2014-11-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3075#issuecomment-61507681 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-1021] Defer the data-driven computation...

2014-11-03 Thread erikerlandson
Github user erikerlandson commented on the pull request: https://github.com/apache/spark/pull/1689#issuecomment-61508261 @marmbrus, FWIW, the `correlationoptimizer14` test appears to be working for me. I ran it using: `env _RUN_SQL_TESTS=true _SQL_TESTS_ONLY=true ./dev/run-tests

[GitHub] spark pull request: [SPARK-3957]: show broadcast variable resource...

2014-11-03 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/2851#discussion_r19746018 --- Diff: core/src/main/scala/org/apache/spark/HeartbeatReceiver.scala --- @@ -19,7 +19,7 @@ package org.apache.spark import akka.actor.Actor

[GitHub] spark pull request: The `hive-0.13.1` maven profile now sets `hive...

2014-11-03 Thread coderfi
Github user coderfi commented on the pull request: https://github.com/apache/spark/pull/3072#issuecomment-61509091 Hello, I'll file a ticket. Suggestion on the title? Fi On Nov 3, 2014 2:09 AM, Cheng Lian notificati...@github.com wrote: This LGTM

[GitHub] spark pull request: [SPARK-3957]: show broadcast variable resource...

2014-11-03 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/2851#discussion_r19746366 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockId.scala --- @@ -30,12 +30,13 @@ import org.apache.spark.annotation.DeveloperApi * If your

[GitHub] spark pull request: [SPARK-4137] [EC2] Don't change working dir on...

2014-11-03 Thread nchammas
Github user nchammas commented on a diff in the pull request: https://github.com/apache/spark/pull/2988#discussion_r19746328 --- Diff: ec2/spark_ec2.py --- @@ -718,12 +726,16 @@ def get_num_disks(instance_type): return 1 -# Deploy the configuration file

[GitHub] spark pull request: [EC2] Factor out Mesos spark-ec2 branch

2014-11-03 Thread nchammas
Github user nchammas commented on the pull request: https://github.com/apache/spark/pull/3008#issuecomment-61509758 cc @JoshRosen @pwendell for final review / merging. --- 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-3957]: show broadcast variable resource...

2014-11-03 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/2851#discussion_r19746633 --- Diff: core/src/main/scala/org/apache/spark/scheduler/SparkListener.scala --- @@ -56,6 +56,11 @@ case class SparkListenerTaskEnd( extends

[GitHub] spark pull request: [EC2] Factor out Mesos spark-ec2 branch

2014-11-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/3008 --- 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: [EC2] Factor out Mesos spark-ec2 branch

2014-11-03 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/3008#issuecomment-61510681 Actually this is a pretty small change -- I just merged this --- 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: [EC2] Factor out Mesos spark-ec2 branch

2014-11-03 Thread nchammas
Github user nchammas commented on the pull request: https://github.com/apache/spark/pull/3008#issuecomment-61511143 Word. Thanks @shivaram. --- 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-3957]: show broadcast variable resource...

2014-11-03 Thread CodingCat
Github user CodingCat commented on a diff in the pull request: https://github.com/apache/spark/pull/2851#discussion_r19747011 --- Diff: core/src/main/scala/org/apache/spark/scheduler/SparkListener.scala --- @@ -56,6 +56,11 @@ case class SparkListenerTaskEnd( extends

[GitHub] spark pull request: [SPARK-3573][MLLIB] Make MLlib's Vector compat...

2014-11-03 Thread sryza
Github user sryza commented on a diff in the pull request: https://github.com/apache/spark/pull/3070#discussion_r19751354 --- Diff: examples/src/main/scala/org/apache/spark/examples/mllib/DatasetExample.scala --- @@ -0,0 +1,121 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-3957]: show broadcast variable resource...

2014-11-03 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/2851#discussion_r19752142 --- Diff: core/src/main/scala/org/apache/spark/scheduler/SparkListener.scala --- @@ -56,6 +56,11 @@ case class SparkListenerTaskEnd( extends

[GitHub] spark pull request: [SPARK-3957]: show broadcast variable resource...

2014-11-03 Thread CodingCat
Github user CodingCat commented on a diff in the pull request: https://github.com/apache/spark/pull/2851#discussion_r19752830 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockId.scala --- @@ -30,12 +30,13 @@ import org.apache.spark.annotation.DeveloperApi * If your

[GitHub] spark pull request: SPARK-4203 Partition directories in random ord...

2014-11-03 Thread tbfenet
GitHub user tbfenet opened a pull request: https://github.com/apache/spark/pull/3076 SPARK-4203 Partition directories in random order when inserting into hive table You can merge this pull request into a Git repository by running: $ git pull https://github.com/tbfenet/spark

  1   2   3   4   5   >