Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/1222#discussion_r15478958
--- Diff: core/src/main/scala/org/apache/spark/util/FileLogger.scala ---
@@ -1,200 +0,0 @@
-/*
--- End diff --
We shouldn't remove this... This is intended to be a general purpose class
for logging that handles compression and buffering and other logic. I notice
that you replaced this with PrintWriter elsewhere. Even with the new naming
scheme you can still use this class for event logging, and it will simplify
`EventLoggingListener` a bunch.
---
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.
---
