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

io: public interface for io:: subsystem #17566

Open
andrwng opened this issue Apr 3, 2024 · 1 comment
Open

io: public interface for io:: subsystem #17566

andrwng opened this issue Apr 3, 2024 · 1 comment

Comments

@andrwng
Copy link
Contributor

andrwng commented Apr 3, 2024

Today, prospective users of the io:: subsystem that want to read and write data via the pager and paging data source must interact with them directly. This is awkward for a couple reasons:

  • The pager and page cache are meant to be internal implementation details
  • Usage patterns expectations of the pager are forced onto callers (e.g. not reading so much that pager::read() returns a large vector), which makes it a bit clunky to use

Perhaps some of these details can be hidden away via some file-like abstraction built atop the pager and co.

JIRA Link: CORE-2015

@dotnwat
Copy link
Member

dotnwat commented Apr 3, 2024

This makes sense. In addition, what kinds of functionality will you need to get started? For example, pager lacks fsync so I think that is needed. That might be about it, right?

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

No branches or pull requests

2 participants