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

Index validate fix #1003

Merged
merged 2 commits into from Aug 10, 2016

Conversation

Projects
None yet
2 participants
@hayley-jean
Copy link
Contributor

commented Aug 10, 2016

Reintroduces the changes in PR #885

Changes UnbufferedFileStream over to longs from ints to cater for large index files.

gregoryyoung and others added some commits Apr 6, 2016

Make midpoints and validate at same time
Previously there were two steps involved one to load mid points
and the other to validate md5. This brings both of these scans
together.

no reason to leave a static buffer timebomb waiting for the next
unsuspecting fool
Use longs in UnbufferedFileStream
Update the ptable test for more than 32 bits of records to save and load a ptable with a valid hash.

@gregoryyoung gregoryyoung merged commit bd5b330 into release-v3.9.0 Aug 10, 2016

2 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@gregoryyoung

This comment has been minimized.

Copy link
Member

commented Aug 10, 2016

👍

@pgermishuys pgermishuys deleted the index-validate-fix branch Aug 22, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.