[ 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)