You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by Nikita Amelchev <ns...@gmail.com> on 2017/04/27 08:56:49 UTC
IGNITE-4763 - ready for review.
Hello. I done it.
Please, review.
https://issues.apache.org/jira/browse/IGNITE-4763 - doSetRollbackOnly
method to be implemented in SpringTransactionManager
PR: https://github.com/apache/ignite/pull/1879/files
latest ci.tests:
http://ci.ignite.apache.org/project.html?projectId=IgniteTests&tab=projectOverview&branch_IgniteTests=pull%2F1879%2Fhead
--
Best wishes,
Amelchev Nikita
Re: IGNITE-4763 - ready for review.
Posted by Nikita Amelchev <ns...@gmail.com>.
Hello.
I add test. Please review.
https://issues.apache.org/jira/browse/IGNITE-4763
PR: https://github.com/apache/ignite/pull/1884/files
ci test:
http://ci.ignite.apache.org/project.html?projectId=IgniteTests&branch_IgniteTests=pull/1884/head
2017-04-27 19:15 GMT+03:00 Denis Magda <dm...@apache.org>:
> Nikita,
>
> I don’t see a single test that validates your fix works and no one will
> break it in the future. Please address this.
>
> —
> Denis
>
> > On Apr 27, 2017, at 1:56 AM, Nikita Amelchev <ns...@gmail.com>
> wrote:
> >
> > Hello. I done it.
> >
> > Please, review.
> >
> > https://issues.apache.org/jira/browse/IGNITE-4763 - doSetRollbackOnly
> > method to be implemented in SpringTransactionManager
> >
> > PR: https://github.com/apache/ignite/pull/1879/files
> >
> > latest ci.tests:
> > http://ci.ignite.apache.org/project.html?projectId=IgniteTests&tab=
> projectOverview&branch_IgniteTests=pull%2F1879%2Fhead
> >
> > --
> > Best wishes,
> > Amelchev Nikita
>
>
--
Best wishes,
Amelchev Nikita
Re: IGNITE-4763 - ready for review.
Posted by Denis Magda <dm...@apache.org>.
Nikita,
I don’t see a single test that validates your fix works and no one will break it in the future. Please address this.
—
Denis
> On Apr 27, 2017, at 1:56 AM, Nikita Amelchev <ns...@gmail.com> wrote:
>
> Hello. I done it.
>
> Please, review.
>
> https://issues.apache.org/jira/browse/IGNITE-4763 - doSetRollbackOnly
> method to be implemented in SpringTransactionManager
>
> PR: https://github.com/apache/ignite/pull/1879/files
>
> latest ci.tests:
> http://ci.ignite.apache.org/project.html?projectId=IgniteTests&tab=projectOverview&branch_IgniteTests=pull%2F1879%2Fhead
>
> --
> Best wishes,
> Amelchev Nikita