Removing commentted code; Change.md updated

Author: anujnegi270

opentelemetry-sdk/CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## vNext
 
+- *Feature*: Introduced a new feature flag, `experimental_metrics_disable_name_validation`, under the `opentelemetry-sdk`, which allows disabling the Instrument Name Validation. This is useful in scenarios where you need to use *special characters*, *Windows Perf Counter Wildcard Path*, or similar cases. For more details, check [#2543](https://github.com/open-telemetry/opentelemetry-rust/pull/2543).
+
 - *Breaking(Affects custom metric exporter authors only)* `start_time` and `time` is moved from DataPoints to aggregations (Sum, Gauge, Histogram, ExpoHistogram) see [#2377](https://github.com/open-telemetry/opentelemetry-rust/pull/2377) and [#2411](https://github.com/open-telemetry/opentelemetry-rust/pull/2411), to reduce memory.
 
 - *Breaking* `start_time` is no longer optional for `Sum` aggregation, see [#2367](https://github.com/open-telemetry/opentelemetry-rust/pull/2367), but is still optional for `Gauge` aggregation see [#2389](https://github.com/open-telemetry/opentelemetry-rust/pull/2389).

opentelemetry-sdk/src/metrics/meter.rs

@@ -583,14 +583,6 @@ fn validate_bucket_boundaries(boundaries: &[f64]) -> MetricResult<()> {
     Ok(())
 }
 
-// Regex pattern for Windows Performance Counter names
-// #[cfg(feature = "experimental_metrics_disable_name_validation")]
-// fn is_valid_windows_perf_counter_name(name: &str) -> bool {
-//     let pattern = r"^\\[A-Za-z0-9_ ]+\\[A-Za-z0-9_ /%-]+$";
-//     let re = Regex::new(pattern).unwrap();
-//     re.is_match(name)
-// }
-
 #[cfg(feature = "experimental_metrics_disable_name_validation")]
 fn validate_instrument_name(_name: &str) -> MetricResult<()> {
     // No name restrictions when name validation is disabled