Fix/sampler shutdown lifecycle

[Debashismitra01]

Changes

Adds shutdown lifecycle support to Samplers.

Previously Sampler implementations had no way to release resources when
SdkTracerProvider was shut down. Resource-owning samplers such as
JaegerRemoteSampler could leave background executors and connections running.

This change:

• Adds default Sampler.shutdown()
• Invokes sampler shutdown from SdkTracerProvider shutdown
• Updates JaegerRemoteSampler to release polling and sender resources
• Adds lifecycle tests

Fixes #8535

[otelbot]

⚠️ API changes detected — additional maintainer review required

@jack-berg @jkwatson

This PR modifies the public API surface area of the following module(s):

• opentelemetry-sdk-extension-jaeger-remote-sampler
• opentelemetry-sdk-trace

Please review the changes in docs/apidiffs/current_vs_latest/ carefully before approving.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.56%. Comparing base (2626179) to head (f246d3e).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #8574      +/-   ##
============================================
+ Coverage     90.96%   91.56%   +0.59%     
- Complexity    10207    10266      +59     
============================================
  Files          1013     1013              
  Lines         27160    27115      -45     
  Branches       3182     3184       +2     
============================================
+ Hits          24706    24827     +121     
+ Misses         1730     1564     -166     
  Partials        724      724              

☔ 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.

[jkwatson]

Thanks! I'll let @jack-berg also approve before merging, as there's an API change.

[jack-berg]

Just a few minor comments.

[jack-berg]

It would be most consistent to add both shutdown and update Sampler to extends Closeable, where the default implementation of close is:

  @Override
  default void close() {
    shutdown().join(10, TimeUnit.SECONDS);
  }

[jack-berg]

Why does this protect against sampler returning a null shutdown result, but not activeSpanProcessor?

[jack-berg]

Why switch from true to false for this?

[jack-berg]

Should protect against calling this multiple times as we do elsewhere in the repo. For example:

opentelemetry-java/sdk/trace/src/main/java/io/opentelemetry/sdk/trace/export/BatchSpanProcessor.java

Lines 132 to 138 in 4079ece

	@Override
	public CompletableResultCode shutdown() {
	if (isShutdown.getAndSet(true)) {
	return CompletableResultCode.ofSuccess();
	}
	return worker.shutdown();
	}

[opentelemetry-pr-dashboard]

This PR has review comments. Review suggestions, whether from maintainers or automated reviewers, aren't always correct or required. Please evaluate each comment on its merits, then make sure each thread has a clear outcome.

For example, link to the commit if you applied a suggestion, explain why it wasn't applied, or ask a follow-up question.

Automation flags a PR for human review once every review thread has a reply or is marked as resolved.

Status across open PRs is visible on the pull request dashboard.