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

Change DataFile to be Cacheless. #8

Open
BenjaminHolland opened this issue Mar 2, 2017 · 0 comments
Open

Change DataFile to be Cacheless. #8

BenjaminHolland opened this issue Mar 2, 2017 · 0 comments

Comments

@BenjaminHolland
Copy link
Owner

DataFile should be able to read file contents without caching the entire data set.

Steps to implement:

  1. Change constructor to simply store the path (and maybe validate it).
  2. Change getStream methods.
    Option A. Change the get
    Stream methods to skip to the required section, read it into a collection, and return it.
    Option B. Change the get*Stream methods to return a Stream-wrapped Iterable that only loads one record at a time.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant