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

int vector mapper #2 #236

Open
wants to merge 11 commits into
base: develop-v3
Choose a base branch
from
Open

Conversation

mpetri
Copy link
Collaborator

@mpetri mpetri commented Apr 10, 2015

This pull request adds functionality to the int-vector-mapper (#232) such as [x] typedef for a read_only_mapper [x] support class which allows the creation of output buffer: write_out_buffer [x] support class which supports the creation of temporary files which are deleted by the int-vector-mapper destructor temp_file_buffer [x] tests for the new functionality the only remaining open functionality of the int-vector-buffer which is not yet implemented is the ability to handle ram_files. However, I suggest the following: - Remove the majority of the ram file functionality. - When construct_im is called, set a flag somewhere which then forces the int-vector-mapper to not unmap the file in the destructor. Thus, if enough memory is available it remains in memory. - When the construct_im method is done, there is some call to "clean up" which would unmap everything and delete the leftover files (or nothing will be written to disk in the first place)

@sdsllitebot
Copy link
Collaborator

Can one of the admins verify this patch?

@mpetri
Copy link
Collaborator Author

mpetri commented Apr 10, 2015

test this please

@mpetri
Copy link
Collaborator Author

mpetri commented Apr 10, 2015

test this please

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

Successfully merging this pull request may close these issues.

2 participants