[GitHub] ignite pull request #4931: IGNITE-9802

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

[GitHub] ignite pull request #4931: IGNITE-9802

andrey-kuznetsov
GitHub user NSAmelchev opened a pull request:

    https://github.com/apache/ignite/pull/4931

    IGNITE-9802

    Fix for the test.
   
    The problem was that PME happens and prevent to closing idle connections. The test needs close connections to recreate comm client. There was only 1s sleep. I replace it with **awaitPartitionMapExchange** and **waitForCondition**.  
   
    Also, code style and inspections fix.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/NSAmelchev/ignite ignite-9802

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/ignite/pull/4931.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #4931
   
----
commit 6899735ed5e7bcb85d06021f3bae95b38b25052d
Author: NSAmelchev <nsamelchev@...>
Date:   2018-10-09T09:49:42Z

    Fix TcpCommunicationSpiFaultyClientTest.testNotAcceptedConnection test

----


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] ignite pull request #4931: IGNITE-9802

andrey-kuznetsov
Github user asfgit closed the pull request at:

    https://github.com/apache/ignite/pull/4931


---