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

Bug on Iterator.Prev() #29

Closed
GoogleCodeExporter opened this issue May 3, 2015 · 4 comments
Closed

Bug on Iterator.Prev() #29

GoogleCodeExporter opened this issue May 3, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

The testcase is attached.

Here's the output:
***** Running db_test
==== Test DBTest.Empty
==== Test DBTest.ReadWrite
==== Test DBTest.PutDeleteGet
==== Test DBTest.GetFromImmutableLayer
==== Test DBTest.GetFromVersions
==== Test DBTest.GetSnapshot
==== Test DBTest.GetLevel0Ordering
==== Test DBTest.GetOrderedByLevels
==== Test DBTest.GetPicksCorrectFile
==== Test DBTest.IterEmpty
==== Test DBTest.IterSingle
==== Test DBTest.IterMulti
==== Test DBTest.IterMultiWithDelete
db/db_test.cc:497: failed: b->va == a->va
make: *** [check] Error 1


Original issue reported on code.google.com by david.yu...@gmail.com on 12 Aug 2011 at 6:15

Attachments:

@GoogleCodeExporter
Copy link
Author

Attaching the fix.
All tests seem to pass.
Please verify.

Original comment by david.yu...@gmail.com on 12 Aug 2011 at 7:01

Attachments:

@GoogleCodeExporter
Copy link
Author

Hi David, thanks for the bug report and the fix!

We'd like to patch this into the LevelDB code - do you mind filling out this 
CLA so we can patch it in?
http://code.google.com/legal/individual-cla-v1.0.html
Let me know when you've completed this.

Thanks for your help,

Gabor

Original comment by ga...@google.com on 15 Aug 2011 at 12:18

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

I've just completed the online form.
Thanks

Original comment by david.yu...@gmail.com on 15 Aug 2011 at 5:40

@GoogleCodeExporter
Copy link
Author

Thanks.

This is fixed in r42.

Original comment by ga...@google.com on 16 Aug 2011 at 1:45

  • Changed state: Fixed

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