-
Notifications
You must be signed in to change notification settings - Fork 41
feat:removes the load_to_cache field #1264
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
Conversation
44896e6
to
c61239e
Compare
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
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.
Please fix lint.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1264 +/- ##
==========================================
- Coverage 48.51% 48.44% -0.08%
==========================================
Files 60 60
Lines 14998 15008 +10
==========================================
- Hits 7277 7270 -7
- Misses 7721 7738 +17
🚀 New features to boost your workflow:
|
This pull request removes the load_to_cache field and adds trait method definitions to dfdaemon_download.rs and dfdaemon_upload.rs. - Removed cache-related processing from Task handling. - Added trait method definitions such as download_cache_task to dfdaemon_download.rs and dfdaemon_upload.rs to comply with the API format of version 2.1.55. - Aim to allow Task to focus on disk interactions while delegating memory cache operations to CacheTask. Signed-off-by: fu220 <2863318196@qq.com>
c61239e
to
1941328
Compare
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
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
Description
This pull request removes the
load_to_cache
field and adds trait method definitions todfdaemon_download.rs
anddfdaemon_upload.rs
.Related Issue
Changes
download_cache_task
todfdaemon_download.rs
anddfdaemon_upload.rs
to comply with the API format of version 2.1.55.Motivation and Context
Screenshots (if appropriate)