-
Notifications
You must be signed in to change notification settings - Fork 24
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
btree iterator doesn't appear to work #216
Comments
Is this still a problem? If so, do you have a test case? |
I just checked, and currently we crash when the iterator appears to get to the end. Jakob is working on a testcase, but isn't done yet. |
Running |
I found the problem: For serialized btrees |
Should be fixed by 09a2e18 |
I don't think this fix works. if i do for (auto it=tree.begin(); it != tree.end(); ++it) {...} It appears to fail as a new end node is constructed and deleted for every loop iteration. Also these lookups in unordered maps could be slow. Would it not be better to fix the comparison to use the offset? |
Works on my computer. |
We should not use unordred maps to cache nodes. We should fix the comparision.. |
Fair enough, should be fixed in 1b6534f |
Is this still a problem? |
The increment operation on the iterator in tpie::btree does not always terminate.
The text was updated successfully, but these errors were encountered: