-
Notifications
You must be signed in to change notification settings - Fork 449
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
feat(ovfs): supplementary opendal related functions #4825
Conversation
@Xuanwo, it's ready for review now. |
struct FileKey(usize); | ||
|
||
/// OpenedFile represents file that opened in memory. | ||
#[allow(dead_code)] |
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.
Hi, from my perspective, adding dead_code
is not ideal. If they are useful, please consider using pub use
to include them.
/// Filesystem is a filesystem implementation with opendal backend, | ||
/// and will decode and process messages from VMs. | ||
pub struct Filesystem {} | ||
#[allow(dead_code)] | ||
pub struct Filesystem { |
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.
Maybe we can export Filesystem
out?
Hi, @Xuanwo. I think this PR may be too old and the updated code may introduce a lot of dead code because there is no clear goal for this PR. This may be a bit strange, so I will resubmit a PR tomorrow. |
Thanks and sorry for reviewing it so late. |
This won't be a problem. 💌 |
No description provided.