[feat] Deduplicate spans based upon their hashcode

[cdanis]

Which problem is this PR solving?

• Resolves [Bug]: ElasticSearch storage not deduplicating multiply-archived spans #6001

Description of the changes

• Rename the Zipkin-legacy span "deduper" to a less confusing name
• Add a real span deduper that deduplicates based on span hashcodes
• Sort tags in spans before we attempt deduplication, so hashcode is deterministic

How was this change tested?

• Unit tested

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: yarn lint and yarn test

[cdanis]

@yurishkuro as discussed! let me know if you want anything else changed

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.79%. Comparing base (d243452) to head (3dcc05c).
Report is 1 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #6009       +/-   ##
===========================================
+ Coverage   51.48%   96.79%   +45.31%     
===========================================
  Files         176      349      +173     
  Lines        8875    16582     +7707     
===========================================
+ Hits         4569    16050    +11481     
+ Misses       3863      343     -3520     
+ Partials      443      189      -254     

Flag	Coverage Δ
badger_v1	8.00% <0.00%> (-0.02%)	⬇️
badger_v2	1.82% <0.00%> (-0.01%)	⬇️
cassandra-4.x-v1	16.56% <0.00%> (-0.04%)	⬇️
cassandra-4.x-v2	1.75% <0.00%> (-0.01%)	⬇️
cassandra-5.x-v1	16.56% <0.00%> (-0.04%)	⬇️
cassandra-5.x-v2	1.75% <0.00%> (-0.01%)	⬇️
elasticsearch-6.x-v1	18.72% <0.00%> (-0.06%)	⬇️
elasticsearch-7.x-v1	18.79% <0.00%> (-0.05%)	⬇️
elasticsearch-8.x-v1	18.98% <0.00%> (-0.04%)	⬇️
elasticsearch-8.x-v2	1.82% <0.00%> (-0.01%)	⬇️
grpc_v1	9.52% <7.40%> (-0.04%)	⬇️
grpc_v2	7.13% <7.40%> (-0.02%)	⬇️
kafka-v1	9.71% <7.40%> (-0.03%)	⬇️
kafka-v2	1.82% <0.00%> (-0.01%)	⬇️
memory_v2	1.82% <0.00%> (-0.01%)	⬇️
opensearch-1.x-v1	18.83% <0.00%> (-0.05%)	⬇️
opensearch-2.x-v1	18.84% <0.00%> (-0.05%)	⬇️
opensearch-2.x-v2	1.81% <0.00%> (-0.01%)	⬇️
tailsampling-processor	0.46% <0.00%> (-0.01%)	⬇️
unittests	95.27% <100.00%> (?)

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.

[cdanis]

PTAL :D

[yurishkuro]

Thanks!

[cdanis]

added additional tests because patch code coverage failed