Update visibility of AttributeSet from pub to pub(crate)

[utpilla]

AttributeSet is no longer required to be a public type. It's only used internally

Changes

• Update visibility of AttributeSet from pub to pub(crate)
• Remove unnecessary methods
• Deleted the related benchmark file (benchmarks can't access AttributeSet now)

Merge requirement checklist

• CONTRIBUTING guidelines followed
• Unit tests added/updated (if applicable)
• Appropriate CHANGELOG.md files updated for non-trivial, user-facing changes
• Changes in public API reviewed (if applicable)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.1%. Comparing base (673c328) to head (3690084).
Report is 1 commits behind head on main.

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #2045   +/-   ##
=====================================
  Coverage   77.1%   77.1%           
=====================================
  Files        123     123           
  Lines      21103   21088   -15     
=====================================
  Hits       16275   16275           
+ Misses      4828    4813   -15     

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

[cijothomas]

Good bye dear AttributeSet! You served us well, ensuring Metric identity and correct aggregation for a long time. Now you can continue to do so, but internally ;)

[lalitb]

Deleted the related benchmark file (benchmarks can't access AttributeSet now)

That's one of the bottleneck with pub(crate) :(