Dmitry, Nikolay, Ignite-3303 is a new Ignite module and there is no changes related to core or other existed modules. So, PR should not affected existed functional ity and can be safely merged.
Thanks. пн, 1 окт. 2018 г., 16:04 Dmitriy Pavlov <dpavlov....@gmail.com>: > Hi Saikat, > > I don't mind merging to master, but I have concern if it will go to 2.7. In > the separate discussion, we agreed on code freeze should happen during last > weekend, September, 30. > > So it is now up to community and release manager to decide if fix should go > to the upcoming release. Usually, after the freeze, only bug/test fixes can > be merged to release branch. > > Hi Nikolay, > > could you please announce that code freeze happened? > > Sincerely, > Dmitriy Pavlov > > пн, 1 окт. 2018 г. в 3:58, Saikat Maitra <saikat.mai...@gmail.com>: > > > Hi Alex, Nicolay > > > > As discussed with Andrew the changes looks good. Would it be ok to merge > > this change to master considering the 2.7 release plan? > > > > Regards, > > Saikat > > > > On Fri, Sep 28, 2018 at 7:15 PM Saikat Maitra <saikat.mai...@gmail.com> > > wrote: > > > > > Thank you Andrew > > > > > > Regards, > > > Saikat > > > > > > On Fri, Sep 28, 2018 at 7:00 PM Andrey Mashenkov < > > > andrey.mashen...@gmail.com> wrote: > > > > > >> Hi Saikat, > > >> > > >> Sorry for late answer. I've checked changes a day ago. Now, looks > good. > > >> Hope, it will be merged soon. > > >> > > >> Alex, would you please merge PR to master. > > >> > > >> сб, 29 сент. 2018 г., 2:29 Saikat Maitra <saikat.mai...@gmail.com>: > > >> > > >> > Hi Andrew, > > >> > > > >> > I have updated the changes. > > >> > > > >> > Can you please review and share feedback. > > >> > > > >> > Regards > > >> > Saikat > > >> > > > >> > On Sat, Sep 22, 2018 at 2:23 PM Saikat Maitra < > > saikat.mai...@gmail.com> > > >> > wrote: > > >> > > > >> > > Hi Andrew > > >> > > > > >> > > > > >> > > I have updated the changes. > > >> > > > > >> > > > > >> > > Can you please review and share feedback. > > >> > > > > >> > > > > >> > > Regards > > >> > > Saikat > > >> > > > > >> > > > > >> > > On Wed, Sep 19, 2018 at 8:11 PM, Saikat Maitra < > > >> saikat.mai...@gmail.com> > > >> > > wrote: > > >> > > > > >> > >> Hi Andrew, > > >> > >> > > >> > >> I have updated the tests and also added java docs. > > >> > >> > > >> > >> Can you please review and share feedback. > > >> > >> > > >> > >> > > >> > >> Regards > > >> > >> Saikat > > >> > >> > > >> > >> > > >> > >> > > >> > >> > > >> > >> On Sun, Sep 16, 2018 at 11:53 AM, Saikat Maitra < > > >> > saikat.mai...@gmail.com> > > >> > >> wrote: > > >> > >> > > >> > >>> Hi Andrew, > > >> > >>> > > >> > >>> I have updated the tests and also added java docs. > > >> > >>> > > >> > >>> Please review and share feedback. > > >> > >>> > > >> > >>> Regards > > >> > >>> Saikat > > >> > >>> > > >> > >>> > > >> > >>> On Sat, Sep 8, 2018 at 2:09 PM, Saikat Maitra < > > >> saikat.mai...@gmail.com > > >> > > > > >> > >>> wrote: > > >> > >>> > > >> > >>>> Hi Andrew, Alexey > > >> > >>>> > > >> > >>>> I have incorporated the review changes. > > >> > >>>> > > >> > >>>> I have also refactored the CacheEventSerializer class and moved > > it > > >> to > > >> > >>>> test folder because it is used only in the > > >> > FlinkIgniteSourceSelfExample and > > >> > >>>> not required for IgniteSource. > > >> > >>>> > > >> > >>>> Build links > > >> > https://ci.ignite.apache.org/viewLog.html?buildId=1821778&; > > >> > >>>> > > >> > >>>> https://ci.ignite.apache.org/viewLog.html?buildId=1821774&; > > >> > >>>> > > >> > >>>> Please review and share feedback. > > >> > >>>> > > >> > >>>> Regards > > >> > >>>> Saikat > > >> > >>>> > > >> > >>>> On Tue, Sep 4, 2018 at 9:57 PM, Saikat Maitra < > > >> > saikat.mai...@gmail.com> > > >> > >>>> wrote: > > >> > >>>> > > >> > >>>>> Hi Alexey, > > >> > >>>>> > > >> > >>>>> Thank you for reviewing the changes and sharing feedback, I am > > >> > >>>>> updating the PR. I will share the changes shortly. > > >> > >>>>> > > >> > >>>>> Regards, > > >> > >>>>> Saikat > > >> > >>>>> > > >> > >>>>> On Tue, Sep 4, 2018 at 10:59 AM, Alexey Goncharuk < > > >> > >>>>> alexey.goncha...@gmail.com> wrote: > > >> > >>>>> > > >> > >>>>>> Hello Saikat, > > >> > >>>>>> > > >> > >>>>>> I see a few fellow Igniters added some comments to your PR > > >> > (including > > >> > >>>>>> me). > > >> > >>>>>> I believe the PR can be merged after you address them. > > >> > >>>>>> > > >> > >>>>>> Thanks, > > >> > >>>>>> AG > > >> > >>>>>> > > >> > >>>>>> пт, 31 авг. 2018 г. в 3:11, Saikat Maitra < > > >> saikat.mai...@gmail.com > > >> > >: > > >> > >>>>>> > > >> > >>>>>> > Thank you, Denis > > >> > >>>>>> > > > >> > >>>>>> > Regards, > > >> > >>>>>> > Saikat > > >> > >>>>>> > > > >> > >>>>>> > On Thu, Aug 30, 2018 at 7:01 PM, Denis Magda < > > >> dma...@apache.org> > > >> > >>>>>> wrote: > > >> > >>>>>> > > > >> > >>>>>> > > Hello Saikat, > > >> > >>>>>> > > > > >> > >>>>>> > > Hopefully, someone from the community will review the > > >> changes in > > >> > >>>>>> the > > >> > >>>>>> > > nearest time. > > >> > >>>>>> > > > > >> > >>>>>> > > -- > > >> > >>>>>> > > Denis > > >> > >>>>>> > > > > >> > >>>>>> > > On Thu, Aug 30, 2018 at 4:37 PM Saikat Maitra < > > >> > >>>>>> saikat.mai...@gmail.com> > > >> > >>>>>> > > wrote: > > >> > >>>>>> > > > > >> > >>>>>> > > > Hello, > > >> > >>>>>> > > > > > >> > >>>>>> > > > The changes for IGNITE-3303 for IgniteSource is > complete. > > >> This > > >> > >>>>>> will > > >> > >>>>>> > help > > >> > >>>>>> > > is > > >> > >>>>>> > > > streaming data from Ignite cluster and process, filter, > > >> > >>>>>> transform and > > >> > >>>>>> > > > publish it back to Ignite using IgniteSink or in any > > other > > >> > data > > >> > >>>>>> sink. > > >> > >>>>>> > > > > > >> > >>>>>> > > > I was hoping if the changes can be approved I can go > > ahead > > >> > >>>>>> merge the > > >> > >>>>>> > > > changes. > > >> > >>>>>> > > > > > >> > >>>>>> > > > > > >> > >>>>>> > > > Regards, > > >> > >>>>>> > > > Saikat > > >> > >>>>>> > > > > > >> > >>>>>> > > > > > >> > >>>>>> > > > > > >> > >>>>>> > > > On Tue, Aug 28, 2018 at 12:56 AM, Saikat Maitra < > > >> > >>>>>> > saikat.mai...@gmail.com > > >> > >>>>>> > > > > > >> > >>>>>> > > > wrote: > > >> > >>>>>> > > > > > >> > >>>>>> > > > > Hi Andrew, > > >> > >>>>>> > > > > > > >> > >>>>>> > > > > As discussed I have incorporated the changes. Please > > >> review > > >> > >>>>>> and let > > >> > >>>>>> > me > > >> > >>>>>> > > > > know if any changes required. > > >> > >>>>>> > > > > > > >> > >>>>>> > > > > Regards, > > >> > >>>>>> > > > > Saikat > > >> > >>>>>> > > > > > > >> > >>>>>> > > > > On Mon, Aug 27, 2018 at 1:45 AM, Saikat Maitra < > > >> > >>>>>> > > saikat.mai...@gmail.com> > > >> > >>>>>> > > > > wrote: > > >> > >>>>>> > > > > > > >> > >>>>>> > > > >> Hi, > > >> > >>>>>> > > > >> > > >> > >>>>>> > > > >> I have updated the PR with additional tests. > > >> > >>>>>> > > > >> > > >> > >>>>>> > > > >> Please review and share feedback. > > >> > >>>>>> > > > >> > > >> > >>>>>> > > > >> This PR is related to IgniteSink but allows to > stream > > >> data > > >> > >>>>>> from > > >> > >>>>>> > > Ignite. > > >> > >>>>>> > > > >> > > >> > >>>>>> > > > >> PR https://github.com/apache/ignite/pull/870/files > > >> > >>>>>> > > > >> > > >> > >>>>>> > > > >> Review > > >> > >>>>>> https://reviews.ignite.apache.org/ignite/review/IGNT-CR-135 > > >> > >>>>>> > > > >> > > >> > >>>>>> > > > >> Regards, > > >> > >>>>>> > > > >> Saikat > > >> > >>>>>> > > > >> > > >> > >>>>>> > > > > > > >> > >>>>>> > > > > > > >> > >>>>>> > > > > > >> > >>>>>> > > > > >> > >>>>>> > > > >> > >>>>>> > > >> > >>>>> > > >> > >>>>> > > >> > >>>> > > >> > >>> > > >> > >> > > >> > > > > >> > > > >> > > > > > >
