Fix Jaeger remote sampler declarative config polling interval key

[thswlsqls]

Fixes #8648

Description

• JaegerRemoteSamplerComponentProvider.create() read config.getLong("internal"), a key that does not exist, so getLong always returned null and the configured polling interval was silently dropped.
• The schema model ExperimentalJaegerRemoteSamplerModel defines the field as @JsonProperty("interval"), matching the sibling keys endpoint and initial_sampler that were already correct.
• Changed the key to "interval" so a user-configured value is honored; when omitted the builder default (60000ms) still applies.
• Regression from Add file configuration ComponentProvider support for samplers #6494.

Testing done

• Added SamplerFactoryTest#create_JaegerRemote_interval (configured 10000ms honored) and #create_JaegerRemote_intervalDefault (omitted → 60000ms default), driving the real declarative pipeline.
• Added a package-private pollingIntervalMs field and getPollingIntervalMs() visible-for-testing accessor on JaegerRemoteSampler so the interval can be observed; no public API change (apidiff unchanged).
• Confirmed the interval test fails on the un-fixed key (expected: 10000 but was: 60000) and passes after the fix.
• ./gradlew :sdk-extensions:jaeger-remote-sampler:check — passed.
• ./gradlew :sdk-extensions:declarative-config:check — passed (SamplerFactoryTest: 19 tests).

[opentelemetry-pr-dashboard]

Pull request dashboard status

Closed · refreshed 2026-08-11 14:30 UTC

Status above doesn't look right?

• Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

[codecov]

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 91.64%. Comparing base (09d6c17) to head (9bfdee7).
⚠️ Report is 41 commits behind head on main.

Files with missing lines	Patch %	Lines
...sion/trace/jaeger/sampler/JaegerRemoteSampler.java	50.00%	1 Missing ⚠️

❌ Your patch check has failed because the patch coverage (66.66%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@            Coverage Diff            @@
##               main    #8649   +/-   ##
=========================================
  Coverage     91.64%   91.64%           
- Complexity    10348    10349    +1     
=========================================
  Files          1013     1013           
  Lines         27380    27382    +2     
  Branches       3218     3218           
=========================================
+ Hits          25092    25095    +3     
  Misses         1558     1558           
+ Partials        730      729    -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jack-berg]

This was already fixed in #8668. The diff was rendering wrongly indicating that JaegerRemoteSamplerComponentProvider still was reading "internal" despite it being updated on main to read "interval": https://github.com/open-telemetry/opentelemetry-java/blob/main/sdk-extensions/jaeger-remote-sampler/src/main/java/io/opentelemetry/sdk/extension/trace/jaeger/sampler/internal/JaegerRemoteSamplerComponentProvider.java#L50

I pushed a merge commit with main and the diff went away. Closing this because its just test coverage at this point and we fixed the coverage gap in #8668 using a different strategy.