Cpp thin client transactions support ready for review.

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Cpp thin client transactions support ready for review.

Zhenya Stanilovsky

 
Igniters, seems i complete with transactions in thin cpp client implementation [1], part of iep-34 [2].
Can anyone review my decision ?
Failed test seems not mine, looks like after fresh master rebase it will gone.
 
[1]  https://issues.apache.org/jira/browse/IGNITE-13308
[2] https://cwiki.apache.org/confluence/display/IGNITE/IEP-34+Thin+client%3A+transactions+support
 
 
 
Reply | Threaded
Open this post in threaded view
|

Cpp thin client transactions support ready for review.

Zhenya Stanilovsky


Thanks Ivan Daschinsky for review, does anyone more who could check it ? 
 
thanks !

>Igniters, seems i complete with transactions in thin cpp client
>implementation [1], part of iep-34 [2].
>Can anyone review my decision ?
>Failed test seems not mine, looks like after fresh master rebase it will
>gone.
>
>[1]  https://issues.apache.org/jira/browse/IGNITE-13308
>[2]
>https://cwiki.apache.org/confluence/display/IGNITE/IEP-34+Thin+client%3A+transactions+support
>
>
 
 
 
 
Reply | Threaded
Open this post in threaded view
|

Re: Cpp thin client transactions support ready for review.

Igor Sapego-2
Great, I'll take a look.

Best Regards,
Igor


On Mon, Aug 24, 2020 at 8:33 AM Zhenya Stanilovsky
<[hidden email]> wrote:

>
>
> Thanks Ivan Daschinsky for review, does anyone more who could check it ?
>
> thanks !
> >Igniters, seems i complete with transactions in thin cpp client
> >implementation [1], part of iep-34 [2].
> >Can anyone review my decision ?
> >Failed test seems not mine, looks like after fresh master rebase it will
> >gone.
> >
> >[1]  https://issues.apache.org/jira/browse/IGNITE-13308
> >[2]
> >
> https://cwiki.apache.org/confluence/display/IGNITE/IEP-34+Thin+client%3A+transactions+support
> >
> >
> >
>
>
>
>