[GitHub] eolivelli commented on issue #774: Open should run recovery by default

2017-11-28 Thread GitBox
eolivelli commented on issue #774: Open should run recovery by default URL: https://github.com/apache/bookkeeper/pull/774#issuecomment-347462497 @ivankelly the new proposal of forceClose() sounds interesting to me. It is more explicit

[GitHub] eolivelli commented on issue #774: Open should run recovery by default

2017-11-27 Thread GitBox
eolivelli commented on issue #774: Open should run recovery by default URL: https://github.com/apache/bookkeeper/pull/774#issuecomment-347334346 @ivankelly when you are migrating your App you will have test cases I hope. I think that 'easy to use' depends on the case. If you are writing

[GitHub] eolivelli commented on issue #774: Open should run recovery by default

2017-11-27 Thread GitBox
eolivelli commented on issue #774: Open should run recovery by default URL: https://github.com/apache/bookkeeper/pull/774#issuecomment-347183565 I think that the core of BK is fencing/recovery + LAC, they are pretty unique and amazing features. When you are using BK you know what you

[GitHub] eolivelli commented on issue #774: Open should run recovery by default

2017-11-23 Thread GitBox
eolivelli commented on issue #774: Open should run recovery by default URL: https://github.com/apache/bookkeeper/pull/774#issuecomment-346683530 Initially I did do, but @sijie wanted this way. For me are ok both modes This