New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: blob sidecars can be filtered by indices #6337
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6337 +/- ##
=============================================
- Coverage 76.61% 60.15% -16.47%
=============================================
Files 248 407 +159
Lines 25898 46464 +20566
Branches 1448 1536 +88
=============================================
+ Hits 19842 27950 +8108
- Misses 6026 18484 +12458
Partials 30 30 |
Performance Report✔️ no performance regression detected Full benchmark results
|
I could validate on a local node the expected behavior using the following command:
|
Co-authored-by: Nico Flaig <nflaig@protonmail.com>
2695885
to
e2365f5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, if all tests pass (or are irrelevant because of their own stablity issues)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM🥳
🎉 This PR is included in v1.16.0 🎉 |
Motivation
Make sure the
getBlobSidecars
endpoint is implemented as specified.Description
getBlobSidecars
doesn't supportindices
filtering, as specified. Add support for the missing query parameters and implement the associated logic.Closes #6185