Slava, These are exactly my thoughts, so I fully support you here. I already wrote about it: http://apache-ignite-developers.2346864.n4.nabble.com/IP-finder-in-tests-td33322.html But I kind of abandoned this activity. Feel free to take over it.
Denis ср, 5 дек. 2018 г. в 17:22, Vladimir Ozerov <voze...@gridgain.com>: > Huge +1 > > On Wed, Dec 5, 2018 at 5:09 PM Vyacheslav Daradur <daradu...@gmail.com> > wrote: > > > Igniters, > > > > I've found that the project's test framework uses > > 'TcpDiscoveryMulticastIpFinder' as default IP finder for tests and > > there are a lot of tests written by Ignite's experts that override it > > to 'TcpDiscoveryVmIpFinder'. > > > > Most of our tests starting Ignite nodes in the same JVM, that allows > > us using shared 'TcpDiscoveryVmIpFinder'. > > > > I think that using of 'TcpDiscoveryMulticastIpFinder' may be useful > > only in platforms tests, BTW multi-JVM tests use the tuned > > 'TcpDiscoveryVmIpFinder'. > > > > I see the 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; > > > > So, I'd suggest changing the default IP finder in tests to > > 'TcpDiscoveryVmIpFinder' as the first step and remove related > > boilerplate as the second step. > > > > What do you think? > > > > -- > > Best Regards, Vyacheslav D. > > >
