Make description of Samplers Locale independent

[lmonkiewicz]

Fixes: #4880

[codecov]

Codecov Report

Base: 90.83% // Head: 90.83% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (0a477da) compared to base (a46dd00).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #4887      +/-   ##
============================================
- Coverage     90.83%   90.83%   -0.01%     
- Complexity     4856     4859       +3     
============================================
  Files           556      556              
  Lines         14474    14483       +9     
  Branches       1410     1410              
============================================
+ Hits          13148    13155       +7     
- Misses          908      909       +1     
- Partials        418      419       +1     

Impacted Files	Coverage Δ
...sion/trace/jaeger/sampler/RateLimitingSampler.java	88.88% <100.00%> (+3.17%)	⬆️
...y/sdk/trace/samplers/TraceIdRatioBasedSampler.java	86.66% <100.00%> (+2.05%)	⬆️
...metry/sdk/metrics/export/PeriodicMetricReader.java	87.14% <0.00%> (-2.86%)	⬇️
.../opentelemetry/sdk/metrics/AbstractInstrument.java	41.66% <0.00%> (+5.30%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[jack-berg]

Just one small comment. Thanks!

[lmonkiewicz]

Just one small comment. Thanks!

Changed.

[trask]

@jack-berg what do you think about calculating it once in the constructor? getDescription is not supposed to be called a lot, but just in case...

and I think the extra small amount of memory is ok since there shouldn't be a huge number of Sampler instances