You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/03/25 13:09:17 UTC

[GitHub] [spark] tomvanbussel commented on issue #24011: [SPARK-27071][CORE] Expose additional metrics in status.api.v1.StageData

tomvanbussel commented on issue #24011: [SPARK-27071][CORE] Expose additional metrics in status.api.v1.StageData
URL: https://github.com/apache/spark/pull/24011#issuecomment-476188534
 
 
   If we cannot break the API, then I see two options:
   
   1. We leave the old fields, and we also add the TaskMetrics. This would mean that some fields are duplicated.
   2. We inline all the fields from TaskMetrics into StageData. This is not as nice (in my opinion), but there will be no duplicates.
   
   Which option is better?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org