Github user kayousterhout commented on the pull request:

    https://github.com/apache/spark/pull/2968#issuecomment-61938133
  
    @sryza this mostly looks good and super helpful!!  One more comment I 
discussed with @pwendell: I think it would be good to add an output type field 
to output metrics (similar to what we have for input metrics) that currently 
can only be Hadoop.  The motivation for this is that there's some annoying 
overhead associated with changing stuff we write out to Json, and it seems like 
we're likely to want to add metrics for other output types (e.g., storing 
output data in the block manager) later on.


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