Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18945
**[Test build #82067 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82067/testReport)**
for PR 18945 at commit
[`6e16cd8`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18945
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82067/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18945
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19281
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19281
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19281
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.ap
Github user jaceklaskowski commented on the issue:
https://github.com/apache/spark/pull/19261
@rxin @gatorsmile Let me ask you a very similar question then, why does
`CurrentDate` operator has the optional timezone parameter? What's the purpose?
Wouldn't that answer your questions?
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19319
I'd go with this PR / approach. This approach and PR look pretty good. Let
me help double check this tonight.
---
-
To unsub
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19320
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19319
**[Test build #82069 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82069/testReport)**
for PR 19319 at commit
[`779eb40`](https://github.com/apache/spark/commit/77
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18015#discussion_r140421501
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -61,7 +59,37 @@ private[ui] class AllExecutionsPage(p
GitHub user guoxiaolongzte opened a pull request:
https://github.com/apache/spark/pull/19320
[SPARK-22099] The 'job ids' list style needs to be changed in the SQL page.
## What changes were proposed in this pull request?
The 'job ids' list style needs to be changed in the SQ
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19319
**[Test build #82068 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82068/testReport)**
for PR 19319 at commit
[`e12f576`](https://github.com/apache/spark/commit/e1
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18945#discussion_r140420898
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1760,13 +1760,39 @@ def toPandas(self):
"if using spark.sql.execution.arrow.enabl
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/19319
[SPARK-21766][PySpark][SQL] DataFrame toPandas() raises ValueError with
nullable int columns
## What changes were proposed in this pull request?
When calling `DataFrame.toPandas()` (without
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18945
**[Test build #82067 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82067/testReport)**
for PR 18945 at commit
[`6e16cd8`](https://github.com/apache/spark/commit/6e
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19303
IIUC, if there's no core left, requesting new executors should be a no-op,
am I right? So there should be no problem even without your fix?
From your patch, it looks like you're putting st
Github user logannc commented on a diff in the pull request:
https://github.com/apache/spark/pull/18945#discussion_r140419964
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1760,13 +1760,39 @@ def toPandas(self):
"if using spark.sql.execution.arrow.enab
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18945
Hey @logannc, let's don't make it complicated for now and go with their
ways first - https://github.com/apache/spark/pull/18945#discussion_r134033952
and https://github.com/apache/spark/pull/189
Github user daweicheng closed the pull request at:
https://github.com/apache/spark/pull/19279
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18945
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/18945
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82066/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18945
**[Test build #82066 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82066/testReport)**
for PR 18945 at commit
[`d93a203`](https://github.com/apache/spark/commit/d
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18945#discussion_r140419255
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1760,13 +1760,39 @@ def toPandas(self):
"if using spark.sql.execution.arrow.enabl
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18945
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82063/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18945
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/18945
**[Test build #82063 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82063/testReport)**
for PR 18945 at commit
[`bd25923`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18945
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/18945
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82065/
Test FAILed.
---
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19315
@animenon can you please fix the PR title like what other PR did. Also is
this only for better readability or do you fix any other issue? IMO, I found
that previous txt is more readable than your
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18015
**[Test build #82064 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82064/testReport)**
for PR 18015 at commit
[`21e2c31`](https://github.com/apache/spark/commit/21
Github user logannc commented on the issue:
https://github.com/apache/spark/pull/18945
I've continued to use @HyukjinKwon 's suggestion because it should be more
performant and is capable of handling it without loss of precision. I believe
I've addressed your concerns by only changing
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18945
**[Test build #82063 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82063/testReport)**
for PR 18945 at commit
[`bd25923`](https://github.com/apache/spark/commit/bd
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19301#discussion_r140416279
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala
---
@@ -72,11 +74,19 @@ object AggregateExpres
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/18015
Yes, I'm fine with it. @ajbozarth would you please take another look on
this PR? Thanks.
---
-
To unsubscribe, e-mail: reviews
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/18015
Jenkins, retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user DaimonPl commented on the issue:
https://github.com/apache/spark/pull/16578
@mallman how about adding comment explaining why such workaround was done +
bug number in parquet-mr ? So in future once that bug is fixed, code can be
cleaned.
Also maybe it's time to rem
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/18015#discussion_r140416046
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -61,7 +59,37 @@ private[ui] class AllExecutionsPage(pa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18945
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/18945
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82062/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18945
**[Test build #82062 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82062/testReport)**
for PR 18945 at commit
[`6e248dd`](https://github.com/apache/spark/commit/6
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18945#discussion_r140415073
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1761,12 +1761,37 @@ def toPandas(self):
raise ImportError("%s\n%s" % (e.message, m
Github user logannc commented on a diff in the pull request:
https://github.com/apache/spark/pull/18945#discussion_r140414783
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1761,12 +1761,37 @@ def toPandas(self):
raise ImportError("%s\n%s" % (e.message, msg))
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18945
**[Test build #82062 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82062/testReport)**
for PR 18945 at commit
[`6e248dd`](https://github.com/apache/spark/commit/6e
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18945#discussion_r140414202
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1761,12 +1761,37 @@ def toPandas(self):
raise ImportError("%s\n%s" % (e.message, msg))
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18945#discussion_r140414042
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1761,12 +1761,37 @@ def toPandas(self):
raise ImportError("%s\n%s" % (e.message, msg))
Github user logannc commented on a diff in the pull request:
https://github.com/apache/spark/pull/18945#discussion_r140413579
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1761,12 +1761,37 @@ def toPandas(self):
raise ImportError("%s\n%s" % (e.message, msg))
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19204
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/19204
Merged into master, thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18945
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/18945
**[Test build #82061 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82061/testReport)**
for PR 18945 at commit
[`14f36c3`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18945
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82061/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18945
**[Test build #82061 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82061/testReport)**
for PR 18945 at commit
[`14f36c3`](https://github.com/apache/spark/commit/14
Github user logannc commented on a diff in the pull request:
https://github.com/apache/spark/pull/18945#discussion_r140412857
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1761,12 +1761,37 @@ def toPandas(self):
raise ImportError("%s\n%s" % (e.message, msg))
Github user logannc commented on a diff in the pull request:
https://github.com/apache/spark/pull/18945#discussion_r140412745
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1761,12 +1761,37 @@ def toPandas(self):
raise ImportError("%s\n%s" % (e.message, msg))
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18945#discussion_r140412632
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1761,12 +1761,37 @@ def toPandas(self):
raise ImportError("%s\n%s" % (e.message, msg))
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18945
**[Test build #82060 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82060/testReport)**
for PR 18945 at commit
[`b313a3b`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18945
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/18945
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82060/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18945
**[Test build #82060 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82060/testReport)**
for PR 18945 at commit
[`b313a3b`](https://github.com/apache/spark/commit/b3
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19229#discussion_r140412254
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -2102,6 +2102,55 @@ class Dataset[T] private[sql](
}
/**
+
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19314
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19229
ping @zhengruifeng @WeichenXu123 Any more comments on this? Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/19314
Thanks! Merging to master and branch-2.2
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comman
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19290
I initially did this, for example,
```
\href{https://spark.apache.org/docs/latest/sparkr.html#data-type-mapping-between-
r-and-spark}{Spark Data Types} for available data types.
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19290
Doh, you mean the current status. Yes, I checked.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addi
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19312#discussion_r140410448
--- Diff: dev/create-release/release-build.sh ---
@@ -95,6 +95,28 @@ if [ -z "$SPARK_VERSION" ]; then
| grep -v INFO | grep -v WARNING | grep -v
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19318
**[Test build #82059 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82059/testReport)**
for PR 19318 at commit
[`efb0fe9`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19318
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82059/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19318
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19290
btw, could you check if haven't already, if `nolint` around the `http`
link, roxygen is going to handle that correctly?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19318
**[Test build #82059 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82059/testReport)**
for PR 19318 at commit
[`efb0fe9`](https://github.com/apache/spark/commit/ef
GitHub user VinceShieh opened a pull request:
https://github.com/apache/spark/pull/19318
[SPARK-22096][ML] use aggregateByKeyLocally in feature frequency calcâ¦
## What changes were proposed in this pull request?
NaiveBayes currently takes aggreateByKey followed by a collec
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19122
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/19122
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82058/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19122
**[Test build #82058 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82058/testReport)**
for PR 19122 at commit
[`3464dfe`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19317
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/19317
cc @VinceShieh
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
GitHub user ConeyLiu opened a pull request:
https://github.com/apache/spark/pull/19317
[SPARK-22098][CORE] Add new method aggregateByKeyLocally in RDD
## What changes were proposed in this pull request?
https://issues.apache.org/jira/browse/SPARK-22096
NaiveBayes cu
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/19316
@cloud-fan Pls take a look. Thanks a lot.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comm
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19312
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82056/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19312
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/19312
**[Test build #82056 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82056/testReport)**
for PR 19312 at commit
[`aa4cbf6`](https://github.com/apache/spark/commit/a
Github user ConeyLiu commented on a diff in the pull request:
https://github.com/apache/spark/pull/19316#discussion_r140408246
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -387,11 +387,18 @@ private[spark] class MemoryStore(
// the
Github user ConeyLiu commented on a diff in the pull request:
https://github.com/apache/spark/pull/19316#discussion_r140408116
--- Diff:
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
@@ -387,11 +387,18 @@ private[spark] class MemoryStore(
// the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19316
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user ConeyLiu opened a pull request:
https://github.com/apache/spark/pull/19316
[SPARK-22097][CORE]Call serializationStream.close after we requested enough
memory
## What changes were proposed in this pull request?
Current code, we close the `serializationStream
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/19168
Sorry for replying so late.
I add some benchmark testing for this pr @kiszk .
And @jerryshao could you help review this pr?Thanks
```
Running benchmark: Benchmark fetch before vs after
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19278
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/19278
**[Test build #82057 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82057/testReport)**
for PR 19278 at commit
[`8f78f59`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19278
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82057/
Test PASSed.
---
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/19278
@jkbradley Sure I tested the backwards compatibility. Part of the reason I
changed into `DefaultParamReader.getAndSetParams` is for backwards
compatibility.
---
-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19122
**[Test build #82058 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82058/testReport)**
for PR 19122 at commit
[`3464dfe`](https://github.com/apache/spark/commit/34
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19122#discussion_r140402700
--- Diff: python/pyspark/ml/tests.py ---
@@ -836,6 +836,27 @@ def test_save_load_simple_estimator(self):
loadedModel = CrossValidatorModel.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19315
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user animenon opened a pull request:
https://github.com/apache/spark/pull/19315
Updated english.txt word ordering
Ordered alphabetically, for better readability.
## What changes were proposed in this pull request?
Alphabetical ordering of the stop words.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19314
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82055/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19314
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/19314
**[Test build #82055 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82055/testReport)**
for PR 19314 at commit
[`a4a02a6`](https://github.com/apache/spark/commit/a
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/13794
cc @srowen Can you help close this ? We won't need this feature for now.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@s
1 - 100 of 422 matches
Mail list logo
