[refactor]: use strings.Builder to improve performance

[claudecodering]

Which problem is this PR solving?

Description of the changes

strings.Builder has fewer memory allocations and better performance.
More info: golang/go#75190

How was this change tested?

No need.

Checklist

• I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
• I have signed all commits
• I have added unit tests for the new functionality
• I have run lint and test steps successfully
  • for jaeger: make lint test
  • for jaeger-ui: npm run lint and npm run test

[mahadzaryab1]

@claudecodering Is there a linter that would enforce this change?

[claudecodering]

@claudecodering Is there a linter that would enforce this change?

I wrote one locally, but it only performs detection — most of the changes still require manual intervention.

[mahadzaryab1]

@claudecodering Is there a linter that would enforce this change?

I wrote one locally, but it only performs detection — most of the changes still require manual intervention.

@claudecodering Looks like perfsprint has a concat-loop option that can be used to check for this exact case. Do you want to turn that on in this PR as well?

[claudecodering]

@claudecodering Is there a linter that would enforce this change?

I wrote one locally, but it only performs detection — most of the changes still require manual intervention.

@claudecodering Looks like perfsprint has a concat-loop option that can be used to check for this exact case. Do you want to turn that on in this PR as well?

Of course! I’d love to do that.

[claudecodering]

Please review again.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.53%. Comparing base (528476f) to head (1ae76e4).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7630      +/-   ##
==========================================
- Coverage   96.59%   96.53%   -0.06%     
==========================================
  Files         384      384              
  Lines       19404    19476      +72     
==========================================
+ Hits        18744    18802      +58     
- Misses        477      489      +12     
- Partials      183      185       +2     

Flag	Coverage Δ
badger_v1	8.82% <ø> (-0.01%)	⬇️
badger_v2	1.72% <ø> (-0.02%)	⬇️
cassandra-4.x-v1-manual	12.55% <ø> (-0.02%)	⬇️
cassandra-4.x-v2-auto	1.72% <ø> (-0.02%)	⬇️
cassandra-4.x-v2-manual	1.72% <ø> (-0.02%)	⬇️
cassandra-5.x-v1-manual	12.55% <ø> (-0.02%)	⬇️
cassandra-5.x-v2-auto	1.72% <ø> (-0.02%)	⬇️
cassandra-5.x-v2-manual	1.72% <ø> (-0.02%)	⬇️
clickhouse	1.65% <ø> (-0.02%)	⬇️
elasticsearch-6.x-v1	16.75% <ø> (-0.02%)	⬇️
elasticsearch-7.x-v1	16.78% <ø> (-0.02%)	⬇️
elasticsearch-8.x-v1	16.92% <ø> (-0.02%)	⬇️
elasticsearch-8.x-v2	1.72% <ø> (-0.02%)	⬇️
elasticsearch-9.x-v2	1.72% <ø> (-0.02%)	⬇️
grpc_v1	10.75% <ø> (-0.02%)	⬇️
grpc_v2	1.72% <ø> (-0.02%)	⬇️
kafka-3.x-v1	10.25% <ø> (-0.01%)	⬇️
kafka-3.x-v2	1.72% <ø> (-0.02%)	⬇️
memory_v2	1.72% <ø> (-0.02%)	⬇️
opensearch-1.x-v1	16.82% <ø> (-0.02%)	⬇️
opensearch-2.x-v1	16.82% <ø> (-0.02%)	⬇️
opensearch-2.x-v2	1.72% <ø> (-0.02%)	⬇️
opensearch-3.x-v2	1.72% <ø> (-0.02%)	⬇️
query	1.72% <ø> (-0.02%)	⬇️
tailsampling-processor	0.50% <ø> (-0.01%)	⬇️
unittests	95.44% <100.00%> (-0.06%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[mahadzaryab1]

@claudecodering The CI is failing. This PR needs to be opened from a branch on your fork instead of main.