Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-03-27 Thread Saikat Maitra
Hi, I have updated the PR as per review feedback. Please take a look and share your feedback. Regards, Saikat On Sat, Mar 7, 2020 at 2:08 PM Saikat Maitra wrote: > Hi Evgenii, > > I have responded to your questions and comments. > > Please take a look and share your thoughts. > > Regards, >

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-03-07 Thread Saikat Maitra
Hi Evgenii, I have responded to your questions and comments. Please take a look and share your thoughts. Regards, Saikat On Wed, Feb 12, 2020 at 7:04 PM Evgenii Zhuravlev wrote: > Hi Saikat, > > I left a couple of comments in pr: > >

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-02-12 Thread Evgenii Zhuravlev
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 : > Hi, > > Can someone please help in review for these

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-02-11 Thread Saikat Maitra
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

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-02-11 Thread Saikat Maitra
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

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-02-10 Thread Alexey Goncharuk
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 : > Hi Alexey, > > Please let me

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-02-01 Thread Saikat Maitra
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:

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-01-26 Thread Saikat Maitra
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:

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-01-17 Thread Saikat Maitra
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.

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-01-17 Thread Alexey Goncharuk
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

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-01-15 Thread Saikat Maitra
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 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 :

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-01-15 Thread Alexey Goncharuk
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 : > Alex Goncharuk, Nikolay Izhikov, > > Could you possibly check the changes or suggest any other > committer/contributor for that? > > - > Denis > > > On Sun, Jan

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-01-13 Thread Denis Magda
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 wrote: > Hello, > > These PRs are part of Modularization effort > >

Re: IGNITE-12361 Migrate Flume module to ignite-extensions

2020-01-12 Thread Saikat Maitra
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

IGNITE-12361 Migrate Flume module to ignite-extensions

2020-01-05 Thread Saikat Maitra
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

[jira] [Created] (IGNITE-12361) Migrate Flume module to ignite-extensions

2019-11-09 Thread Saikat Maitra (Jira)
Saikat Maitra created IGNITE-12361: -- Summary: Migrate Flume module to ignite-extensions Key: IGNITE-12361 URL: https://issues.apache.org/jira/browse/IGNITE-12361 Project: Ignite Issue Type