tkalkirill commented on code in PR #2717:
URL: https://github.com/apache/ignite-3/pull/2717#discussion_r1365096556
##########
modules/raft/src/test/java/org/apache/ignite/raft/jraft/rpc/IgniteRpcTest.java:
##########
@@ -113,37 +117,22 @@ public IgniteRpcTest(TestInfo testInfo) {
}
/** {@inheritDoc} */
Review Comment:
```suggestion
```
##########
modules/raft/src/test/java/org/apache/ignite/raft/jraft/rpc/AbstractRpcTest.java:
##########
@@ -249,5 +249,5 @@ private class Request2RpcProcessor implements
RpcProcessor<TestMessages.Request2
* @param expected Expected count.
* @param timeout The timeout in millis.
*/
- protected abstract boolean waitForTopology(RpcClient client, int expected,
long timeout);
+ protected abstract boolean waitForTopology(RpcClient client, int expected,
int timeout);
Review Comment:
Why? timeout value always long value, please fix also
`org.apache.ignite.raft.jraft.test.TestUtils#waitForTopology`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
