traverse(EQ, false, 0) returns incorrect result
Bug #1023549 reported by
Peter Beaman
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Akiban Persistit |
Fix Released
|
Medium
|
Peter Beaman | ||
Bug Description
This method returns true even when the tree is empty. traverse(EQ, true, 0) returns the correct value.
Related branches
lp:~pbeaman/akiban-persistit/fix-several-small-bugs
- Nathan Williams: Pending requested
-
Diff: 770 lines (+287/-80)18 files modifiedsrc/main/java/com/persistit/BufferPool.java (+6/-5)
src/main/java/com/persistit/CheckpointManager.java (+2/-2)
src/main/java/com/persistit/JournalManager.java (+3/-20)
src/main/java/com/persistit/JournalManagerBench.java (+2/-1)
src/main/java/com/persistit/Persistit.java (+53/-27)
src/main/java/com/persistit/SessionId.java (+10/-0)
src/main/java/com/persistit/Transaction.java (+0/-1)
src/main/java/com/persistit/TransactionIndexBucket.java (+5/-9)
src/main/java/com/persistit/TransactionStatus.java (+20/-0)
src/main/java/com/persistit/VolumeHeader.java (+9/-1)
src/main/java/com/persistit/logging/LogBase.java (+3/-0)
src/main/java/com/persistit/util/Util.java (+4/-0)
src/test/java/com/persistit/IOFailureTest.java (+2/-0)
src/test/java/com/persistit/TransactionTest2.java (+43/-9)
src/test/java/com/persistit/stress/AbstractStressTest.java (+3/-1)
src/test/java/com/persistit/stress/AbstractSuite.java (+4/-4)
src/test/java/com/persistit/stress/StartStop.java (+2/-0)
src/test/java/com/persistit/unit/ExchangeTest.java (+116/-0)
lp:~pbeaman/akiban-persistit/fix-1023549-traverse-wrong
- Nathan Williams: Approve
-
Diff: 95 lines (+28/-7)2 files modifiedsrc/main/java/com/persistit/Exchange.java (+16/-7)
src/test/java/com/persistit/unit/ExchangeTest.java (+12/-0)
| Changed in akiban-persistit: | |
| milestone: | none → 3.1.8 |
| assignee: | nobody → Peter Beaman (pbeaman) |
Revision history for this message
| Peter Beaman (pbeaman) wrote | #1 |
| Changed in akiban-persistit: | |
| status: | New → Invalid |
Revision history for this message
| Peter Beaman (pbeaman) wrote | #2 |
| Changed in akiban-persistit: | |
| status: | Invalid → Confirmed |
| Changed in akiban-persistit: | |
| status: | Confirmed → In Progress |
| visibility: | private → public |
| Changed in akiban-persistit: | |
| status: | In Progress → Fix Committed |
| Changed in akiban-persistit: | |
| milestone: | 3.2.0 → 3.2.1 |
| Changed in akiban-persistit: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Bug is incorrect: traverse(EQ, false, 0) appears to return the correct result. Added unit test for a number of variations of EQ, LTEQ, GTEQ, LT and GT.