Re: Exchange stucks while node restoring state from WAL

2018-10-29 Thread Maxim Muzafarov
Pavel, Dmitry G., Thank you for the help and review this improvement! Igniters, I'd like to finalize this topic with the following changes [1] `Update Grid(Ignite)CacheDatabaseSharedManager according to accepted coding guidelines`. I've consciously excluded them from the original PR not to

Re: Exchange stucks while node restoring state from WAL

2018-09-18 Thread Maxim Muzafarov
Igniters, I would like this issue to be part of release 2.7. There is not much time left considering that I will have to correct comments after the review. Will anyone help with review? PR: https://github.com/apache/ignite/pull/4520/files Upsource:

Re: Exchange stucks while node restoring state from WAL

2018-09-13 Thread Maxim Muzafarov
Igniters, I need your help with the review of this patch. In general, it will help to reduce PME duration. I've moved binary memory recovery at the moment of node startup as we've discussed it with Pavel previously this topic. Changes are relatively small (+299 −95) and they are ready. I've left

Re: Exchange stucks while node restoring state from WAL

2018-08-28 Thread Pavel Kovalenko
Hello Maxim, I think you're going in the right direction, but why you perform binary recovery only in the case when a node in the baseline? I think that node can also perform binary recovery before the first exchange is started in case of 1) First activation 2) Baseline change In all of these

Re: Exchange stucks while node restoring state from WAL

2018-08-22 Thread Pavel Kovalenko
Hello Maxim, Thank you for your work. I will review your changes within the next several days. 2018-08-22 11:12 GMT+03:00 Maxim Muzafarov : > Pavel, > > Thank you for so detailed introduction into the root of the problem of > ticket IGNITE-7196. > As you mentioned before, we can divide this

Re: Exchange stucks while node restoring state from WAL

2018-08-22 Thread Maxim Muzafarov
Pavel, Thank you for so detailed introduction into the root of the problem of ticket IGNITE-7196. As you mentioned before, we can divide this ticket into two sub-tasks. So, I will file new issue for `Logical consistency recovery`. I think it's more convenient way for reviewing and merging each of

Re: Exchange stucks while node restoring state from WAL

2018-08-03 Thread Pavel Kovalenko
Hello Maxim, 1) Yes, Discovery Manager is starting after GridCacheProcessor, which starts GridCacheDatabaseSharedManager which invokes readMetastorage on start. 2) Before we complete the local join future, we create and add Exchange future on local node join to ExchangeManager. So, when local

Re: Exchange stucks while node restoring state from WAL

2018-08-03 Thread Nikolay Izhikov
Hello, Maxim. > 1) Is it correct that readMetastore() happens after node starts> but before > including node into the ring?> I think yes. You can have some kind of metainformation required on node join. > 5) Does in our final solution for new joined node readMetastore> and > restoreMemory

Exchange stucks while node restoring state from WAL

2018-08-03 Thread Maxim Muzafarov
Hi Igniters, I'm working on bug [1] and have some questions about the final implementation. Probably, I've already found answers on some of them but I want to be sure. Please, help me to clarify details. The key problem here is that we are reading WAL and restoring memory state of new joined