Alexey,
>From my point of view, the feature is fully self-sufficient and ready for a release (with a small caveat): - administrators will be able to create snapshots without writing java-code; - developers will be able to create snapshots through java API; The documentation pages for creating and restoring procedures with examples will be completed by me prior to release this feature for our end-users. All other features mentioned in this list [1] adds convenience for users but not mandatory. I'll try to finish these tasks from the list [1] prior to release: - support snapshot creation from a client node - add starting snapshot via control.sh Are there any details I've missed? [1] https://github.com/apache/ignite/pull/7607#issuecomment-618964647 On Mon, 27 Apr 2020 at 18:12, Alexey Goncharuk <alexey.goncha...@gmail.com> wrote: > > Maxim, > > I saw the list of the tickets you want to work on in the PR, it looks nice. > I was wondering, what part of that list are you planning to implement > before the feature is released to end users? For example, I agree with > Slava that we should implement a command-line utility part for snapshots > before the release, however I think it's better to do it in a separate > ticket. > > I know we do not have a strict policy regarding big features development in > the community, so perhaps it's a good time to discuss this? If we are ok > with merging separate tickets to master, how we ensure a complete feature > is released to public? If not, should we create a feature branch and wait > for all related tickets to be merged there? Will be glad to discuss this in > a separate thread if needed. > > пн, 27 апр. 2020 г. в 14:38, Maxim Muzafarov <mmu...@apache.org>: > > > Folks, > > > > > > Are there any cases left which we need to discuss? > > > > Do you have any questions? > > I'm ready to provide all the details you need for the review. > > > > Who else what to take a look at my changes [1] [2]? > > > > > > [1] https://issues.apache.org/jira/browse/IGNITE-11073 > > [2] https://github.com/apache/ignite/pull/7607 > > > > On Fri, 24 Apr 2020 at 15:01, Maxim Muzafarov <mmu...@apache.org> wrote: > > > > > > Alexey, > > > > > > > > > I've addressed all your comments, please, take a look at the PR [1]. > > > Additional tests were added. > > > Additional comments with further steps were added. > > > > > > > > > [1] https://github.com/apache/ignite/pull/7607 > > > [2] https://issues.apache.org/jira/browse/IGNITE-11073 > > > > > > On Tue, 21 Apr 2020 at 09:53, Alexey Goncharuk > > > <alexey.goncha...@gmail.com> wrote: > > > > > > > > Maxim, > > > > > > > > I've left my comments in the PR. > > > > > > > > пн, 20 апр. 2020 г. в 12:52, Maxim Muzafarov <mmu...@apache.org>: > > > > > > > > > Alex P, > > > > > Thank you for the great sophisticated review. > > > > > > > > > > > > > > > Alexey G, > > > > > Will you take a look at my changes[1]? > > > > > The fresh TC.Bot visa attached. > > > > > > > > > > > > > > > [1] https://issues.apache.org/jira/browse/IGNITE-11073 > > > > > > > > > > On Mon, 20 Apr 2020 at 11:54, Alex Plehanov <plehanov.a...@gmail.com > > > > > > > > wrote: > > > > > > > > > > > > Maxim, I've reviewed your PR and it looks good to me. Good job! > > > > > > > > > > > > пт, 10 апр. 2020 г. в 19:43, Alexey Goncharuk < > > > > > alexey.goncha...@gmail.com>: > > > > > > > > > > > > > Maxim, > > > > > > > > > > > > > > Thanks for raising this PR. I will do a review during next week. > > > > > > > > > > > > > > --AG > > > > > > > > > > > > > >
