Github user mateiz commented on the pull request:

    https://github.com/apache/spark/pull/3302#issuecomment-63407717
  
    Basically it is weird to have a var shared with subclasses, but I think 
this will be more obvious. Probably the very best way is to make elementsRead 
be modified only in Spillable, and add a method called addElementRead or 
something like that subclasses can call to tell it to increment the count.


---
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]

Reply via email to