Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/12858#issuecomment-220207574
Hi @cloud-fan , Could I please ask a question? It seems old accumulator is
deprecated but new accumulator should be used everywhere. I am trying to change
some tests and codes to use this but I am a bit confused if I should use
`LongAccumulator` for `Accumulator[Int]`.
I thought I should leave `Accumulator[Int]` as it is but it seems some
tests in `AccumulatorV2Suite` uses int (eg. `0` not `0L`).
So.. Should I use `LongAccumulator` for `Accumulator[Int]` or just leave
them? It seems there is no implementation of `IntAccumulator`. This is
intendedly not implemented?
---
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]
