Details
-
Improvement
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
1.4.0, 1.4.1
-
None
Description
The JsonProtocol added in SPARK-3454 doesn't include code for ser/de of SparkListenerExecutorMetricsUpdate messages.
The comment notes that they are "not used", which presumably refers to the fact that the EventLoggingListener doesn't write these events.
However, individual listeners can and should make that determination for themselves; I have recently written custom listeners that would like to consume metrics-update messages as JSON, so it would be nice to round out the JsonProtocol implementation by supporting them.