Merge lp:~pbeaman/akiban-server/persistit-3.2.9-changes into lp:~akiban-technologies/akiban-server/trunk
Proposed by
Peter Beaman
Status: | Rejected |
---|---|
Rejected by: | Nathan Williams |
Proposed branch: | lp:~pbeaman/akiban-server/persistit-3.2.9-changes |
Merge into: | lp:~akiban-technologies/akiban-server/trunk |
Diff against target: |
85 lines (+8/-10) 3 files modified
pom.xml (+1/-1) src/main/java/com/akiban/server/AccumulatorAdapter.java (+4/-5) src/main/java/com/akiban/server/store/PersistitStoreSchemaManager.java (+3/-4) |
To merge this branch: | bzr merge lp:~pbeaman/akiban-server/persistit-3.2.9-changes |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Akiban Technologies | Pending | ||
Review via email: mp+158223@code.launchpad.net |
Description of the change
Minimal changes necessitated by Persistit Accumulator API changes proposed for Persistit 3.2.9. Note that the generic methods used by AccumulatorAdapter are deprecated in Persistit (and not removed specifically to avoid having to do more damage to the server code). Perhaps we should instead rethink the code structure, get rid of AccumulatorAdapter (or make one for each relevant subtype) and then get rid of the deprecated Persistit methods. Advice welcome.
To post a comment you must log in.
Unmerged revisions
- 2618. By Peter Beaman
-
Changes to support modified Accumulator API in Persistit 3.2.9
I have the code working in my branch fixing the logic, so I'll go ahead and incorporate the version bump there since it's touching a number of the same places.
Rethinking of AccumulatorAdapter is welcome, but I'd prefer another branch to reduce noise.