Skip to content
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

new feature: direct IO support in fs #5125

Open
1 task
sundy-li opened this issue Sep 18, 2024 · 3 comments
Open
1 task

new feature: direct IO support in fs #5125

sundy-li opened this issue Sep 18, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@sundy-li
Copy link
Contributor

Feature Description

Support direct io mode in fs backend

Problem and Solution

By using direct io, we can directly write/read into file system without through page-cache, which is useful in DBMS to control the memory.

Additional Context

No response

Are you willing to contribute to the development of this feature?

  • Yes, I am willing to contribute to the development of this feature.
@sundy-li sundy-li added the enhancement New feature or request label Sep 18, 2024
@Lzzzzzt
Copy link
Contributor

Lzzzzzt commented Oct 29, 2024

I'd like to implement this.

But should this feature be a directory level or file level?

I mean the direct-io flag should apply to the whole root dir or the specific file? If the file, it seems currently no way to pass the flag, should add it to OpRead/OpWrite/...?

Also, the read/write file using the internal FsReader/FsWriter which have internal buffer, is this a little bit conflict?

@Xuanwo
Copy link
Member

Xuanwo commented Oct 29, 2024

But should this feature be a directory level or file level?

I believe it should at Service level, users should specify this flag while configuring this backend.

Also, the read/write file using the internal FsReader/FsWriter which have internal buffer, is this a little bit conflict?

It's still meaningful since we can avoid in-kernel memory cache.

@Lzzzzzt
Copy link
Contributor

Lzzzzzt commented Oct 29, 2024

I believe it should at Service level, users should specify this flag while configuring this backend.

It's still meaningful since we can avoid in-kernel memory cache.

OK, got it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants