Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/654#issuecomment-45555739
Looks good. I wonder if we should just add timestamps to all events
eventually. Adding an extra field to each event is a little ugly, but the
alternative of using the time when EventLoggingListener receives the event is a
little inaccurate.
---
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.
---
