[GitHub] spark pull request: [SPARK-10317] [Core] Compatibility between his...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8758#issuecomment-141760915 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-10718] [BUILD] Check License should not...

2015-09-20 Thread rekhajoshm
Github user rekhajoshm commented on the pull request: https://github.com/apache/spark/pull/8842#issuecomment-141760641 retest 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-10317] [Core] Compatibility between his...

2015-09-20 Thread rekhajoshm
Github user rekhajoshm commented on the pull request: https://github.com/apache/spark/pull/8758#issuecomment-141760385 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-10714][SPARK-8632][SPARK-10685][SQL] Re...

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

[GitHub] spark pull request: [SPARK-10718] [BUILD] Check License should not...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8842#issuecomment-141753849 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-10714][SPARK-8632][SPARK-10685][SQL] Re...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8835#issuecomment-141753896 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-10714][SPARK-8632][SPARK-10685][SQL] Re...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8835#issuecomment-141753897 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6457#issuecomment-141754266 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6457#issuecomment-141754265 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-10317] [Core] Compatibility between his...

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

[GitHub] spark pull request: [SPARK-10706] [MLLIB] Add java wrapper for ran...

2015-09-20 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/8841#issuecomment-141768398 This LGTM, @srowen can hopefully take a look too. --- 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-6028][Core]A new RPC implemetation base...

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

[GitHub] spark pull request: [SPARK-10718] [BUILD] Check License should not...

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

[GitHub] spark pull request: [SPARK-10718] [BUILD] Check License should not...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8842#issuecomment-141753851 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10317] [Core] Compatibility between his...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8758#issuecomment-141760881 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: Log error when spill file wasn't deleted

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

[GitHub] spark pull request: [SPARK-10626][MLLIB] create java friendly meth...

2015-09-20 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/8782#issuecomment-141814077 I'm looking at https://github.com/apache/spark/pull/8841 now, for comparison, and noted that the Java overloads there call the scala overloads directly, rather than call

[GitHub] spark pull request: [SPARK-10718] [BUILD] Check License should not...

2015-09-20 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/8842#discussion_r39932759 --- Diff: .rat-excludes --- @@ -15,15 +15,15 @@ TAGS RELEASE control docs -docker.properties.template -fairscheduler.xml.template

[GitHub] spark pull request: [SPARK-10718] [BUILD] Check License should not...

2015-09-20 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/8842#discussion_r39932853 --- Diff: .rat-excludes --- @@ -15,15 +15,15 @@ TAGS RELEASE control docs -docker.properties.template -fairscheduler.xml.template

[GitHub] spark pull request: [SPARK-10577] [PySpark] DataFrame hint for bro...

2015-09-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8801#issuecomment-141820394 [Test build #1778 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1778/console) for PR 8801 at commit

[GitHub] spark pull request: Log error when spill file wasn't deleted

2015-09-20 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/8843#discussion_r39933300 --- Diff: core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeSorterSpillReader.java --- @@ -73,7 +76,9 @@ public void loadNext() throws

[GitHub] spark pull request: Log error when spill file wasn't deleted

2015-09-20 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/8843#discussion_r39933628 --- Diff: core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeSorterSpillReader.java --- @@ -73,7 +76,9 @@ public void loadNext() throws

[GitHub] spark pull request: [SPARK-10685] [SPARK-8632] [SQL] [PYSPARK] Pyt...

2015-09-20 Thread justinuang
Github user justinuang commented on a diff in the pull request: https://github.com/apache/spark/pull/8833#discussion_r39933648 --- Diff: python/pyspark/sql/functions.py --- @@ -1414,7 +1414,7 @@ def __init__(self, func, returnType, name=None): def _create_judf(self, name):

[GitHub] spark pull request: Log error when spill file wasn't deleted

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141832065 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: Log error when spill file wasn't deleted

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141832056 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: Log error when spill file wasn't deleted

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

[GitHub] spark pull request: Log error when spill file wasn't deleted

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141811849 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: Log error when spill file wasn't deleted

2015-09-20 Thread tedyu
GitHub user tedyu opened a pull request: https://github.com/apache/spark/pull/8843 Log error when spill file wasn't deleted You can merge this pull request into a Git repository by running: $ git pull https://github.com/tedyu/spark master Alternatively you can review and

[GitHub] spark pull request: [SPARK-10626][MLLIB] create java friendly meth...

2015-09-20 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/8782#issuecomment-141815598 @srowen : So the implicit tag is needed because the signature for this one is different (all the other ones are calling concrete types and this is a templated type).

[GitHub] spark pull request: [SPARK-10577] [PySpark] DataFrame hint for bro...

2015-09-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8801#issuecomment-141817299 [Test build #1778 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1778/consoleFull) for PR 8801 at commit

[GitHub] spark pull request: [SPARK-10718] [BUILD] Check License should not...

2015-09-20 Thread rekhajoshm
Github user rekhajoshm commented on the pull request: https://github.com/apache/spark/pull/8842#issuecomment-141820464 thanks @rxin and @srowen for your quick review comments. @rxin i need it for ./dev/run-tests to run correctly. imo nothing under conf/ folder needs to be checked

[GitHub] spark pull request: [SPARK-10317] [Core] Compatibility between his...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8758#issuecomment-141820859 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-10317] [Core] Compatibility between his...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8758#issuecomment-141820868 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: Log error when spill file wasn't deleted

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141823440 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: Log error when spill file wasn't deleted

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141823435 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-10718] [BUILD] Check License should not...

2015-09-20 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/8842#issuecomment-141826632 Running tests including things like RAT checks is usually for preparing a packaging of Spark. If you have `spark-defaults.conf` in there then you are also running a

[GitHub] spark pull request: Log error when spill file wasn't deleted

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141826638 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: Log error when spill file wasn't deleted

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

[GitHub] spark pull request: [SPARK-9852] Let reduce tasks fetch multiple m...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8844#issuecomment-141841907 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-9852] Let reduce tasks fetch multiple m...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8844#issuecomment-141841910 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: Log error when spill file wasn't deleted

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141842474 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: Log error when spill file wasn't deleted

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141842480 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: Log error when spill file wasn't deleted

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

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141843428 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-10721 Log warning when file deletion fai...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141843737 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-10721 Log warning when file deletion fai...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141843745 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-5905] [MLLIB] Note requirements for cer...

2015-09-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8839 --- 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-10686] [ML] Add quantilesCol to AFTSurv...

2015-09-20 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8836#issuecomment-141844411 The changes look good to me. Just want to discuss the semantics to enable quantiles in output. We set a default value for `quantilesCol`, and use `quantileProbabilities`

[GitHub] spark pull request: [SPARK-9642] [ML] [WIP] LinearRegression shoul...

2015-09-20 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/8631#discussion_r39937155 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -493,26 +515,28 @@ private class LeastSquaresAggregator(

[GitHub] spark pull request: [SPARK-10631][Documentation, MLlib, PySpark]Ad...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8834#issuecomment-141850846 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-9642] [ML] [WIP] LinearRegression shoul...

2015-09-20 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/8631#discussion_r39937140 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -520,28 +544,28 @@ private class LeastSquaresAggregator( *

[GitHub] spark pull request: [SPARK-10474][SQL][WIP]Avoid eat out all of th...

2015-09-20 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/8798#issuecomment-141850806 Thank you @andrewor14 , #8827 hits the right point. --- 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-9642] [ML] [WIP] LinearRegression shoul...

2015-09-20 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/8631#discussion_r39937145 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -520,28 +544,28 @@ private class LeastSquaresAggregator( *

[GitHub] spark pull request: [SPARK-10631][Documentation, MLlib, PySpark]Ad...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8834#issuecomment-141850855 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-10474][SQL][WIP]Avoid eat out all of th...

2015-09-20 Thread chenghao-intel
Github user chenghao-intel closed the pull request at: https://github.com/apache/spark/pull/8798 --- 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

[GitHub] spark pull request: [SPARK-10631][Documentation, MLlib, PySpark]Ad...

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

[GitHub] spark pull request: [SPARK-9642] [ML] [WIP] LinearRegression shoul...

2015-09-20 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/8631#discussion_r39937180 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -31,21 +31,30 @@ import org.apache.spark.ml.util.Identifiable

[GitHub] spark pull request: [SPARK-10677] [SQL] [WIP] UnsafeExternalSorter...

2015-09-20 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/8805#discussion_r39937272 --- Diff: core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeExternalSorter.java --- @@ -254,22 +281,25 @@ public int

[GitHub] spark pull request: [SPARK-9642] [ML] [WIP] LinearRegression shoul...

2015-09-20 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/8631#discussion_r39937365 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/LinearRegressionSuite.scala --- @@ -510,4 +513,90 @@ class LinearRegressionSuite extends

[GitHub] spark pull request: [SPARK-9642] [ML] [WIP] LinearRegression shoul...

2015-09-20 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/8631#discussion_r39937357 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/LinearRegressionSuite.scala --- @@ -510,4 +513,90 @@ class LinearRegressionSuite extends

[GitHub] spark pull request: [SPARK-9642] [ML] [WIP] LinearRegression shoul...

2015-09-20 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/8631#discussion_r39937361 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/LinearRegressionSuite.scala --- @@ -510,4 +513,90 @@ class LinearRegressionSuite extends

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141851910 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-10721 Log warning when file deletion fai...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141851916 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-9852] Let reduce tasks fetch multiple m...

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

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

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

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141843429 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10630][SQL] Add a createDataFrame API t...

2015-09-20 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8779#discussion_r39936067 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala --- @@ -477,6 +477,20 @@ class SQLContext(@transient val sparkContext: SparkContext)

[GitHub] spark pull request: [SPARK-5905] [MLLIB] Note requirements for cer...

2015-09-20 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8839#issuecomment-141843685 LGTM. Merged into master. Thanks! --- 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-10721 Log warning when file deletion fai...

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

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

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

[GitHub] spark pull request: [SPARK-10715][ML] Duplicate initialization fla...

2015-09-20 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8837#issuecomment-141844522 Merged into master. Thanks! We can skip JIRA for minor changes like in this PR. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-10715][ML] Duplicate initialization fla...

2015-09-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8837 --- 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-10721 Log warning when file deletion fai...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141844541 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10631][Documentation, MLlib, PySpark]Ad...

2015-09-20 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8834#discussion_r39936250 --- Diff: python/pyspark/mllib/linalg/__init__.py --- @@ -301,11 +301,14 @@ def __reduce__(self): return DenseVector, (self.array.tostring(),)

[GitHub] spark pull request: SPARK-10721 Log warning when file deletion fai...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8843#issuecomment-141844539 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-10631][Documentation, MLlib, PySpark]Ad...

2015-09-20 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8834#discussion_r39936251 --- Diff: python/pyspark/mllib/linalg/__init__.py --- @@ -397,10 +400,16 @@ def squared_distance(self, other): return np.dot(diff, diff)

[GitHub] spark pull request: [SPARK-10718] [BUILD] Check License should not...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8842#issuecomment-141846138 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-10718] [BUILD] Check License should not...

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

[GitHub] spark pull request: [SPARK-10718] [BUILD] Check License should not...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8842#issuecomment-141846139 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: SPARK-10329 Cost RDD in k-means|| initializati...

2015-09-20 Thread HuJiayin
Github user HuJiayin closed the pull request at: https://github.com/apache/spark/pull/8546 --- 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-9642] [ML] [WIP] LinearRegression shoul...

2015-09-20 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/8631#discussion_r39937291 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -598,17 +629,14 @@ private class LeastSquaresCostFun(

[GitHub] spark pull request: [SPARK-10631][Documentation, MLlib, PySpark]Ad...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8834#issuecomment-141851671 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-9642] [ML] [WIP] LinearRegression shoul...

2015-09-20 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/8631#discussion_r39937401 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/LinearRegressionSuite.scala --- @@ -510,4 +513,90 @@ class LinearRegressionSuite extends

[GitHub] spark pull request: [SPARK-10631][Documentation, MLlib, PySpark]Ad...

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

[GitHub] spark pull request: [SPARK-9642] [ML] [WIP] LinearRegression shoul...

2015-09-20 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/8631#discussion_r39937392 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/LinearRegressionSuite.scala --- @@ -510,4 +513,90 @@ class LinearRegressionSuite extends

[GitHub] spark pull request: [SPARK-9642] [ML] [WIP] LinearRegression shoul...

2015-09-20 Thread dbtsai
Github user dbtsai commented on the pull request: https://github.com/apache/spark/pull/8631#issuecomment-141851806 Can you merge the master to resolve the conflicts? Also, add warning in training summary that it ignores the training weights currently (except for the objective trace).

[GitHub] spark pull request: [SPARK-10631][Documentation, MLlib, PySpark]Ad...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8834#issuecomment-141851670 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-10721 Log warning when file deletion fai...

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

[GitHub] spark pull request: [SPARK-9852] Let reduce tasks fetch multiple m...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8844#issuecomment-141852573 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-9852] Let reduce tasks fetch multiple m...

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

[GitHub] spark pull request: [SPARK-9852] Let reduce tasks fetch multiple m...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8844#issuecomment-141852575 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-4131][SQL] support writing data into th...

2015-09-20 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/4380#issuecomment-141853176 @litao-buptsse, i will update this soon thanks. --- 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-10662][DOCS] Code snippets are not prop...

2015-09-20 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/8795#issuecomment-141783385 Your PR is making the whitespace changes which it ideally would not. I am not proposing whitespace changes; you are. It wouldn't make sense to commit the changes

[GitHub] spark pull request: [SPARK-10559] DataFrame schema ArrayType shoul...

2015-09-20 Thread maver1ck
Github user maver1ck commented on the pull request: https://github.com/apache/spark/pull/8712#issuecomment-141810492 I think so. --- 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-10317] [Core] Compatibility between his...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8758#issuecomment-141781233 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-10317] [Core] Compatibility between his...

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

[GitHub] spark pull request: [SPARK-10317] [Core] Compatibility between his...

2015-09-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8758#issuecomment-141781235 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10559] DataFrame schema ArrayType shoul...

2015-09-20 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/8712#issuecomment-141810199 Given the discussion in the JIRA, is this to be closed? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-10662][DOCS] Code snippets are not prop...

2015-09-20 Thread jaceklaskowski
Github user jaceklaskowski commented on the pull request: https://github.com/apache/spark/pull/8795#issuecomment-141779426 Do you want me to split the pull requests to two - one with code formatting in table and another for the *unfortunate* excessive spaces? And what JIRA would that

[GitHub] spark pull request: [SPARK-10535] Sync up API for matrix factoriza...

2015-09-20 Thread smartkiwi
Github user smartkiwi commented on the pull request: https://github.com/apache/spark/pull/8700#issuecomment-141793243 Jenkins please test again --- 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

  1   2   >