Closed zhongzc closed 1 week ago
The changes involve significant updates across various files in the codebase, such as altering method return types from usize
to u64
, adding a compression_codec
field to the Blob
struct, and changing dependency listings in Cargo.toml
. Additionally, new modules and error variants are introduced, enhancing functionality related to file handling, compression, and cache management.
Files | Change Summary |
---|---|
src/mito2/src/sst/index.rs |
Changed return type of the finish method from usize to u64 in the Indexer struct. |
src/mito2/src/sst/index/... |
Updated Blob struct by adding compression_codec field and renaming data to compressed_data . Altered ByteCount type and error handling in SstIndexCreator . |
src/mito2/src/sst/parquet/writer.rs |
Removed type conversion to u64 in assignment of index_file_size . |
src/puffin/Cargo.toml |
Added async-compression and async-walkdir dependencies. Moved uuid from dev-dependencies to regular dependencies. |
src/puffin/src/blob_metadata.rs |
Changed CompressionCodec enum's derive attributes from Clone to Copy . |
src/puffin/src/error.rs |
Added new error variants for file operations and updated ErrorExt implementation to handle them. |
src/puffin/src/file_format/writer.rs |
Added compression_codec to Blob , modified add_blob and finish methodsβ return types to u64 for PuffinSyncWriter and PuffinAsyncWriter traits. |
src/puffin/src/file_format/writer/file.rs |
Updated PuffinFileWriter to change return types of add_blob and finish methods to u64 and reference compressed_data . |
src/puffin/src/lib.rs |
Added #![feature(trait_alias)] attribute. |
src/puffin/src/puffin_manager.rs |
Added new module declarations for cache_manager and cached_puffin_manager . |
src/puffin/src/puffin_manager/cache_manager.rs |
Introduced CacheManager and DirWriterProvider traits, and new type aliases for cache operations. |
src/puffin/src/puffin_manager/cached_puffin_manager.rs |
Added DirMetadata , DirFileMetadata structs, and CachedPuffinWriter implementing PuffinWriter . |
src/puffin/src/tests.rs |
Renamed data to compressed_data in Blob struct and added compression_codec field in test functions. |
sequenceDiagram
participant User
participant PuffinFileWriter
participant Blob
participant CacheManager
User->>PuffinFileWriter: add_blob()
PuffinFileWriter-->>Blob: Write compressed_data
PuffinFileWriter-->>User: Return u64 (byte count)
User->>PuffinFileWriter: finish()
PuffinFileWriter-->>CacheManager: Cache directory
PuffinFileWriter-->>User: Return u64 (total bytes)
In the code where bytes do flow,
Changes came, updates did grow.
Compression codecs now in sight,
File sizes altered to new height.
Dependencies lined up in a row, For Puffin's features to smoothly blow.
π π Here's to the future, set aglow! π
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 17.90123%
with 133 lines
in your changes missing coverage. Please review.
Project coverage is 84.70%. Comparing base (
cdd4baf
) to head (b0a4f29
). Report is 1 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?
Mainly implement
put_blob
andput_dir
.Checklist
Summary by CodeRabbit
New Features
Bug Fixes
Improvements
usize
tou64
, enhancing consistency.Blob
struct by introducing acompression_codec
field and renamingdata
tocompressed_data
.Dependency Updates
async-compression
andasync-walkdir
as dependencies.uuid
from development to regular dependencies.