Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21495
**[Test build #91653 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91653/testReport)**
for PR 21495 at commit
[`82ca5f6`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21506
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21506
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91651/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21506
**[Test build #91651 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91651/testReport)**
for PR 21506 at commit
[`3d0e23f`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91654/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20208
**[Test build #91654 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91654/testReport)**
for PR 20208 at commit
[`a8026b8`](https://github.com/apache/spark/commit/a
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21505#discussion_r194365155
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala
---
@@ -111,6 +113,23 @@ object DateTimeUtils {
compute
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21525
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91655/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21525
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21525
**[Test build #91655 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91655/testReport)**
for PR 21525 at commit
[`18b5b25`](https://github.com/apache/spark/commit/1
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21520
@kiszk Seems good to me. I will go to create and use those APIs in pieces
of PRs.
---
-
To unsubscribe, e-mail: reviews-unsubscr.
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21501#discussion_r194346755
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/StopWordsRemoverSuite.scala ---
@@ -65,6 +65,56 @@ class StopWordsRemoverSuite extends MLTest with
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21501#discussion_r194346431
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/StopWordsRemoverSuite.scala ---
@@ -65,6 +65,56 @@ class StopWordsRemoverSuite extends MLTest with
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21501#discussion_r194343283
--- Diff: python/pyspark/ml/feature.py ---
@@ -2582,25 +2582,31 @@ class StopWordsRemover(JavaTransformer,
HasInputCol, HasOutputCol, JavaMLReadabl
Github user dongjinleekr commented on a diff in the pull request:
https://github.com/apache/spark/pull/21501#discussion_r194342986
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StopWordsRemover.scala ---
@@ -84,7 +86,28 @@ class StopWordsRemover @Since("1.5.0") (@Since
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21501#discussion_r194341627
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/StopWordsRemover.scala ---
@@ -84,7 +86,28 @@ class StopWordsRemover @Since("1.5.0") (@Since("1.5.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21526
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21526
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user caneGuy opened a pull request:
https://github.com/apache/spark/pull/21526
[SPARK-24515][CORE] No need to warning when output commit coordination
enabled
## What changes were proposed in this pull request?
No need to warning user when output commit coordination e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21525
**[Test build #91655 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91655/testReport)**
for PR 21525 at commit
[`18b5b25`](https://github.com/apache/spark/commit/18
Github user dongjinleekr commented on the issue:
https://github.com/apache/spark/pull/21501
Finally, I made it work by adding `StopWordsRemover.getDefaultLocale`! How
about this approach? @mengxr @viirya
---
-
To un
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21525
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21525
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user dongjinleekr opened a pull request:
https://github.com/apache/spark/pull/21525
[SPARK-24513][ML] Attribute support in UnaryTransformer
## What changes were proposed in this pull request?
This PR adds Metadata support in `UnaryTransformer`, as a preliminary work
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21501
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91652/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21501
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21501
**[Test build #91652 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91652/testReport)**
for PR 21501 at commit
[`80d664f`](https://github.com/apache/spark/commit/8
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21505#discussion_r194325153
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala
---
@@ -111,6 +113,23 @@ object DateTimeUtils {
comput
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20887#discussion_r194321737
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -2792,4 +2793,40 @@ class SQLQuerySuite extends QueryTest with
Shar
Github user ssonker commented on a diff in the pull request:
https://github.com/apache/spark/pull/21505#discussion_r194319491
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala
---
@@ -111,6 +113,23 @@ object DateTimeUtils {
compu
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20208
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3890/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21495
**[Test build #91653 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91653/testReport)**
for PR 21495 at commit
[`82ca5f6`](https://github.com/apache/spark/commit/82
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20208
**[Test build #91654 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91654/testReport)**
for PR 20208 at commit
[`a8026b8`](https://github.com/apache/spark/commit/a8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21501
**[Test build #91652 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91652/testReport)**
for PR 21501 at commit
[`80d664f`](https://github.com/apache/spark/commit/80
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21495
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21495
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3889/
Tes
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/21495
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21506
**[Test build #91651 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91651/testReport)**
for PR 21506 at commit
[`3d0e23f`](https://github.com/apache/spark/commit/3d
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21506
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21505#discussion_r194309823
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala
---
@@ -111,6 +113,23 @@ object DateTimeUtils {
comput
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/21510#discussion_r194309780
--- Diff: core/src/main/scala/org/apache/spark/ui/WebUI.scala ---
@@ -101,12 +101,12 @@ private[spark] abstract class WebUI(
}
/**
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/21514#discussion_r194308977
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/StandaloneSchedulerBackend.scala
---
@@ -100,7 +100,7 @@ private[spark] class Standalone
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21495
**[Test build #91650 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91650/testReport)**
for PR 21495 at commit
[`82ca5f6`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21506
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91649/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21495
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91650/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21506
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21495
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21506
**[Test build #91649 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91649/testReport)**
for PR 21506 at commit
[`3d0e23f`](https://github.com/apache/spark/commit/3
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21427
How about we just start to have configurations classified for each version
if it sounds better to have configurations for each behaviour change? For
example, we could have postfix like spark_23
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21523
I think here the problem is that `filters` are only added for those handler
who added before server starts.
This may not be the only problem for `ThriftserverTab`, I guess `SQLTable`
or `
501 - 552 of 552 matches
Mail list logo