Saikat, Code-wise the PR looks ok because it basically moves the module to another repo. I have some infrastructure/process questions though before we merge the PRs * I see that there are some objections from Alexey Zinoviev [1] on whether the streaming modules should be placed in extensions or in a separate streaming project. Sorry if I missed the corresponding discussion on the dev-list. If there is one, we need to edit the IEP page, if not, I think we should come to a consensus before making this change * When are we planning to release the extensions? Are we going to release them one-by-one incrementally or do we want to release them in one batch? * Now that the release lifecycle of Ignite and extensions is split, we need to understand how this is going to be tested. Will we run a candidate Ignite build against already released extensions? What if we make changes to both Ignite and an extension? Is it even allowed, or do we need to wait for an Ignite release before making any changes in an extension? * I think there should be some sort of compatibility matrix now that releases of Ignite and extensions are separated
I would not delete the extension from Ignite codebase until we agree on the points above. Would you mind taking care of the discussion about it and formalizing it on the wiki? чт, 16 янв. 2020 г. в 06:27, Saikat Maitra <saikat.mai...@gmail.com>: > Hi Denis, Alexey > > Thank you so much for your email. I really appreciate it. > > Regards, > Saikat > > On Wed, Jan 15, 2020 at 4:11 AM Alexey Goncharuk < > alexey.goncha...@gmail.com> > wrote: > > > Saikat, > > > > Thanks for working on this! I'll do my best to take a look at it this > week. > > > > --AG > > > > пн, 13 янв. 2020 г. в 21:37, Denis Magda <dma...@apache.org>: > > > > > Alex Goncharuk, Nikolay Izhikov, > > > > > > Could you possibly check the changes or suggest any other > > > committer/contributor for that? > > > > > > - > > > Denis > > > > > > > > > On Sun, Jan 12, 2020 at 9:31 AM Saikat Maitra <saikat.mai...@gmail.com > > > > > wrote: > > > > > >> Hello, > > >> > > >> These PRs are part of Modularization effort > > >> > > >> > > > https://cwiki.apache.org/confluence/display/IGNITE/IEP-36%3A+Modularization#IEP-36:Modularization-IndependentIntegrations > > >> > > >> If the changes looks good, I can go ahead and merge the changes. > > >> > > >> Regards, > > >> Saikat > > >> > > >> On Sun, Jan 5, 2020 at 1:08 PM Saikat Maitra <saikat.mai...@gmail.com > > > > >> wrote: > > >> > > >> > Hi, > > >> > > > >> > I have raised PR for Ignite Flume migration to Ignite Extensions > repo. > > >> > > > >> > Jira https://issues.apache.org/jira/browse/IGNITE-12361 > > >> > > > >> > PR https://github.com/apache/ignite-extensions/pull/4 > > >> > https://github.com/apache/ignite/pull/7227 > > >> > > > >> > Please review and share feedback. > > >> > > > >> > This is part of our Modularization effort for Streamer modules > > >> > > > >> > https://issues.apache.org/jira/browse/IGNITE-12355 > > >> > > > >> > Regards, > > >> > Saikat > > >> > > > >> > > > > > >
