Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(s3stream): add read block cache context to trace status #835

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

SCNieh
Copy link
Contributor

@SCNieh SCNieh commented Dec 15, 2023

No description provided.

Signed-off-by: Shichao Nie <niesc@automq.com>
@SCNieh SCNieh requested a review from superhx December 15, 2023 07:13
Copy link

codecov bot commented Dec 15, 2023

Codecov Report

Merging #835 (9e04c92) into main (08d1462) will increase coverage by 0.02%.
Report is 3 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #835      +/-   ##
============================================
+ Coverage     55.47%   55.49%   +0.02%     
- Complexity     1242     1245       +3     
============================================
  Files           132      132              
  Lines          8530     8530              
  Branches        787      787              
============================================
+ Hits           4732     4734       +2     
  Misses         3380     3380              
+ Partials        418      416       -2     
Components Coverage Δ
RocketMQ Broker ∅ <ø> (∅)
RocketMQ Common 5.80% <ø> (ø)
RocketMQ Controller 54.58% <ø> (ø)
RocketMQ Proxy 37.81% <ø> (+0.10%) ⬆️
RocketMQ Store 74.27% <ø> (ø)
RocketMQ Stream ∅ <ø> (∅)
RocketMQ Metadata 35.44% <ø> (ø)

see 1 file with indirect coverage changes

@superhx superhx merged commit 848cdde into main Dec 15, 2023
9 checks passed
@superhx superhx deleted the block_cache_context branch December 15, 2023 07:27
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants