Code review comment for lp:~pbeaman/akiban-persistit/allow-long-keys

Revision history for this message
Peter Beaman (pbeaman) wrote :

After the Approval I determined that the use case in Akiban Server is much cleaner with a different API. So the new 3-arg constructor has been removed and instead there is a new method setMaximumSize(size) that reallocates the backing byte array and clears the state.

I was going to make that an additional new proposal, but I guess my changes got added before the existing proposal was merged.

At any rate, the server version that will be proposed shortly depends on this new API.

« Back to merge proposal