IGNITE-12357 Migrate Twitter module to ignite-extensions

[Saikat Maitra]

Hi,

I have raised PRs for migration of twitter module in ignite-extensions
repository.

Jira
https://issues.apache.org/jira/browse/IGNITE-12357

PRs
https://github.com/apache/ignite-extensions/pull/12
https://github.com/apache/ignite/pull/7733

Please review and share your feedback.

Regards,
Saikat

[vveider]

Please, remember to update TC configuration or create issue to do that.

> On 26 Apr 2020, at 20:40, Saikat Maitra <[hidden email]> wrote:
>
> Hi,
>
> I have raised PRs for migration of twitter module in ignite-extensions
> repository.
>
> Jira
> https://issues.apache.org/jira/browse/IGNITE-12357
>
> PRs
> https://github.com/apache/ignite-extensions/pull/12
> https://github.com/apache/ignite/pull/7733
>
> Please review and share your feedback.
>
> Regards,
> Saikat

[Saikat Maitra]

Hi Petr,

Yes, I have created an issue to update TC configuration once changes are
merged.

https://issues.apache.org/jira/browse/IGNITE-12960

Please let me know if the PR changes looks good.

Regards,
Saikat


On Mon, Apr 27, 2020 at 1:39 AM Petr Ivanov <[hidden email]> wrote:

> Please, remember to update TC configuration or create issue to do that.
>
>
> > On 26 Apr 2020, at 20:40, Saikat Maitra <[hidden email]> wrote:
> >
> > Hi,
> >
> > I have raised PRs for migration of twitter module in ignite-extensions
> > repository.
> >
> > Jira
> > https://issues.apache.org/jira/browse/IGNITE-12357
> >
> > PRs
> > https://github.com/apache/ignite-extensions/pull/12
> > https://github.com/apache/ignite/pull/7733
> >
> > Please review and share your feedback.
> >
> > Regards,
> > Saikat
>
>

[dmagda]

Saikat, please go ahead and merge the changes. They look good to me.

-
Denis


On Mon, Apr 27, 2020 at 4:09 PM Saikat Maitra <[hidden email]>
wrote:

> Hi Petr,
>
> Yes, I have created an issue to update TC configuration once changes are
> merged.
>
> https://issues.apache.org/jira/browse/IGNITE-12960
>
> Please let me know if the PR changes looks good.
>
> Regards,
> Saikat
>
>
> On Mon, Apr 27, 2020 at 1:39 AM Petr Ivanov <[hidden email]> wrote:
>
> > Please, remember to update TC configuration or create issue to do that.
> >
> >
> > > On 26 Apr 2020, at 20:40, Saikat Maitra <[hidden email]>
> wrote:
> > >
> > > Hi,
> > >
> > > I have raised PRs for migration of twitter module in ignite-extensions
> > > repository.
> > >
> > > Jira
> > > https://issues.apache.org/jira/browse/IGNITE-12357
> > >
> > > PRs
> > > https://github.com/apache/ignite-extensions/pull/12
> > > https://github.com/apache/ignite/pull/7733
> > >
> > > Please review and share your feedback.
> > >
> > > Regards,
> > > Saikat
> >
> >
>

[Saikat Maitra]

Thank you Denis for reviewing the changes.

Regards,
Saikat

On Wed, Apr 29, 2020 at 1:24 PM Denis Magda <[hidden email]> wrote:

> Saikat, please go ahead and merge the changes. They look good to me.
>
> -
> Denis
>
>
> On Mon, Apr 27, 2020 at 4:09 PM Saikat Maitra <[hidden email]>
> wrote:
>
> > Hi Petr,
> >
> > Yes, I have created an issue to update TC configuration once changes are
> > merged.
> >
> > https://issues.apache.org/jira/browse/IGNITE-12960
> >
> > Please let me know if the PR changes looks good.
> >
> > Regards,
> > Saikat
> >
> >
> > On Mon, Apr 27, 2020 at 1:39 AM Petr Ivanov <[hidden email]> wrote:
> >
> > > Please, remember to update TC configuration or create issue to do that.
> > >
> > >
> > > > On 26 Apr 2020, at 20:40, Saikat Maitra <[hidden email]>
> > wrote:
> > > >
> > > > Hi,
> > > >
> > > > I have raised PRs for migration of twitter module in
> ignite-extensions
> > > > repository.
> > > >
> > > > Jira
> > > > https://issues.apache.org/jira/browse/IGNITE-12357
> > > >
> > > > PRs
> > > > https://github.com/apache/ignite-extensions/pull/12
> > > > https://github.com/apache/ignite/pull/7733
> > > >
> > > > Please review and share your feedback.
> > > >
> > > > Regards,
> > > > Saikat
> > >
> > >
> >
>