Github user pwendell commented on the pull request: https://github.com/apache/incubator-spark/pull/624#issuecomment-35776248 @andrewor14 Looks good to me. This adds a few function calls in the `next` path... would you mind doing some local regression testing just to make sure there are no regressions? I doubt there are but if you did a local test I'd feel better about it before merging in into 0.9.
--- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. To do so, please top-post your response. 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---