Closed zhongzc closed 1 week ago
The recent updates to the puffin codebase introduce new error variants for more granular error handling, add a new module for file access, restructure the cached puffin manager, and implement new functionalities for directory and file metadata management. These enhancements aim to improve error management, streamline code structure, and enhance file handling capabilities within the puffin system.
Files | Summary |
---|---|
src/puffin/src/error.rs |
Added new error variants BlobNotFound , BlobIndexOutOfBound , and FileKeyNotMatch . |
src/puffin/src/puffin_manager.rs |
Added the new file_accessor module. |
cached_puffin_manager.rs |
Restructured module imports; removed directory and file metadata declarations. |
dir_meta.rs |
Defined DirMetadata and DirFileMetadata structs. |
reader.rs |
Introduced CachedPuffinReader struct implementing the PuffinReader trait. |
writer.rs |
Restructured import statements; added new method to CachedPuffinWriter . |
file_accessor.rs |
Added PuffinFileAccessor trait for opening readers and writers for puffin files. |
sequenceDiagram
participant Client
participant CachedPuffinReader
participant CacheManager
participant FileAccessor
Client->>+CachedPuffinReader: Request Blob
CachedPuffinReader->>+CacheManager: Retrieve Blob from Cache
CacheManager-->>-CachedPuffinReader: Blob
CachedPuffinReader->>+FileAccessor: Decompress Blob
FileAccessor-->>-CachedPuffinReader: Decompressed Blob
CachedPuffinReader-->>-Client: Blob Data
In the world of puffin's code, so bright and vast, New errors catch issues, clever and fast. Modules reshuffled, metadata in place, Readers and writers dancing in grace. With every bit, the system grows, A journey of code, see how it flows. 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 0%
with 133 lines
in your changes missing coverage. Please review.
Project coverage is 84.64%. Comparing base (
1e815dd
) to head (0f70f9b
). Report is 4 commits behind head on main.
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
4193
What's changed and what's your intention?
Cooperate with
CacheManager
to implementCachedPuffinWriter
Checklist
Summary by CodeRabbit
New Features
Enhancements
CachedPuffinReader
.Bug Fixes