Github user wongxingjun commented on the issue:
https://github.com/apache/spark/pull/14437
Yeah well that might be my OCD.
---
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 user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14406
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/14416
Merged to master/2.0
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/14380
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/14406
Merged to master/2.0
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/14380
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, or i
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14420
I'll wait a moment to see if @JoshRosen has concerns about it. The original
change was approved, for branch 1.6, but in re-reading, I wonder if it was only
accidentally discussed as 1.6-only.
---
I
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14432
This is WontFix per the JIRA, please close the PR
---
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 fe
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14436
OK, it doesn't hurt. OK to merge if this seems to help your case.
---
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 user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14438
**[Test build #63077 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63077/consoleFull)**
for PR 14438 at commit
[`2a480ca`](https://github.com/apache/spark/commit/2
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14437
I think this is too trivial to bother with. The structure isn't parallel
but the fragment isn't exactly ungrammatical
---
If your project is set up for it, you can reply to this email and have your
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14438
Nice, does it work? :)
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/14438
Jenkins add to whitelist
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/14438
Jenkins test this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user nsyca commented on the issue:
https://github.com/apache/spark/pull/14411
@hvanhovell,
Thank you for your comment. There are quite a few patterns being
blacklisted already, such as correlation under set operators (UNION, EXCEPT,
INTERSECT), correlation outside of W
GitHub user lins05 opened a pull request:
https://github.com/apache/spark/pull/14438
[SPARK-16822][DOC] Support latex in scaladoc.
## What changes were proposed in this pull request?
Support using latex in scaladoc by adding MathJax javascript to the js
template.
#
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/12896
Hm, let me go through the logic with you one more time here. Isn't it
better in theory to fix the model to not return NaN, but rather return _some_
default answer, even if it's "0" or equivalent? thi
Github user lins05 commented on the issue:
https://github.com/apache/spark/pull/14438
@srowen could you take look?
---
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 wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14438
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14400
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/14400
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63076/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14400
**[Test build #63076 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63076/consoleFull)**
for PR 14400 at commit
[`e0549a9`](https://github.com/apache/spark/commit/
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/14212
The fix LGTM - I'm in two minds about whether this example is "good"
(illustrating using both `ml` and `mllib` APIs together) or "bad" (for the same
reason, since there is LDA available in `ml`).
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r72971469
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -423,18 +432,35 @@ private[spark] clas
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r72971304
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -423,18 +432,35 @@ private[spark] clas
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14437
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 wongxingjun opened a pull request:
https://github.com/apache/spark/pull/14437
fix 2 typos
## What changes were proposed in this pull request?
2 typos are fixed.
## How was this patch tested?
(Please explain how this patch was tested. E.g. u
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11157
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63075/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11157
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11157
**[Test build #63075 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63075/consoleFull)**
for PR 11157 at commit
[`2493d2a`](https://github.com/apache/spark/commit/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14400
we have an individual style check script: `./dev/scalastyle`
---
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 no
Github user eyalfa commented on the issue:
https://github.com/apache/spark/pull/14400
@cloud-fan, just for notice for the next patches: is there a way to run mvn
in a mode that fails on style issues? it could save a lot of round-trips if it
existed.
---
If your project is set up for
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14400
**[Test build #63076 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63076/consoleFull)**
for PR 14400 at commit
[`e0549a9`](https://github.com/apache/spark/commit/e
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14400
https://github.com/apache/spark/pull/14400#discussion_r72897142
---
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 user eyalfa commented on the issue:
https://github.com/apache/spark/pull/14400
@cloud-fan, any update on this?
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11157
**[Test build #63075 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63075/consoleFull)**
for PR 11157 at commit
[`2493d2a`](https://github.com/apache/spark/commit/2
Github user junyangq commented on a diff in the pull request:
https://github.com/apache/spark/pull/14258#discussion_r72950957
--- Diff: R/pkg/R/install.R ---
@@ -36,7 +36,7 @@
#' \code{without-hadoop}.
#'
#' @param hadoopVersion Version of Hadoop to install. Default i
Github user junyangq commented on a diff in the pull request:
https://github.com/apache/spark/pull/14258#discussion_r72950905
--- Diff: R/pkg/R/install.R ---
@@ -0,0 +1,232 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14435
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14435
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63074/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14435
**[Test build #63074 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63074/consoleFull)**
for PR 14435 at commit
[`1392c8a`](https://github.com/apache/spark/commit/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14391
cc @mengxr what's the plan for UDT? Is it a public API now?
---
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 user phalodi commented on the issue:
https://github.com/apache/spark/pull/14436
@srowen @rxin Hey guyss i make changes as you suggested but yeah it not
exactly what you suggested, i change in example which is also reflect in
documentation, so please review it and give give your
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14436
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 phalodi opened a pull request:
https://github.com/apache/spark/pull/14436
modify java example which is also refelect in documentation exmaple
## What changes were proposed in this pull request?
Modify java example which is also reflect in document.
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14363
Well, Spark SQL is not announced to be fully compatible with hive, I think
it's reasonable to have some issues. cc @rxin @yhuai should we fix this?
---
If your project is set up for it, you can r
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14355
---
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 hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14355
LGTM - merging to master. Thanks for working on this!
---
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 user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14155
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/14155
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63073/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14155
**[Test build #63073 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63073/consoleFull)**
for PR 14155 at commit
[`9ae7a71`](https://github.com/apache/spark/commit/
Github user lianhuiwang commented on the issue:
https://github.com/apache/spark/pull/14363
@cloud-fan There is a case that i met. The varchar(length)/char(length)
type is not a String Type. But now SparkSQL consider them a string type. So
there are different result with the following
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14435
**[Test build #63074 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63074/consoleFull)**
for PR 14435 at commit
[`1392c8a`](https://github.com/apache/spark/commit/1
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/14435
[SPARK-16756][SQL][WIP] Add `sql` function to LogicalPlan and `NonSQLPlan`
trait
## What changes were proposed in this pull request?
This PR is a part of
[SPARK-16576](https://issues.apa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14434
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63072/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14434
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/14434
**[Test build #63072 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63072/consoleFull)**
for PR 14434 at commit
[`ce0b9fe`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11673
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63070/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11673
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/11673
**[Test build #63070 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63070/consoleFull)**
for PR 11673 at commit
[`ce0b9fe`](https://github.com/apache/spark/commit/
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/14401
cc @rxin
---
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
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/11601
@hhbyyh could you update the since annotations to target `2.1.0`?
@jkbradley if you have a chance to review would be great. Thanks!
---
If your project is set up for it, you can reply to this email
Github user lw-lin commented on the issue:
https://github.com/apache/spark/pull/14118
Here are some findings as I dug a little:
1. Since https://github.com/databricks/spark-csv/pull/102(Jul, 2015), we
would cast `""` as `null` for all types other than strings. For strings, `""
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/12896
cc also @xsankar who was interested on the related JIRAs.
---
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 user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14363
Your concern is valid. We are missing the test cases for verifying these
scenarios.
I saw a discussion in a wechat group about the issue in integration between
Hive and Spark. They are
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12896#discussion_r72931611
--- Diff: python/pyspark/ml/recommendation.py ---
@@ -332,6 +338,20 @@ def getFinalStorageLevel(self):
"""
return self.getOrDefault(s
301 - 366 of 366 matches
Mail list logo
