Sure Sijie and Ivan. It sounds appropriate. Thanks, Charan
On Tue, Oct 24, 2017 at 1:00 AM, Sijie Guo <guosi...@gmail.com> wrote: > 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 >> > >