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

Allow prefetching of a range of mapped file #66

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CAFxX
Copy link

@CAFxX CAFxX commented Mar 4, 2019

This is an attempt at implementing #65. The PR is currently untested because I've been unable to build larray locally, so hopefully travis will be able to do it.

@CAFxX
Copy link
Author

CAFxX commented Mar 4, 2019

Seems like travis is not rebuilding the native libraries

@CAFxX
Copy link
Author

CAFxX commented Mar 4, 2019

@xerial I'm unable to get make all to work (tried both on Ubuntu and Mac OS), and I can't find instructions on how to set up the build env to be able to build the native libraries. Do you have a VM or container image that I can use? Alternatively, can you build the libraries and update the PR?

@xerial
Copy link
Owner

xerial commented Mar 5, 2019

@CAFxX Thanks for the update. I’m thinking about using docker images for supporting building binary images on Travis CI so that you don’t need to set up native compiler on your machine.

I did the same approach in sqlite-jdbc project, and it has been working well. I just need to migrate some scripts from this to here.

@CAFxX
Copy link
Author

CAFxX commented Mar 5, 2019

@xerial that would be great. Please let me know once it's ready; unfortunately, it feels like there's not much I can do until then...

@CAFxX
Copy link
Author

CAFxX commented Mar 19, 2019

@xerial any update?

@xerial
Copy link
Owner

xerial commented Mar 19, 2019

@CAFxX Sorry about the delay. I still don't have sufficient time to work on this.

@CAFxX
Copy link
Author

CAFxX commented Apr 10, 2019

Should I close the PR then?

@xerial
Copy link
Owner

xerial commented Apr 10, 2019

@CAFxX Please leave it open.

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.

None yet

2 participants