Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/17457
cc @cloud-fan @gatorsmile
---
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 w
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17452#discussion_r108407452
--- Diff: R/check-cran.sh ---
@@ -20,14 +20,14 @@
set -o pipefail
set -e
-FWDIR="$(cd `dirname "${BASH_SOURCE[0]}"`; pwd)"
-pushd $FWD
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17453
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/17453
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75305/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17453
**[Test build #75305 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75305/testReport)**
for PR 17453 at commit
[`058dd74`](https://github.com/apache/spark/commit/0
Github user zuotingbing commented on a diff in the pull request:
https://github.com/apache/spark/pull/17452#discussion_r108400132
--- Diff: R/check-cran.sh ---
@@ -20,14 +20,14 @@
set -o pipefail
set -e
-FWDIR="$(cd `dirname "${BASH_SOURCE[0]}"`; pwd)"
-pushd
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17428
---
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/17428
Merging to 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 not have this feature
enabled and w
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17453
**[Test build #75310 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75310/testReport)**
for PR 17453 at commit
[`e2249c0`](https://github.com/apache/spark/commit/e2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17456
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 srowen commented on the issue:
https://github.com/apache/spark/pull/17456
@lnmohankumar please close 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 lnmohankumar opened a pull request:
https://github.com/apache/spark/pull/17456
Branch 2.1
## What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
## How was this patch tested?
(Please explain how this pa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17428
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75308/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17428
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/17428
**[Test build #75308 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75308/testReport)**
for PR 17428 at commit
[`bf7d202`](https://github.com/apache/spark/commit/b
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17453#discussion_r108392855
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/plans/PlanTest.scala
---
@@ -127,7 +127,8 @@ abstract class PlanTest extends SparkFunSuit
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17453#discussion_r108391785
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/plans/PlanTest.scala
---
@@ -127,7 +127,8 @@ abstract class PlanTest extends SparkFunSuit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17453#discussion_r108389791
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/plans/PlanTest.scala
---
@@ -127,7 +127,8 @@ abstract class PlanTest extends SparkFu
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/17446
> Ideally, our optimizer rule BooleanSimplification should be able to
remove the null literal, like what we did for true and false literals. Here, I
think we should just assume our filter estimation d
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/17428
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 wish
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17428
**[Test build #75309 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75309/testReport)**
for PR 17428 at commit
[`0cc6137`](https://github.com/apache/spark/commit/0c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17454
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75307/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17454
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/17454
**[Test build #75307 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75307/testReport)**
for PR 17454 at commit
[`29a0143`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15332
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75306/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15332
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/15332
**[Test build #75306 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75306/testReport)**
for PR 15332 at commit
[`796b6b1`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17434
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75302/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17434
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/17434
**[Test build #75302 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75302/testReport)**
for PR 17434 at commit
[`eafb626`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17406
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75304/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17406
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/17406
**[Test build #75304 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75304/testReport)**
for PR 17406 at commit
[`9ddd515`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17406
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/17406
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75303/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17406
**[Test build #75303 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75303/testReport)**
for PR 17406 at commit
[`83f5d6e`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17434
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/17434
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75301/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17434
**[Test build #75301 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75301/testReport)**
for PR 17434 at commit
[`eb3337e`](https://github.com/apache/spark/commit/e
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17428#discussion_r108373968
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/QueryPlan.scala
---
@@ -289,6 +289,9 @@ abstract class QueryPlan[PlanType <:
Query
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17428
**[Test build #75308 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75308/testReport)**
for PR 17428 at commit
[`bf7d202`](https://github.com/apache/spark/commit/bf
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17428#discussion_r108373273
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/JoinOptimizationSuite.scala
---
@@ -123,6 +136,31 @@ class JoinOptimizationSuit
Github user liujianhuiouc commented on the issue:
https://github.com/apache/spark/pull/17208
@squito update the no-args ManualClock constructor with initialized time
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If you
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17449
It seems reasonable to me, if we already support the flag, even. It's
unfortunate we have to call log4j directly but I don't know of a way around
that.
---
If your project is set up for it, you can
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17452#discussion_r108363062
--- Diff: R/check-cran.sh ---
@@ -40,7 +40,7 @@ fi
if [ -d "$SPARK_JARS_DIR" ]; then
# Build a zip file containing the source package with v
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17452#discussion_r108360431
--- Diff: R/check-cran.sh ---
@@ -20,14 +20,14 @@
set -o pipefail
set -e
-FWDIR="$(cd `dirname "${BASH_SOURCE[0]}"`; pwd)"
-pushd $FWD
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17455
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 okoethibm opened a pull request:
https://github.com/apache/spark/pull/17455
[Spark-20044][Web UI] Support Spark UI behind front-end reverse proxy using
a path prefix
## What changes were proposed in this pull request?
Allow to run the Spark web UI behind a rever
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17454
**[Test build #75307 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75307/testReport)**
for PR 17454 at commit
[`29a0143`](https://github.com/apache/spark/commit/29
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17454
cc @liancheng
---
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 cloud-fan opened a pull request:
https://github.com/apache/spark/pull/17454
[SPARK-20125][SQL] Dataset of type option of map does not work
## What changes were proposed in this pull request?
When we build the deserializer expression for map type, we will use
`St
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/17451#discussion_r108359620
--- Diff: python/pyspark/ml/tests.py ---
@@ -389,6 +389,21 @@ def test_word2vec_param(self):
# Check windowSize is set properly
self
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/17451#discussion_r108359474
--- Diff: python/pyspark/ml/feature.py ---
@@ -2674,6 +2674,18 @@ def findSynonyms(self, word, num):
word = _convert_to_vector(word)
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/17324#discussion_r108358270
--- Diff: examples/src/main/python/ml/imputer_example.py ---
@@ -0,0 +1,51 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/17324#discussion_r108358002
--- Diff: examples/src/main/python/ml/imputer_example.py ---
@@ -0,0 +1,51 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/17324#discussion_r108357950
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaImputerExample.java ---
@@ -0,0 +1,72 @@
+/*
+ * Licensed to the Apache Software Fo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15332
**[Test build #75306 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75306/testReport)**
for PR 15332 at commit
[`796b6b1`](https://github.com/apache/spark/commit/79
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/17324#discussion_r108357089
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaImputerExample.java ---
@@ -0,0 +1,72 @@
+/*
+ * Licensed to the Apache Software Fo
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/17324#discussion_r108356925
--- Diff: examples/src/main/python/ml/imputer.py ---
@@ -0,0 +1,46 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17453
**[Test build #75305 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75305/testReport)**
for PR 17453 at commit
[`058dd74`](https://github.com/apache/spark/commit/05
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/17453
cc @cloud-fan please review
---
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 wishe
GitHub user wzhfy opened a pull request:
https://github.com/apache/spark/pull/17453
[SPARK-20124][SQL] Join reorder should keep the same order of final project
attributes
## What changes were proposed in this pull request?
Join reorder algorithm should keep exactly the same
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17448
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17406
**[Test build #75304 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75304/testReport)**
for PR 17406 at commit
[`9ddd515`](https://github.com/apache/spark/commit/9d
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/17448
LGTM - merging to 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 not have this feature
enable
Github user superbobry commented on a diff in the pull request:
https://github.com/apache/spark/pull/17230#discussion_r108351585
--- Diff: core/src/main/scala/org/apache/spark/rdd/PipedRDD.scala ---
@@ -198,17 +183,114 @@ private[spark] class PipedRDD[T: ClassTag](
val
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17434
**[Test build #75302 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75302/testReport)**
for PR 17434 at commit
[`eafb626`](https://github.com/apache/spark/commit/ea
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17406
**[Test build #75303 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75303/testReport)**
for PR 17406 at commit
[`83f5d6e`](https://github.com/apache/spark/commit/83
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17406#discussion_r108349691
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/DataType.scala ---
@@ -103,6 +104,12 @@ object DataType {
def fromJson(json: St
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17452
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 zuotingbing opened a pull request:
https://github.com/apache/spark/pull/17452
[SPARK-20123]$SPARK_HOME variable might have spaces in it(e.g. $SPARKâ¦
JIRA Issue: https://issues.apache.org/jira/browse/SPARK-20123
## What changes were proposed in this pull request
Github user szalai1 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17435#discussion_r108348472
--- Diff: python/pyspark/sql/types.py ---
@@ -57,7 +57,25 @@ def __ne__(self, other):
@classmethod
def typeName(cls):
-re
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17434
**[Test build #75301 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75301/testReport)**
for PR 17434 at commit
[`eb3337e`](https://github.com/apache/spark/commit/eb
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17451
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75300/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17451
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/17451
**[Test build #75300 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75300/testReport)**
for PR 17451 at commit
[`f84be09`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17451
**[Test build #75300 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75300/testReport)**
for PR 17451 at commit
[`f84be09`](https://github.com/apache/spark/commit/f8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17394
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75295/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17394
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/17406
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/17208
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75299/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17406
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75296/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17450
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/17450
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75298/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17208
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 keypointt opened a pull request:
https://github.com/apache/spark/pull/17451
[SPARK-19866][ML][PySpark] Add local version of Word2Vec findSynonyms for
spark.ml: Python API
https://issues.apache.org/jira/browse/SPARK-19866
## What changes were proposed in this pul
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/17355
Hope you feel better soon @shaneknapp :)
---
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
en
301 - 387 of 387 matches
Mail list logo
