Github user yangw1234 commented on the issue:
https://github.com/apache/spark/pull/16820
@hvanhovell thanks for your review. Whole stage code generation seems fine
and unit test is added.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16815
**[Test build #72435 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72435/testReport)**
for PR 16815 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16820
**[Test build #72451 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72451/testReport)**
for PR 16820 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16787
**[Test build #72452 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72452/testReport)**
for PR 16787 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16815
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16815
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72436/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16815
**[Test build #3558 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3558/testReport)**
for PR 16815 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16815
**[Test build #72436 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72436/testReport)**
for PR 16815 at commit
Github user squito commented on the issue:
https://github.com/apache/spark/pull/15237
merged to master, thanks @erenavsarogullari
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15237
---
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
Github user valek commented on the issue:
https://github.com/apache/spark/pull/16818
> Just make sure we can construct a range frame that respects longs, and
throw an error for row frames.
This seems totally reasonable
---
If your project is set up for it, you can reply to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16787
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72449/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16787
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16787
**[Test build #72449 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72449/testReport)**
for PR 16787 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16820
**[Test build #72450 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72450/testReport)**
for PR 16820 at commit
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/16818
@hvanhovell Thanks for your suggestions, it is just what I failed to notice
or consider.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user zhoucen closed the pull request at:
https://github.com/apache/spark/pull/16816
---
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
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16387
@samkum Any update?
---
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 windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/16787#discussion_r99588025
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -841,5 +841,6 @@ private[client] class Shim_v1_2 extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16787
**[Test build #72449 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72449/testReport)**
for PR 16787 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16386
Sorry, I missed the ping. Will review it tonight.
---
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 a diff in the pull request:
https://github.com/apache/spark/pull/16787#discussion_r99585661
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -841,5 +841,6 @@ private[client] class Shim_v1_2 extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16787
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72448/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16787
**[Test build #72448 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72448/testReport)**
for PR 16787 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16787#discussion_r99585644
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -841,5 +841,6 @@ private[client] class Shim_v1_2 extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16787
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
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16787#discussion_r99585445
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveShim.scala
---
@@ -276,4 +276,4 @@ private[hive] object HiveShim {
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16269
LGTM pending test
---
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/16787
**[Test build #72448 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72448/testReport)**
for PR 16787 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16626
**[Test build #72447 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72447/testReport)**
for PR 16626 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16680
**[Test build #72446 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72446/testReport)**
for PR 16680 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16626
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16803
Adding a new option `spark.jars.repositories` afffects more than loading
hive jars, right?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16680
(I am fine with changing the name only for CSV ones for now as well. I
would appreciate if you confirm please)
---
If your project is set up for it, you can reply to this email and have your
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16680
@cloud-fan, I just mainly resembled ones in JSON datasource and I am pretty
sure you knew this when you added some comments. But let me just rebase this as
is for now just in case maybe you are
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16680#discussion_r99583376
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVUtils.scala
---
@@ -0,0 +1,134 @@
+/*
+ * Licensed to the
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16680#discussion_r99583374
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVInferSchema.scala
---
@@ -39,22 +37,76 @@ private[csv] object
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16680#discussion_r99583137
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVFileFormat.scala
---
@@ -170,32 +111,21 @@ class CSVFileFormat
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16803#discussion_r99583052
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -480,7 +479,12 @@ object SparkSubmit extends CommandLineUtils {
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16803
Please rename it to [SPARK-19458][BUILD][SQL]load hive jars from local repo
which has downloaded
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16820
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16820
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72445/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16820
**[Test build #72445 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72445/testReport)**
for PR 16820 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16809
Found the design doc:
https://docs.google.com/document/d/1h5SzfC5UsvIrRpeLNDKSMKrKJvohkkccFlXo-GBAwQQ/edit?ts=574f717f#
> An alternative is to support a new command REFRESH path that
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16677
@sujith71955 Thanks for the test! The test number looks promising!
---
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16809
@cloud-fan The new behavior looks reasonable to me, unless users are
expecting to keey the original cached data.
I went over the change history. I found @sameeragarwal did this in
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16820
**[Test build #72445 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72445/testReport)**
for PR 16820 at commit
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/16820
@yangw1234 could you also check if we need to do this for whole stage code
generation?
...and you really need to add tests.
---
If your project is set up for it, you can reply to this
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/16820
ok to test
---
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16747
(FWIW, I am OK but just worried if it might be supposed to be internal
type, maybe in the future)
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/16818
@uncleGen I think we should limit this to allowing long values for range
frames only; row frames should not get larger than `1 << 31 + 1`. The reason
for this is that we also need to be able to
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16747
It seems there are several ones here and there. Maybe
https://github.com/apache/spark/pull/15751#issuecomment-258518577 is related
too because it is about supporting reading/writing out that
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16792
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16792
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72441/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16792
**[Test build #72441 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72441/testReport)**
for PR 16792 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16269
**[Test build #72444 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72444/testReport)**
for PR 16269 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16269#discussion_r99578597
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -374,25 +375,24 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16819
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72434/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16819
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16819
**[Test build #72434 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72434/testReport)**
for PR 16819 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16787
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16789
**[Test build #3557 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3557/testReport)**
for PR 16789 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16787
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72443/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16787
**[Test build #72443 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72443/testReport)**
for PR 16787 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16787
**[Test build #72443 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72443/testReport)**
for PR 16787 at commit
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/16818
cc @srowen
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16787
**[Test build #72442 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72442/testReport)**
for PR 16787 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16787
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72442/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16787
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
Github user yangw1234 commented on the issue:
https://github.com/apache/spark/pull/16820
@mengxr @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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16787
**[Test build #72442 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72442/testReport)**
for PR 16787 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16792
**[Test build #72441 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72441/testReport)**
for PR 16792 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16820
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
GitHub user yangw1234 opened a pull request:
https://github.com/apache/spark/pull/16820
[SPARK-19471] AggregationIterator does not initialize the generated result
projection before using it
## What changes were proposed in this pull request?
When AggregationIterator
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16792
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16787
**[Test build #72440 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72440/testReport)**
for PR 16787 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16792
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72439/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16792
**[Test build #72439 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72439/testReport)**
for PR 16792 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16787
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72440/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16787
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16787
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16787
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72438/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16787
**[Test build #72438 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72438/testReport)**
for PR 16787 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16787
**[Test build #72440 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72440/testReport)**
for PR 16787 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16810
**[Test build #72437 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72437/testReport)**
for PR 16810 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16738
**[Test build #3556 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3556/testReport)**
for PR 16738 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16815
**[Test build #72436 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72436/testReport)**
for PR 16815 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16819
**[Test build #72434 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72434/testReport)**
for PR 16819 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16789
**[Test build #3557 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3557/testReport)**
for PR 16789 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16792
**[Test build #72439 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72439/testReport)**
for PR 16792 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16787
**[Test build #72438 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72438/testReport)**
for PR 16787 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16815
**[Test build #72435 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72435/testReport)**
for PR 16815 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16815
**[Test build #3558 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3558/testReport)**
for PR 16815 at commit
Github user zero323 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16792#discussion_r99572943
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1272,16 +1272,18 @@ def replace(self, to_replace, value, subset=None):
"""Returns a new
Github user windpiger commented on the issue:
https://github.com/apache/spark/pull/16787
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16818
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72432/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16818
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16818
**[Test build #72432 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72432/testReport)**
for PR 16818 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/16811#discussion_r99572391
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Word2Vec.scala
---
@@ -232,19 +232,40 @@ class Word2VecModel private[ml] (
@Since("1.5.0")
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/16747
CC @cloud-fan for https://github.com/apache/spark/pull/13008#r62947902 and
@yhuai for https://github.com/apache/spark/pull/8597#r38769233 as they might be
what you're referring to?
---
If your
401 - 500 of 557 matches
Mail list logo
