basho / leveldb

Clone of http://code.google.com/p/leveldb/
BSD 3-Clause "New" or "Revised" License
408 stars 182 forks source link

Only switch manifest format if expiry actually used #205

Closed matthewvon closed 7 years ago

matthewvon commented 7 years ago

Actively set second param of VersionEdit.EncodeTo() based upon whether or not expiry module is active. The flag controls whether manifest is traditional format or new Expiry enabled format which is NOT backward compatible.

Implementation created a generic ExpiryActivated() const function in the Options and ExpiryModule classes. Adjusted code everywhere to use this accessor function instead of explicit tests.

fadushin commented 7 years ago

+1 9d260b8