It seems to be a good topic, but it seems to be left to the reviewer's discretion.
08.12.2020, 18:36, "Nikolay Izhikov" <nizhi...@apache.org>: > Hello, Igniters. > > Currently, we have a lot of usages `Future.get` without a timeout in tests. > In case the test that uses `Future.get` is flaky it can lead to the whole > suite hang. > > Is there any reason to use the get method without a timeout? > Can we > > a. Replace all invocation of get with the timeoutable companion. > b. Introduce code style check that will prohibit to use non-timeout method in > tests. > > What do you think?
