Are there good reasons for using literals instead of constants here (and in a few other places where MetricData.Type.XXXX were used)?