We need to make sure we don't do this for string or boolean metrics.

Not totally sure on whether we might see string/boolean metrics passed in here. If it is possible, we should not call insertLocator for those. If its not possible, let's throw if we don't see CF_METRICS_FULL/CF_METRICS_PREAGGREGATED_FULL, to make it clear to readers what the limitations of the method are