I just noticed that we have `Metric.DataType` and `MetricData.Type`.  I don't know whether I am more offended that we've done this or that I didn't notice it earlier.