Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/2867#issuecomment-60442759
I've decided to hold off on introducing my table-builder abstraction, since
it's kind of leaky because there are corner-cases that it doesn't handle
cleanly. Therefore, this shouldn't block on that. It would be great to try to
get this merged so that it can unblock your other metrics patch.
Regarding tooptips: I agree with Andrew that inline descriptions of the
metrics might look a little cluttered, so hover tooltips would be a nice
addition. This is a net improvement over what we have now, though, so I
suppose we could wait until later to add those tooltips.
---
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
