Closed MichaelScofield closed 1 month ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
The recent changes improve the caching system's monitoring capabilities by adding metrics for cache evictions and performance tracking during Parquet metadata loading. The CacheManagerBuilder
now logs eviction reasons more effectively, while new metrics provide insights into cache behavior and loading efficiency. These enhancements allow developers to better diagnose performance issues, leading to improved overall system observability.
Files | Change Summary |
---|---|
src/cache.rs |
Enhanced CacheManagerBuilder to include CACHE_EVICTION metric for improved logging of eviction causes. |
src/mito2/src/metrics.rs |
Introduced CACHE_EVICTION counter and PARQUET_METADATA_LOAD_ELAPSED histogram for monitoring cache evictions and metadata loading times. |
src/sst/parquet/metadata.rs |
Added performance monitoring in MetadataLoader for Parquet metadata loading using the new histogram metric. |
🐇 In the meadow where bunnies play,
Metrics bloom in a lively array.
Evictions tracked with a hop and a cheer,
Loading data faster, the path is now clear!
Cache and metadata, a dance so bright,
Observability shines in the soft moonlight! ✨
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 89.65517%
with 3 lines
in your changes missing coverage. Please review.
Project coverage is 85.02%. Comparing base (
4cd5ec7
) to head (85d6617
). Report is 22 commits behind head on main.
@v0y4g3r PTAL
@evenyag PTAL
@evenyag PTAL
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
as title
useful for diagnosing slow queries
Checklist
Summary by CodeRabbit
New Features
Improvements