Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/9051#issuecomment-148552339
Chiming in a little late here. I actually like (1) in @pwendell's
[comment](https://github.com/apache/spark/pull/9051#issuecomment-147493632),
where we fix the semantics of Duration. IMO this is a bug; most people
understand "duration" of a stage to be the time it took to actually run the
stage, excluding the waiting time. My concern with (2) is that any term we
choose adds more confusion; what does "service time" or "active time" mean? Why
is there a "service time" AND a duration?
I also don't see how introducing a max task time makes the stage duration
any less confusing. I actually think it's even worse since now the user has to
reconcile two metrics that don't agree with each other.
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
