Review request for IGNITE-8740: Support reuse of already initialized Ignite in IgniteSpringBean

[Amir Akhmedov]

Hi All,
Can you please review my changes for IGNITE-8740.

PR: https://github.com/apache/ignite/pull/4208
TC:
https://ci.ignite.apache.org/viewLog.html?buildId=1397283&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunAll

Thanks,
Amir

[Dmitriy Pavlov]

Hi Amir,

let me say sincere thank you for continuing to contribute.

Bumping up this thread.

Igniters, who has an expertise here?


вс, 17 июн. 2018 г. в 17:59, Amir Akhmedov <[hidden email]>:

> Hi All,
> Can you please review my changes for IGNITE-8740.
>
> PR: https://github.com/apache/ignite/pull/4208
> TC:
>
> https://ci.ignite.apache.org/viewLog.html?buildId=1397283&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunAll
>
> Thanks,
> Amir
>

[Valentin Kulichenko]

Hi Amir,

I reviewed the changes and I'm not sure I understood how they fix they
issue. I left more detailed comment in the ticket, can you please clarify?

-Val

On Fri, Jun 22, 2018 at 9:53 AM Dmitry Pavlov <[hidden email]> wrote:

> Hi Amir,
>
> let me say sincere thank you for continuing to contribute.
>
> Bumping up this thread.
>
> Igniters, who has an expertise here?
>
>
> вс, 17 июн. 2018 г. в 17:59, Amir Akhmedov <[hidden email]>:
>
> > Hi All,
> > Can you please review my changes for IGNITE-8740.
> >
> > PR: https://github.com/apache/ignite/pull/4208
> > TC:
> >
> >
> https://ci.ignite.apache.org/viewLog.html?buildId=1397283&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunAll
> >
> > Thanks,
> > Amir
> >
>

[Amir Akhmedov]

Hi Val,
Thanks for your comments. I replied in the ticket with my vision of the
issue and how I tried to solve it. Please check it and let me know.

Thanks,
Amir


On Fri, Jun 22, 2018 at 5:42 PM Valentin Kulichenko <
[hidden email]> wrote:

> Hi Amir,
>
> I reviewed the changes and I'm not sure I understood how they fix they
> issue. I left more detailed comment in the ticket, can you please clarify?
>
> -Val
>
> On Fri, Jun 22, 2018 at 9:53 AM Dmitry Pavlov <[hidden email]>
> wrote:
>
>> Hi Amir,
>>
>> let me say sincere thank you for continuing to contribute.
>>
>> Bumping up this thread.
>>
>> Igniters, who has an expertise here?
>>
>>
>> вс, 17 июн. 2018 г. в 17:59, Amir Akhmedov <[hidden email]>:
>>
>> > Hi All,
>> > Can you please review my changes for IGNITE-8740.
>> >
>> > PR: https://github.com/apache/ignite/pull/4208
>> > TC:
>> >
>> >
>> https://ci.ignite.apache.org/viewLog.html?buildId=1397283&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunAll
>> >
>> > Thanks,
>> > Amir
>> >
>>
>

[Valentin Kulichenko]

Amir,

Thanks for quick reaction. I added a follow up question in the ticket.

-Val

On Fri, Jun 22, 2018 at 3:48 PM Amir Akhmedov <[hidden email]>
wrote:

> Hi Val,
> Thanks for your comments. I replied in the ticket with my vision of the
> issue and how I tried to solve it. Please check it and let me know.
>
> Thanks,
> Amir
>
>
> On Fri, Jun 22, 2018 at 5:42 PM Valentin Kulichenko <
> [hidden email]> wrote:
>
>> Hi Amir,
>>
>> I reviewed the changes and I'm not sure I understood how they fix they
>> issue. I left more detailed comment in the ticket, can you please clarify?
>>
>> -Val
>>
>> On Fri, Jun 22, 2018 at 9:53 AM Dmitry Pavlov <[hidden email]>
>> wrote:
>>
>>> Hi Amir,
>>>
>>> let me say sincere thank you for continuing to contribute.
>>>
>>> Bumping up this thread.
>>>
>>> Igniters, who has an expertise here?
>>>
>>>
>>> вс, 17 июн. 2018 г. в 17:59, Amir Akhmedov <[hidden email]>:
>>>
>>> > Hi All,
>>> > Can you please review my changes for IGNITE-8740.
>>> >
>>> > PR: https://github.com/apache/ignite/pull/4208
>>> > TC:
>>> >
>>> >
>>> https://ci.ignite.apache.org/viewLog.html?buildId=1397283&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunAll
>>> >
>>> > Thanks,
>>> > Amir
>>> >
>>>
>>

[Amir Akhmedov]

Val,
I replied to it already :)

Thanks,
Amir


On Fri, Jun 22, 2018 at 7:20 PM Valentin Kulichenko <
[hidden email]> wrote:

> Amir,
>
> Thanks for quick reaction. I added a follow up question in the ticket.
>
> -Val
>
> On Fri, Jun 22, 2018 at 3:48 PM Amir Akhmedov <[hidden email]>
> wrote:
>
>> Hi Val,
>> Thanks for your comments. I replied in the ticket with my vision of the
>> issue and how I tried to solve it. Please check it and let me know.
>>
>> Thanks,
>> Amir
>>
>>
>> On Fri, Jun 22, 2018 at 5:42 PM Valentin Kulichenko <
>> [hidden email]> wrote:
>>
>>> Hi Amir,
>>>
>>> I reviewed the changes and I'm not sure I understood how they fix they
>>> issue. I left more detailed comment in the ticket, can you please clarify?
>>>
>>> -Val
>>>
>>> On Fri, Jun 22, 2018 at 9:53 AM Dmitry Pavlov <[hidden email]>
>>> wrote:
>>>
>>>> Hi Amir,
>>>>
>>>> let me say sincere thank you for continuing to contribute.
>>>>
>>>> Bumping up this thread.
>>>>
>>>> Igniters, who has an expertise here?
>>>>
>>>>
>>>> вс, 17 июн. 2018 г. в 17:59, Amir Akhmedov <[hidden email]>:
>>>>
>>>> > Hi All,
>>>> > Can you please review my changes for IGNITE-8740.
>>>> >
>>>> > PR: https://github.com/apache/ignite/pull/4208
>>>> > TC:
>>>> >
>>>> >
>>>> https://ci.ignite.apache.org/viewLog.html?buildId=1397283&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunAll
>>>> >
>>>> > Thanks,
>>>> > Amir
>>>> >
>>>>
>>>

[Valentin Kulichenko]

Amir,

I merged you change to master and 2.6. Thanks!

-Val

On Fri, Jun 22, 2018 at 4:21 PM Amir Akhmedov <[hidden email]>
wrote:

> Val,
> I replied to it already :)
>
> Thanks,
> Amir
>
>
> On Fri, Jun 22, 2018 at 7:20 PM Valentin Kulichenko <
> [hidden email]> wrote:
>
>> Amir,
>>
>> Thanks for quick reaction. I added a follow up question in the ticket.
>>
>> -Val
>>
>> On Fri, Jun 22, 2018 at 3:48 PM Amir Akhmedov <[hidden email]>
>> wrote:
>>
>>> Hi Val,
>>> Thanks for your comments. I replied in the ticket with my vision of the
>>> issue and how I tried to solve it. Please check it and let me know.
>>>
>>> Thanks,
>>> Amir
>>>
>>>
>>> On Fri, Jun 22, 2018 at 5:42 PM Valentin Kulichenko <
>>> [hidden email]> wrote:
>>>
>>>> Hi Amir,
>>>>
>>>> I reviewed the changes and I'm not sure I understood how they fix they
>>>> issue. I left more detailed comment in the ticket, can you please clarify?
>>>>
>>>> -Val
>>>>
>>>> On Fri, Jun 22, 2018 at 9:53 AM Dmitry Pavlov <[hidden email]>
>>>> wrote:
>>>>
>>>>> Hi Amir,
>>>>>
>>>>> let me say sincere thank you for continuing to contribute.
>>>>>
>>>>> Bumping up this thread.
>>>>>
>>>>> Igniters, who has an expertise here?
>>>>>
>>>>>
>>>>> вс, 17 июн. 2018 г. в 17:59, Amir Akhmedov <[hidden email]>:
>>>>>
>>>>> > Hi All,
>>>>> > Can you please review my changes for IGNITE-8740.
>>>>> >
>>>>> > PR: https://github.com/apache/ignite/pull/4208
>>>>> > TC:
>>>>> >
>>>>> >
>>>>> https://ci.ignite.apache.org/viewLog.html?buildId=1397283&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunAll
>>>>> >
>>>>> > Thanks,
>>>>> > Amir
>>>>> >
>>>>>
>>>>

[Amir Akhmedov]

Great, thanks! As always, happy to contribute!

Thanks,
Amir


On Fri, Jun 22, 2018 at 7:32 PM Valentin Kulichenko <
[hidden email]> wrote:

> Amir,
>
> I merged you change to master and 2.6. Thanks!
>
> -Val
>
> On Fri, Jun 22, 2018 at 4:21 PM Amir Akhmedov <[hidden email]>
> wrote:
>
>> Val,
>> I replied to it already :)
>>
>> Thanks,
>> Amir
>>
>>
>> On Fri, Jun 22, 2018 at 7:20 PM Valentin Kulichenko <
>> [hidden email]> wrote:
>>
>>> Amir,
>>>
>>> Thanks for quick reaction. I added a follow up question in the ticket.
>>>
>>> -Val
>>>
>>> On Fri, Jun 22, 2018 at 3:48 PM Amir Akhmedov <[hidden email]>
>>> wrote:
>>>
>>>> Hi Val,
>>>> Thanks for your comments. I replied in the ticket with my vision of the
>>>> issue and how I tried to solve it. Please check it and let me know.
>>>>
>>>> Thanks,
>>>> Amir
>>>>
>>>>
>>>> On Fri, Jun 22, 2018 at 5:42 PM Valentin Kulichenko <
>>>> [hidden email]> wrote:
>>>>
>>>>> Hi Amir,
>>>>>
>>>>> I reviewed the changes and I'm not sure I understood how they fix they
>>>>> issue. I left more detailed comment in the ticket, can you please clarify?
>>>>>
>>>>> -Val
>>>>>
>>>>> On Fri, Jun 22, 2018 at 9:53 AM Dmitry Pavlov <[hidden email]>
>>>>> wrote:
>>>>>
>>>>>> Hi Amir,
>>>>>>
>>>>>> let me say sincere thank you for continuing to contribute.
>>>>>>
>>>>>> Bumping up this thread.
>>>>>>
>>>>>> Igniters, who has an expertise here?
>>>>>>
>>>>>>
>>>>>> вс, 17 июн. 2018 г. в 17:59, Amir Akhmedov <[hidden email]>:
>>>>>>
>>>>>> > Hi All,
>>>>>> > Can you please review my changes for IGNITE-8740.
>>>>>> >
>>>>>> > PR: https://github.com/apache/ignite/pull/4208
>>>>>> > TC:
>>>>>> >
>>>>>> >
>>>>>> https://ci.ignite.apache.org/viewLog.html?buildId=1397283&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunAll
>>>>>> >
>>>>>> > Thanks,
>>>>>> > Amir
>>>>>> >
>>>>>>
>>>>>

[Dmitriy Govorukhin]

Valentin,

Seems that these changes have classes without license head. TC link
<https://ci.ignite.apache.org/viewLog.html?buildId=1418488&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_LicensesHeaders>

/data/teamcity/work/c182b70f2dfa6507/modules/spring/src/test/java/org/apache/ignite/transactions/spring/GridSpringTransactionManagerSpringBeanSelfTest.java
/data/teamcity/work/c182b70f2dfa6507/modules/spring/src/test/java/org/apache/ignite/transactions/spring/GridSpringTransactionManagerAbstractTest.java

Please, add headers for these classes.

On Sat, Jun 23, 2018 at 2:33 AM Amir Akhmedov <[hidden email]>
wrote:

> Great, thanks! As always, happy to contribute!
>
> Thanks,
> Amir
>
>
> On Fri, Jun 22, 2018 at 7:32 PM Valentin Kulichenko <
> [hidden email]> wrote:
>
> > Amir,
> >
> > I merged you change to master and 2.6. Thanks!
> >
> > -Val
> >
> > On Fri, Jun 22, 2018 at 4:21 PM Amir Akhmedov <[hidden email]>
> > wrote:
> >
> >> Val,
> >> I replied to it already :)
> >>
> >> Thanks,
> >> Amir
> >>
> >>
> >> On Fri, Jun 22, 2018 at 7:20 PM Valentin Kulichenko <
> >> [hidden email]> wrote:
> >>
> >>> Amir,
> >>>
> >>> Thanks for quick reaction. I added a follow up question in the ticket.
> >>>
> >>> -Val
> >>>
> >>> On Fri, Jun 22, 2018 at 3:48 PM Amir Akhmedov <[hidden email]
> >
> >>> wrote:
> >>>
> >>>> Hi Val,
> >>>> Thanks for your comments. I replied in the ticket with my vision of
> the
> >>>> issue and how I tried to solve it. Please check it and let me know.
> >>>>
> >>>> Thanks,
> >>>> Amir
> >>>>
> >>>>
> >>>> On Fri, Jun 22, 2018 at 5:42 PM Valentin Kulichenko <
> >>>> [hidden email]> wrote:
> >>>>
> >>>>> Hi Amir,
> >>>>>
> >>>>> I reviewed the changes and I'm not sure I understood how they fix
> they
> >>>>> issue. I left more detailed comment in the ticket, can you please
> clarify?
> >>>>>
> >>>>> -Val
> >>>>>
> >>>>> On Fri, Jun 22, 2018 at 9:53 AM Dmitry Pavlov <[hidden email]
> >
> >>>>> wrote:
> >>>>>
> >>>>>> Hi Amir,
> >>>>>>
> >>>>>> let me say sincere thank you for continuing to contribute.
> >>>>>>
> >>>>>> Bumping up this thread.
> >>>>>>
> >>>>>> Igniters, who has an expertise here?
> >>>>>>
> >>>>>>
> >>>>>> вс, 17 июн. 2018 г. в 17:59, Amir Akhmedov <[hidden email]
> >:
> >>>>>>
> >>>>>> > Hi All,
> >>>>>> > Can you please review my changes for IGNITE-8740.
> >>>>>> >
> >>>>>> > PR: https://github.com/apache/ignite/pull/4208
> >>>>>> > TC:
> >>>>>> >
> >>>>>> >
> >>>>>>
> https://ci.ignite.apache.org/viewLog.html?buildId=1397283&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunAll
> >>>>>> >
> >>>>>> > Thanks,
> >>>>>> > Amir
> >>>>>> >
> >>>>>>
> >>>>>
>

[Valentin Kulichenko]

Dmitry,

Thanks for pointing this out. Fixed in master and 2.6.

-Val

On Sat, Jun 23, 2018 at 10:49 AM Dmitriy Govorukhin <
[hidden email]> wrote:

> Valentin,
>
> Seems that these changes have classes without license head. TC link
> <https://ci.ignite.apache.org/viewLog.html?buildId=1418488&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_LicensesHeaders>
>
> /data/teamcity/work/c182b70f2dfa6507/modules/spring/src/test/java/org/apache/ignite/transactions/spring/GridSpringTransactionManagerSpringBeanSelfTest.java
> /data/teamcity/work/c182b70f2dfa6507/modules/spring/src/test/java/org/apache/ignite/transactions/spring/GridSpringTransactionManagerAbstractTest.java
>
> Please, add headers for these classes.
>
> On Sat, Jun 23, 2018 at 2:33 AM Amir Akhmedov <[hidden email]>
> wrote:
>
>> Great, thanks! As always, happy to contribute!
>>
>> Thanks,
>> Amir
>>
>>
>> On Fri, Jun 22, 2018 at 7:32 PM Valentin Kulichenko <
>> [hidden email]> wrote:
>>
>> > Amir,
>> >
>> > I merged you change to master and 2.6. Thanks!
>> >
>> > -Val
>> >
>> > On Fri, Jun 22, 2018 at 4:21 PM Amir Akhmedov <[hidden email]>
>> > wrote:
>> >
>> >> Val,
>> >> I replied to it already :)
>> >>
>> >> Thanks,
>> >> Amir
>> >>
>> >>
>> >> On Fri, Jun 22, 2018 at 7:20 PM Valentin Kulichenko <
>> >> [hidden email]> wrote:
>> >>
>> >>> Amir,
>> >>>
>> >>> Thanks for quick reaction. I added a follow up question in the ticket.
>> >>>
>> >>> -Val
>> >>>
>> >>> On Fri, Jun 22, 2018 at 3:48 PM Amir Akhmedov <
>> [hidden email]>
>> >>> wrote:
>> >>>
>> >>>> Hi Val,
>> >>>> Thanks for your comments. I replied in the ticket with my vision of
>> the
>> >>>> issue and how I tried to solve it. Please check it and let me know.
>> >>>>
>> >>>> Thanks,
>> >>>> Amir
>> >>>>
>> >>>>
>> >>>> On Fri, Jun 22, 2018 at 5:42 PM Valentin Kulichenko <
>> >>>> [hidden email]> wrote:
>> >>>>
>> >>>>> Hi Amir,
>> >>>>>
>> >>>>> I reviewed the changes and I'm not sure I understood how they fix
>> they
>> >>>>> issue. I left more detailed comment in the ticket, can you please
>> clarify?
>> >>>>>
>> >>>>> -Val
>> >>>>>
>> >>>>> On Fri, Jun 22, 2018 at 9:53 AM Dmitry Pavlov <
>> [hidden email]>
>> >>>>> wrote:
>> >>>>>
>> >>>>>> Hi Amir,
>> >>>>>>
>> >>>>>> let me say sincere thank you for continuing to contribute.
>> >>>>>>
>> >>>>>> Bumping up this thread.
>> >>>>>>
>> >>>>>> Igniters, who has an expertise here?
>> >>>>>>
>> >>>>>>
>> >>>>>> вс, 17 июн. 2018 г. в 17:59, Amir Akhmedov <
>> [hidden email]>:
>> >>>>>>
>> >>>>>> > Hi All,
>> >>>>>> > Can you please review my changes for IGNITE-8740.
>> >>>>>> >
>> >>>>>> > PR: https://github.com/apache/ignite/pull/4208
>> >>>>>> > TC:
>> >>>>>> >
>> >>>>>> >
>> >>>>>>
>> https://ci.ignite.apache.org/viewLog.html?buildId=1397283&tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_RunAll
>> >>>>>> >
>> >>>>>> > Thanks,
>> >>>>>> > Amir
>> >>>>>> >
>> >>>>>>
>> >>>>>
>>
>