Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22763
sure, thanks @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22763
ah i see. @mgaido91 can you resubmit it and update the description? The
method is not deprecated now.
---
-
To unsubscribe,
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22763
Actually I don't see at all the need to revert this. Even without
deprecating `computeCost`, using it for evaluating a model is a very poor
approach. We should anyway advice the users to a better
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22763
Ur, @mengxr and @gatorsmile seems to determine to revert #22756 on
`branch-2.4` only.
---
-
To unsubscribe, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22763
This has been reverted from master/2.4
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22763
thanks, merging to master/2.4!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22763
done @dongjoon-hyun , thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22763
Can we use a more positive title, @mgaido91 ?
- Update BisectingKMeans example not to use deprecated computeCost method
- Update BisectingKMeans example to use ClusteringEvaluator
---
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/22763
LGTM. 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/22763
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97527/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22763
**[Test build #97527 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97527/testReport)**
for PR 22763 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22763
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22763
**[Test build #97527 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97527/testReport)**
for PR 22763 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22763
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22763
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22763
cc @cloud-fan @dongjoon-hyun @holdenk @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
16 matches
Mail list logo
