Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/4311#issuecomment-75324157
@twinkle-sachdeva I just noticed that there are a lot of style guide
violations in this patch. Please look at how the rest of the code is formatted
for reference. Also, this patch is currently highly tailored to the
implementation in event log format in Spark 1.2, which should be very different
from that used in Spark 1.3. I would imagine that it will be a non-trivial
amount of work to bring this patch up to date to master. Would you still prefer
to do this, or have one of @vanzin or me take over? In both cases we'll be sure
to give you credit in the release notes.
---
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]
