Re: [VOTE] Add micronaut dependency to Ignite 3

2022-05-27 Thread Aleksandr Pakhomov
Andrey, Thank you, I will start a new vote soon. > On 27 May 2022, at 17:29, Andrey Gura wrote: > > Aleksandr, > > please, start a new vote after updating the IEP-87. I think that > Micronaut Security is a good reason for using this framework in the > Apache Ignite. > > > > On Wed, May 25,

Re: [DISCUSSION] IEP-89: PITR

2022-05-27 Thread Maksim Timonin
Hi Roman, Thanks for your comments! Yes, at the beginning I considered using a single class for two types of records. But it is confusing, as you're correctly noticed. I reworked IEP, now there are 2 different classes, and both contain a list of transactions to include. Also I added info about whe

Re: [VOTE] Add micronaut dependency to Ignite 3

2022-05-27 Thread Andrey Gura
Aleksandr, please, start a new vote after updating the IEP-87. I think that Micronaut Security is a good reason for using this framework in the Apache Ignite. On Wed, May 25, 2022 at 5:41 PM Andrey Gura wrote: > > -1 (binding) from me. > > Because (if I understood correctly) the main value of

Re: [DISCUSSION] Add additional 3rd party libraries to Ignite 3 Code style and practices

2022-05-27 Thread Andrey Gura
Hi, Hmmm... It seems that [2] is just a marketing page that doesn't give any arguments about the micronaut. Anyway, security is a good point. Aleksandr, could you please add some note to the IEP about security related value of the micronaut framework? On Thu, May 26, 2022 at 9:48 AM Mikhail

Re: [DISCUSSION] IEP-88: CLI Tool

2022-05-27 Thread Aleksandr Pakhomov
Roman, Thank you, that makes sense. > On 27 May 2022, at 14:26, Roman Puchkovskiy > wrote: > > Aleksandr, > > The thing is that `cluster init` is not just for setting some kind of > a configuration, it's more about doing cluster initialization > described in [1]. This init process transitio

Re: [DISCUSSION] IEP-88: CLI Tool

2022-05-27 Thread Roman Puchkovskiy
Aleksandr, The thing is that `cluster init` is not just for setting some kind of a configuration, it's more about doing cluster initialization described in [1]. This init process transitions the cluster from 'empty' state to 'initialized state'; this can be only made once per cluster, and it has t

Re: [DISCUSSION] IEP-88: CLI Tool

2022-05-27 Thread Aleksandr Pakhomov
Hi Roman, That is a good point. In the proposal I mean the analogue for existing ‘cluster init’. Maybe “distributed configuration” confuses you and probably I have to name it like “meta-storage” configuration or something like this. As for 'ignite init’ I think it would be more clearer if we

Re: [DISCUSSION] IEP-88: CLI Tool

2022-05-27 Thread Roman Puchkovskiy
Hi Aleksandr. There is a command named 'init' in your proposal. According to its description, it initializes the cluster with a distributed configuration. I'm not sure how it's mapped to the existing commands. The thing is that currently, there is `ignite init` command that initializes (actually,