Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13403
Thank you, @srowen .
I updated the PR according to one of your advices.
For the other advice, I tried like the following. It looks good, but a
little bit inconsistent with the remaining parts of the codes.
```
/** Compute the population variance of this RDD's elements. */
- def variance(): Double = self.withScope {
- stats().variance
- }
+ def variance(): Double = popVariance
/** Compute the population standard deviation of this RDD's elements. */
- def stdev(): Double = self.withScope {
- stats().stdev
- }
+ def stdev(): Double = popStdev
```
It's because the functions of `DoubleRDDFunctions.scala` and
`JavaDoubleRDD.scala` simply forwards to their correspondents. May I keep the
corrent logic for 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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
