IGNITE-4157 improvement patch available

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

IGNITE-4157 improvement patch available

Sergey Chugunov
Folks,

I finished development on IGNITE-4157
<https://issues.apache.org/jira/browse/IGNITE-4157> *Use discovery custom
messages instead of marshaller cache.*

Latest CI build
<http://ci.ignite.apache.org/viewLog.html?buildId=392762&tab=buildResultsDiv&buildTypeId=IgniteTests_RunAll>
in team city looks very good, there are no tests failures related to my
refactoring (only flaky tests are broken, this set varies from build to
build) nor build hangs.

I believe the code is stable enough and ready to be reviewed and merged to
ignite-2.0 branch. Please take a look at pull request pull/1271
<https://github.com/apache/ignite/pull/1271>.

Thanks,
Sergey.
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-4157 improvement patch available

Alexey Goncharuk
Sergey, thanks for the careful work! I've merged your contribution to
ignite-2.0 branch.

On Dec 16, 2016 11:50, "Sergey Chugunov" <[hidden email]> wrote:

> Folks,
>
> I finished development on IGNITE-4157
> <https://issues.apache.org/jira/browse/IGNITE-4157> *Use discovery custom
> messages instead of marshaller cache.*
>
> Latest CI build
> <http://ci.ignite.apache.org/viewLog.html?buildId=392762&
> tab=buildResultsDiv&buildTypeId=IgniteTests_RunAll>
> in team city looks very good, there are no tests failures related to my
> refactoring (only flaky tests are broken, this set varies from build to
> build) nor build hangs.
>
> I believe the code is stable enough and ready to be reviewed and merged to
> ignite-2.0 branch. Please take a look at pull request pull/1271
> <https://github.com/apache/ignite/pull/1271>.
>
> Thanks,
> Sergey.
>