-
Notifications
You must be signed in to change notification settings - Fork 193
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: merge io from same blob panic #1347
Conversation
When merging io from same blob with different id, assertion breaks. The images without blob deduplication suffers from it. This pr removes the assertion that requires merging in same blob index. By design, it makes sense, because different blob layer may share same blob file. A continuous read from same blob for different layer is helpful for performance. Signed-off-by: 泰友 <cuichengxu.ccx@antgroup.com>
@ccx1024cc , a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/83046 |
@ccx1024cc , the title has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/83047 |
Codecov Report
@@ Coverage Diff @@
## master #1347 +/- ##
==========================================
+ Coverage 45.69% 45.79% +0.09%
==========================================
Files 123 123
Lines 37287 37339 +52
Branches 37287 37339 +52
==========================================
+ Hits 17040 17099 +59
+ Misses 19349 19341 -8
- Partials 898 899 +1
|
@ccx1024cc , The CI test is completed, please check result:
Congratulations, your test job passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for your work.
@ccx1024cc , The CI test is completed, please check result:
Sorry, your test job failed. Please get the details in the link. |
Details
When merging io from same blob with different id, assertion breaks. The images without blob deduplication suffer from it.
This pr removes the assertion that requires merging in same blob index. By design, it makes sense, because different blob layer may share same blob file. A continuous read from same blob for different layer is helpful for performance.
Types of changes
Checklist