Avoid linear queue.size() calls in span producers by storing queue size separately

[raipc]

Related to #7140

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: raipc / name: Anton Rybochkin (f62710d, bd2362d)

[codecov]

Codecov Report

Attention: Patch coverage is 71.42857% with 4 lines in your changes missing coverage. Please review.

Project coverage is 89.95%. Comparing base (56941a5) to head (bd2362d).
Report is 55 commits behind head on main.

Files with missing lines	Patch %	Lines
...a/io/opentelemetry/sdk/trace/internal/JcTools.java	20.00%	4 Missing ⚠️

❌ Your patch status has failed because the patch coverage (71.42%) 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    #7141      +/-   ##
============================================
+ Coverage     89.85%   89.95%   +0.10%     
- Complexity     6610     6678      +68     
============================================
  Files           740      750      +10     
  Lines         19986    20168     +182     
  Branches       1964     1977      +13     
============================================
+ Hits          17958    18142     +184     
+ Misses         1439     1434       -5     
- Partials        589      592       +3     

☔ View full report in Codecov by Sentry.
📢 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.

[raipc]

JcTools tests exist but jacoco reports are not generated

[jkwatson]

hmm. This modifies a shaded-in dependency. Is it possible to have this fixed upstream, and then get it incorporated in here? I'd hate to have our version diverge from the jctool code, if at all possible.

[raipc]

hmm. This modifies a shaded-in dependency. Is it possible to have this fixed upstream, and then get it incorporated in here? I'd hate to have our version diverge from the jctool code, if at all possible.

Could you clarify it, please?

io.opentelemetry.sdk.trace.internal.JcTools is not a shaded class from JcTools library, it's a helper class to work with JcTools queue.
The JCTools library itself is shaded into io.opentelemetry.internal.shaded.jctools package

[jkwatson]

hmm. This modifies a shaded-in dependency. Is it possible to have this fixed upstream, and then get it incorporated in here? I'd hate to have our version diverge from the jctool code, if at all possible.

Could you clarify it, please?

io.opentelemetry.sdk.trace.internal.JcTools is not a shaded class from JcTools library, it's a helper class to work with JcTools queue. The JCTools library itself is shaded into io.opentelemetry.internal.shaded.jctools package

whoops. I was misreading. Never mind! Carry on.

[jack-berg]

This looks good to me. Nice optimization!