Hi Saikat, I left a couple of comments in pr: https://github.com/apache/ignite-extensions/pull/4#pullrequestreview-357891629. Please tell me what do you think about it.
Best Regards, Evgenii вт, 11 февр. 2020 г. в 17:15, Saikat Maitra <saikat.mai...@gmail.com>: > Hi, > > Can someone please help in review for these following PRs. I have > received approval for release process from Alexey and would need a code > review approval for following PR. > > 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 > > Regards, > Saikat > > On Tue, Feb 11, 2020 at 6:47 PM Saikat Maitra <saikat.mai...@gmail.com> > wrote: > > > Hi Alexey, > > > > > > I think we can release for spring boot autoconfigure module. > > > > Nikolay - Do you have tentative timeline when you are planning for > release > > of spring boot autoconfigure module. > > > > > > After that we are planning to make release for flink ext. > > > > > > Since, each module are independent so they will be released > independently. > > > > > > Regards, > > Saikat > > > > On Mon, 10 Feb 2020 at 7:33 AM, Alexey Goncharuk < > > alexey.goncha...@gmail.com> wrote: > > > >> Saikat, > >> > >> Yes, I think we can go ahead with the modules PRs as long as reviewers > are > >> ok with the changes. Given that there is an activity around the spring > >> module, which modules do you think will get to the first release? > >> > >> сб, 1 февр. 2020 г. в 21:37, Saikat Maitra <saikat.mai...@gmail.com>: > >> > >> > Hi Alexey, > >> > > >> > Please let me know if I can share more info on the release process. I > >> have > >> > updated the issue confluence page on discussed approach for Ignite > >> > Extensions. Do you think the open PRs can be merged in Ignite > Extensions > >> > repo? > >> > > >> > Independent Integrations: > >> > > >> > > >> > https://cwiki.apache.org/confluence/display/IGNITE/IEP-36%3A+Modularization#IEP-36:Modularization-IndependentIntegrations > >> > Discussion Links: > >> > > >> > > >> > https://cwiki.apache.org/confluence/display/IGNITE/IEP-36%3A+Modularization#IEP-36:Modularization-DiscussionLinks > >> > Tickets: > >> > > >> > > >> > https://cwiki.apache.org/confluence/display/IGNITE/IEP-36%3A+Modularization#IEP-36:Modularization-Tickets > >> > > >> > Regards, > >> > Saikat > >> > > >> > On Sun, Jan 26, 2020 at 3:11 PM Saikat Maitra < > saikat.mai...@gmail.com> > >> > wrote: > >> > > >> > > Hi Alexey, > >> > > > >> > > As discussed I have updated the wiki with agreed solution. > >> > > > >> > > Independent Integrations: > >> > > > >> > > >> > https://cwiki.apache.org/confluence/display/IGNITE/IEP-36%3A+Modularization#IEP-36:Modularization-IndependentIntegrations > >> > > > >> > > Discussion Links: > >> > > > >> > > >> > https://cwiki.apache.org/confluence/display/IGNITE/IEP-36%3A+Modularization#IEP-36:Modularization-DiscussionLinks > >> > > > >> > > Tickets: > >> > > > >> > > >> > https://cwiki.apache.org/confluence/display/IGNITE/IEP-36%3A+Modularization#IEP-36:Modularization-Tickets > >> > > > >> > > Please let me know if I can share more information. > >> > > > >> > > Regards, > >> > > Saikat > >> > > > >> > > > >> > > On Fri, Jan 17, 2020 at 9:16 PM Saikat Maitra < > >> saikat.mai...@gmail.com> > >> > > wrote: > >> > > > >> > >> Hello Alexey, > >> > >> > >> > >> Thank you for your email. > >> > >> > >> > >> 1. Yes, we discussed in dev list and agreed on creating a new > >> repository > >> > >> for hosting our Ignite integrations. Please find the discussion > >> thread > >> > >> below. I will update the wiki page as well and share updates. > >> > >> > >> > >> > >> > >> > >> > > >> > http://apache-ignite-developers.2346864.n4.nabble.com/DISCUSS-Proposal-for-Ignite-Extensions-as-a-separate-Bahir-module-or-Incubator-project-td44064.html > >> > >> > >> > >> 2. I was hoping to complete migration of the following modules > >> before we > >> > >> go ahead with release. I am tracking the jira story here > >> > >> https://issues.apache.org/jira/browse/IGNITE-12355 > >> > >> > >> > >> - Flink > >> > >> - Twitter > >> > >> - Storm > >> > >> - ZeroMQ > >> > >> - RocketMQ > >> > >> - Flume > >> > >> - MQTT > >> > >> - Camel > >> > >> - JMS > >> > >> > >> > >> 3. The dependencies for modules are pointing to latest snapshot of > >> > >> ignite project and if there are changes in ignite master branch > then > >> > >> related affected Ignite extensions module also need to be modified. > >> We > >> > will > >> > >> verify all the extensions for upcoming release but release only the > >> one > >> > >> that are impacted. We will plan to avoid publishing any extension > >> unless > >> > >> there are changes. Here is the discussion thread on release > process: > >> > >> > >> > >> > >> > >> > >> > > >> > http://apache-ignite-developers.2346864.n4.nabble.com/DISCUSS-dependencies-and-release-process-for-Ignite-Extensions-td44478.html > >> > >> > >> > >> 4. Sounds good, we can maintain a compatibility matrix to ensure we > >> can > >> > >> share Ignite Core and related Ignite Extensions. I have seen > >> something > >> > >> similar in Apache Beam and we can also create a similar wiki page > to > >> > >> maintain Ignite Core and Ignite Extensions dependencies > >> > >> > >> > >> https://beam.apache.org/documentation/runners/capability-matrix/ > >> > >> > >> > >> Regards, > >> > >> Saikat > >> > >> > >> > >> > >> > >> > >> > >> On Fri, Jan 17, 2020 at 6:27 AM Alexey Goncharuk < > >> > >> alexey.goncha...@gmail.com> wrote: > >> > >> > >> > >>> 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 > >> > >>> > > >> > > >> > >>> > > >> > >> > >>> > > > > >> > >>> > > > >> > >>> > > >> > >>> > >> > >> > >> > > >> > > >