Add read methods to the snapshotter#21
Conversation
|
@SamuelSarle Thank you for your contribution! Your pull request is now a part of the Race of Sloths! Current status: executed
Your contribution is much appreciated with a final score of 5! @artob received 25 Sloth Points for reviewing and scoring this pull request. Another weekly streak completed, well done @SamuelSarle! To keep your weekly streak and get another bonus make pull request next week! Looking forward to see you in race-of-sloths What is the Race of SlothsRace of Sloths is a friendly competition where you can participate in challenges and compete with other open-source contributors within your normal workflow For contributors:
For maintainers:
Feel free to check our website for additional details! Bot commands
|
There was a problem hiding this comment.
Pull Request Overview
This PR adds read functionality to the Snapshotter by exposing read and read_current methods that forward calls to the underlying storage layer. The changes also improve the API consistency by converting string parameters to use generic AsRef<str> bounds and enhance tracing instrumentation.
- Adds two new public methods
readandread_currentto expose storage read functionality - Converts string parameters from
&strtoimpl AsRef<str>for better API flexibility - Improves tracing instrumentation by adding URL field logging and removing unused output format option
|
@race-of-sloths score 5 |
Tiny patch to forward the
readandread_currentmethods from the storage.@race-of-sloths