Yes, we should fix this ASAP. Charan, what is your opinion on the fix?
- Sijie On Tue, Oct 24, 2017 at 12:12 AM, Ivan Kelly <iv...@apache.org> wrote: > > Use the approach that I used before (as in the old commits at twitter’s > > branch). Ledger storage is responsible for instantiating the checkpoints. > I would go with this approach. It was probably me that asked for the > changes in the other direction before, but I can't remember why I > asked for them, so the view held strongly then isn't held strongly > now. > > It would be good to get this change in soon, so that when the yahoo > DbLedgerStorage goes in, we can verify that checkpointing works as > expected. > > -Ivan >
