blockservice: optimize getBlocks filter by not rescanning the already valid leading elements

[Jorropo]

We would scan the valid leading elements twice.

[Jorropo]

Current dependencies on/for this PR:

• main
  • PR blockservice: optimize getBlocks filter by not rescanning the already valid leading elements #558 👈
    • PR blockservice: fix symbol name in docs #559
      • PR blockservice: stop using deprecated NewWriteThrough in tests #560
        • PR blockservice: remove deprecated NewWriteThrough function #565
        • PR blockservice: move session handling as part of the interface #563
          • PR blockservice: remove inner fields access methods #564
        • PR blockservice: move to single unique struct and add WithContentBlocker option #561
          • PR blockservice: remove inner fields access methods #562
            • PR Move providing responsabilities from bitswap to blockservice #534
              • PR Remove content routing from bitswap network #535
                • PR Remove provider query manager #536
        • PR blockservice: add WithContentBlocker option #556
          • PR blockservice: optimize getBlocks filter by not rescanning the already valid leading elements #557

This stack of pull requests is managed by Graphite.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c7f33e6) 65.45% compared to head (d5be896) 65.45%.

@@            Coverage Diff             @@
##             main     #558      +/-   ##
==========================================
- Coverage   65.45%   65.45%   -0.01%     
==========================================
  Files         207      207              
  Lines       25622    25623       +1     
==========================================
  Hits        16772    16772              
- Misses       7369     7371       +2     
+ Partials     1481     1480       -1     

Files	Coverage Δ
blockservice/blockservice.go	78.37% <100.00%> (+3.12%)	⬆️

... and 10 files with indirect coverage changes