-
Notifications
You must be signed in to change notification settings - Fork 76
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
NSFS | MPU | optimization #7434
Conversation
8daded1
to
bdfc30b
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.
Some comments but LGTM
c99d6e7
to
617609a
Compare
8955419
to
d33c756
Compare
src/sdk/namespace_fs.js
Outdated
let open_mode = 'r+'; | ||
const md_open_mode = 'w+'; |
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.
we can add a single open mode if that can help here -
{ "w*", O_CREAT | O_RDWR },
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.
instead, what I did was to use * as truncate remover, let me know what you think
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.
I'd summon @eshelmarc to comment on that. WDYT on this mode. We don't want to truncate, but keep writing to the file in offsets that are multiplies of the part size. Romy said that O_APPEND is also not suitable so we wanted to open with just O_CREAT - Does this make sense?
d33c756
to
f8f7697
Compare
0e368bf
to
bf7b37b
Compare
2741ae5
to
8917331
Compare
Signed-off-by: Romy <35330373+romayalon@users.noreply.github.com>
8917331
to
669d405
Compare
Explain the changes
Issues: Fixed #xxx / Gap #xxx
Testing Instructions: