Github user edwinalu commented on the issue:
https://github.com/apache/spark/pull/21221
@squito , I'm modifying ExecutorMetrics to take in the metrics array --
this will be easier for tests where we pass in set values, and seems fine for
the actual code. It will check that the length of the passed in array is the
same as MetricGetter.values.length. Let me know if you have any concerns.
@felixcheung , I'll finish the current changes, then rebase.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
