Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18323
**[Test build #80112 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80112/testReport)**
for PR 18323 at commit
[`e0478f5`](https://github.com/apache/spark/commit/e0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18749
**[Test build #80111 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80111/testReport)**
for PR 18749 at commit
[`389fc6e`](https://github.com/apache/spark/commit/38
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18337
This seems to have a lot of superfluous change. Why is the mean gradient
better? It is different just by a scale factor. Also disagree about not
clipping predictions but that's separate
---
If your
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18792
---
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 ena
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/18323#discussion_r130526899
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -1186,3 +1186,124 @@ case class BRound(child:
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/18323#discussion_r130526538
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -1186,3 +1186,124 @@ case class BRound(child:
Github user 2ooom commented on the issue:
https://github.com/apache/spark/pull/18783
Thank you @ajbozarth
---
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,
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/18768
I think the document about these unsupported functions is enough for users,
so I'll close this for now. If we have more response from users, we could
revisit this again. Thanks.
---
If your project
Github user maropu closed the pull request at:
https://github.com/apache/spark/pull/18768
---
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 ena
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18792
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80109/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18792
**[Test build #80109 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80109/testReport)**
for PR 18792 at commit
[`29f1108`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18792
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18668
**[Test build #80110 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80110/testReport)**
for PR 18668 at commit
[`9ba8f53`](https://github.com/apache/spark/commit/9b
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18668
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 feature
enabled and wishes so,
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18731#discussion_r130524511
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -347,13 +347,18 @@ class JacksonParser(
U
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18779
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80105/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18779
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18779
**[Test build #80105 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80105/testReport)**
for PR 18779 at commit
[`a5667c8`](https://github.com/apache/spark/commit/a
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18792
LGTM pending Jenkins
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18792
**[Test build #80109 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80109/testReport)**
for PR 18792 at commit
[`29f1108`](https://github.com/apache/spark/commit/29
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18742#discussion_r130523538
--- Diff: python/pyspark/ml/util.py ---
@@ -283,3 +289,124 @@ def numFeatures(self):
Returns the number of features the model was trained o
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18742#discussion_r130519716
--- Diff: python/pyspark/ml/param/__init__.py ---
@@ -375,6 +375,18 @@ def copy(self, extra=None):
that._defaultParamMap = {}
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18742#discussion_r130521214
--- Diff: python/pyspark/ml/util.py ---
@@ -283,3 +289,124 @@ def numFeatures(self):
Returns the number of features the model was trained o
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18742#discussion_r130521964
--- Diff: python/pyspark/ml/util.py ---
@@ -283,3 +289,124 @@ def numFeatures(self):
Returns the number of features the model was trained o
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18742#discussion_r130520335
--- Diff: python/pyspark/ml/util.py ---
@@ -283,3 +289,124 @@ def numFeatures(self):
Returns the number of features the model was trained o
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18742#discussion_r130522066
--- Diff: python/pyspark/ml/util.py ---
@@ -283,3 +289,124 @@ def numFeatures(self):
Returns the number of features the model was trained o
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18792#discussion_r130523581
--- Diff: docs/sql-programming-guide.md ---
@@ -1903,6 +1903,23 @@ releases of Spark SQL.
Hive can optionally merge the small files into fewer large fi
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18792#discussion_r130523169
--- Diff: docs/sql-programming-guide.md ---
@@ -1903,6 +1903,23 @@ releases of Spark SQL.
Hive can optionally merge the small files into fewer large fi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18727
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18727
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80102/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18727
**[Test build #80102 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80102/testReport)**
for PR 18727 at commit
[`1fcbf53`](https://github.com/apache/spark/commit/1
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18746#discussion_r130518147
--- Diff: python/pyspark/ml/base.py ---
@@ -116,3 +121,44 @@ class Model(Transformer):
"""
__metaclass__ = ABCMeta
+
+
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18746#discussion_r130518773
--- Diff: python/pyspark/ml/base.py ---
@@ -116,3 +121,44 @@ class Model(Transformer):
"""
__metaclass__ = ABCMeta
+
+
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18787
**[Test build #80108 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80108/testReport)**
for PR 18787 at commit
[`f906156`](https://github.com/apache/spark/commit/f9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18792
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18792
**[Test build #80107 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80107/testReport)**
for PR 18792 at commit
[`c703d57`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18792
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80107/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18792
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80106/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18792
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18792
**[Test build #80106 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80106/testReport)**
for PR 18792 at commit
[`7d07e6b`](https://github.com/apache/spark/commit/7
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/17419#discussion_r130517322
--- Diff: mllib/src/main/scala/org/apache/spark/ml/stat/Summarizer.scala ---
@@ -0,0 +1,799 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18695#discussion_r130517275
--- Diff: python/pyspark/broadcast.py ---
@@ -139,6 +140,24 @@ def __reduce__(self):
return _from_id, (self._jbroadcast.id(),)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18792
**[Test build #80107 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80107/testReport)**
for PR 18792 at commit
[`c703d57`](https://github.com/apache/spark/commit/c7
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/18792
@gatorsmile ok, fixed.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18792
**[Test build #80106 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80106/testReport)**
for PR 18792 at commit
[`7d07e6b`](https://github.com/apache/spark/commit/7d
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/17419#discussion_r130515498
--- Diff: mllib/src/main/scala/org/apache/spark/ml/stat/Summarizer.scala ---
@@ -0,0 +1,746 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18695#discussion_r130515139
--- Diff: python/pyspark/broadcast.py ---
@@ -139,6 +140,24 @@ def __reduce__(self):
return _from_id, (self._jbroadcast.id(),)
+c
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18695
Will merge this one after few days if there are no more comments from
@holdenk and @davies.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18695
Last question to check if I read correctly. So, the problem is around
`_prepare_for_python_RDD` in `rdd.py` basically because it adds the pickled
ones into the registry when it's pickled but thi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18695
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80104/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18695
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18695
**[Test build #80104 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80104/testReport)**
for PR 18695 at commit
[`d4d1fed`](https://github.com/apache/spark/commit/d
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18695#discussion_r130514254
--- Diff: python/pyspark/broadcast.py ---
@@ -139,6 +140,24 @@ def __reduce__(self):
return _from_id, (self._jbroadcast.id(),)
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18695#discussion_r130514229
--- Diff: python/pyspark/broadcast.py ---
@@ -139,6 +140,24 @@ def __reduce__(self):
return _from_id, (self._jbroadcast.id(),)
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/18786#discussion_r130514013
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2973,15 +2974,51 @@ setMethod("describe",
dataFrame(sdf)
})
+#' summary
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/18786#discussion_r130513953
--- Diff: R/pkg/tests/fulltests/test_sparkSQL.R ---
@@ -2500,8 +2500,15 @@ test_that("describe() and summarize() on a
DataFrame", {
expect_equal(
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/18786#discussion_r130513793
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2973,15 +2974,51 @@ setMethod("describe",
dataFrame(sdf)
})
+#' summary
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/18786#discussion_r130513739
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2973,15 +2974,51 @@ setMethod("describe",
dataFrame(sdf)
})
+#' summary
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/18786#discussion_r130514066
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2973,15 +2974,51 @@ setMethod("describe",
dataFrame(sdf)
})
+#' summary
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18765#discussion_r130513976
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2571,6 +2572,23 @@ private[spark] object Utils extends Logging {
sparkJ
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18605
---
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 ena
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/18605
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 this feature
enabled and wishes so,
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18106
ping @wangyum
---
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
Github user daniellaah commented on the issue:
https://github.com/apache/spark/pull/18337
Try another dataset and you will find the algorithm works well.
---
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 doe
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18792
Thanks for working on it! Just left some minor comments.
---
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 h
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18792#discussion_r130512423
--- Diff: docs/sql-programming-guide.md ---
@@ -1903,6 +1903,25 @@ releases of Spark SQL.
Hive can optionally merge the small files into fewer larg
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18792
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18792
**[Test build #80103 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80103/testReport)**
for PR 18792 at commit
[`1434bde`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18792
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80103/
Test PASSed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18792#discussion_r130512311
--- Diff: docs/sql-programming-guide.md ---
@@ -1903,6 +1903,25 @@ releases of Spark SQL.
Hive can optionally merge the small files into fewer larg
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18779
**[Test build #80105 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80105/testReport)**
for PR 18779 at commit
[`a5667c8`](https://github.com/apache/spark/commit/a5
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18792#discussion_r130512174
--- Diff: docs/sql-programming-guide.md ---
@@ -1903,6 +1903,25 @@ releases of Spark SQL.
Hive can optionally merge the small files into fewer larg
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18792#discussion_r130512125
--- Diff: docs/sql-programming-guide.md ---
@@ -1903,6 +1903,25 @@ releases of Spark SQL.
Hive can optionally merge the small files into fewer larg
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18695
**[Test build #80104 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80104/testReport)**
for PR 18695 at commit
[`d4d1fed`](https://github.com/apache/spark/commit/d4
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18792#discussion_r130511955
--- Diff: docs/sql-programming-guide.md ---
@@ -1903,6 +1903,25 @@ releases of Spark SQL.
Hive can optionally merge the small files into fewer larg
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/18695#discussion_r130511618
--- Diff: python/pyspark/tests.py ---
@@ -858,6 +858,49 @@ def test_multiple_broadcasts(self):
self.assertEqual(N, size)
self.a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18792
**[Test build #80103 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80103/testReport)**
for PR 18792 at commit
[`1434bde`](https://github.com/apache/spark/commit/14
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/18792
@gatorsmile If you get time, could you check this? 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 not hav
.
This pr relates to #18768 and #18527.
## How was this patch tested?
N/A
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/maropu/spark HOTFIX-20170731
Alternatively you can review and apply these changes as the patch at:
https
Github user 10110346 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18779#discussion_r130507794
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1010,7 +1014,16 @@ class Analyzer(
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18779#discussion_r130506005
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1010,7 +1014,16 @@ class Analyzer(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18790
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80101/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18790
**[Test build #80101 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80101/testReport)**
for PR 18790 at commit
[`0a837ae`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18789
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18789
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80097/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18789
**[Test build #80097 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80097/testReport)**
for PR 18789 at commit
[`3ee0e92`](https://github.com/apache/spark/commit/3
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18779#discussion_r130505545
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1010,7 +1014,16 @@ class Analyzer(
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18779#discussion_r130505351
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1010,7 +1014,16 @@ class Analyzer(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18727
**[Test build #80102 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80102/testReport)**
for PR 18727 at commit
[`1fcbf53`](https://github.com/apache/spark/commit/1f
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18779#discussion_r130504819
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1010,7 +1014,16 @@ class Analyzer(
Github user 10110346 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18779#discussion_r130504796
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1010,7 +1014,16 @@ class Analyzer(
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/18727
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 feature
enabled and wishes so, or
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18791
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 project does not have this feat
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18695#discussion_r130504060
--- Diff: python/pyspark/tests.py ---
@@ -858,6 +858,49 @@ def test_multiple_broadcasts(self):
self.assertEqual(N, size)
self.assert
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18695
LGTM except for one minor comment.
---
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 a
GitHub user ericvandenbergfb opened a pull request:
https://github.com/apache/spark/pull/18791
[SPARK-21571][WEB UI] Spark history server leaves incomplete or unreaâ¦
â¦dable history files
around forever.
Fix logic
1. checkForLogs excluded 0-size files so they
Github user 10110346 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18779#discussion_r130503562
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1010,7 +1014,16 @@ class Analyzer(
Github user mpjlu commented on the issue:
https://github.com/apache/spark/pull/18748
Did you test the performance of this, I tested the performance of MLLIB
recommendForUserSubset some days ago, the performance is not good. Suppose the
time of recommendForAll is 35s, recommend for 1/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18787
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 project does not have this feature
e
1 - 100 of 352 matches
Mail list logo
