Merge lp:~pbeaman/akiban-persistit/optimize-value-get-2 into lp:akiban-persistit
Proposed by
Peter Beaman
Status: | Merged |
---|---|
Approved by: | Nathan Williams |
Approved revision: | 397 |
Merged at revision: | 397 |
Proposed branch: | lp:~pbeaman/akiban-persistit/optimize-value-get-2 |
Merge into: | lp:akiban-persistit |
Diff against target: |
417 lines (+41/-70) 14 files modified
src/main/java/com/persistit/Exchange.java (+3/-2) src/main/java/com/persistit/Persistit.java (+2/-2) src/main/java/com/persistit/Value.java (+1/-0) src/test/java/com/persistit/BackupTaskTest.java (+0/-1) src/test/java/com/persistit/Bug911849Test.java (+0/-1) src/test/java/com/persistit/Bug915594Test.java (+0/-2) src/test/java/com/persistit/Bug918909Test.java (+0/-2) src/test/java/com/persistit/Bug932097Test.java (+0/-2) src/test/java/com/persistit/Bug980292Test.java (+0/-2) src/test/java/com/persistit/DumpTaskTest.java (+1/-1) src/test/java/com/persistit/JournalManagerTest.java (+0/-1) src/test/java/com/persistit/TreeTest2.java (+0/-1) src/test/java/com/persistit/VolumeTest.java (+0/-1) src/test/java/com/persistit/unit/ValueTest3.java (+34/-52) |
To merge this branch: | bzr merge lp:~pbeaman/akiban-persistit/optimize-value-get-2 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Nathan Williams | Approve | ||
Peter Beaman | Needs Resubmitting | ||
Review via email: mp+136061@code.launchpad.net |
Description of the change
Add previously omitted call from directPut to preparePut.
Source code cleanup (small amount of formatting and removal of some obsolete imports)
To post a comment you must log in.
Presumably trunk could hit an issue without this? Can we add a test for it?
Wasn't the "canonical reformatting part 3" suppose to avoid 175 lines of diff for 1 line of change? :)