As suggested before, please do not put blank ticket numbers into subjects because no one understands ticket numbers. Please add titles or some other context to the subject. This will improve the level of engagement from the community.
I have changed the subject of this thread, let's continue the discussion here. D. On Tue, May 29, 2018 at 9:10 AM, Александр Меньшиков <sharple...@gmail.com> wrote: > Hi, Andrew. > > You suggested continuing the discussion on dev-list about IGNITE-8238 [1] > I have reworked my PR [2]. Have I moved in the right direction? > > I see 57 usages of `checkpointReadLock` in the core module (without tests). > And it still unclear for me should I catch all exceptions from > `checkpointReadLock` or not? > Some places look okay like usage inside overrides of `GridWorker#body`. > But for example `GridCacheAdapter#getAllAsync0` makes me worry. > > [1] https://issues.apache.org/jira/browse/IGNITE-8238 > [2] https://github.com/apache/ignite/pull/3993/files >