Nit fix by avoiding bound validation unless View feature is enabled (#…

…2355)

opentelemetry-sdk/src/metrics/internal/histogram.rs

@@ -74,9 +74,15 @@ pub(crate) struct Histogram<T: Number> {
 }
 
 impl<T: Number> Histogram<T> {
+    #[allow(unused_mut)]
     pub(crate) fn new(mut bounds: Vec<f64>, record_min_max: bool, record_sum: bool) -> Self {
-        bounds.retain(|v| !v.is_nan());
-        bounds.sort_by(|a, b| a.partial_cmp(b).expect("NaNs filtered out"));
+        #[cfg(feature = "spec_unstable_metrics_views")]
+        {
+            // TODO: When views are used, validate this upfront
+            bounds.retain(|v| !v.is_nan());
+            bounds.sort_by(|a, b| a.partial_cmp(b).expect("NaNs filtered out"));
+        }
+
         let buckets_count = bounds.len() + 1;
         Histogram {
             value_map: ValueMap::new(buckets_count),