[ 
https://issues.apache.org/jira/browse/IGNITE-10555?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16722880#comment-16722880
 ] 

ASF GitHub Bot commented on IGNITE-10555:
-----------------------------------------

Github user daradurvs closed the pull request at:

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


> Set 'TcpDiscoveryVmIpFinder' as default IP finder for tests instead of 
> 'TcpDiscoveryMulticastIpFinder'
> ------------------------------------------------------------------------------------------------------
>
>                 Key: IGNITE-10555
>                 URL: https://issues.apache.org/jira/browse/IGNITE-10555
>             Project: Ignite
>          Issue Type: Improvement
>            Reporter: Vyacheslav Daradur
>            Assignee: Vyacheslav Daradur
>            Priority: Major
>             Fix For: 2.8
>
>
> Most of our tests starting Ignite nodes in the same JVM, that allows
> us using shared 'TcpDiscoveryVmIpFinder'.
> There are following main advantages of using 'TcpDiscoveryVmIpFinder':
> * reducing possible conflicts in the development environment, when
> nodes from different clusters may find each other;
> * speedup of nodes initial discovery, especially on Windows;
> * avoiding of overwriting 'getConfiguration' and copypasta only to set
> up static IP finder in tests;
> We should change the default IP finder in tests to
> 'TcpDiscoveryVmIpFinder' as the first step and remove related
> boilerplate as the second step.
> Once the first step of the change will be finished then we should fill a 
> separate task for removing related boilerplate.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to