ivanzlenko commented on code in PR #4779:
URL: https://github.com/apache/ignite-3/pull/4779#discussion_r1855958477
##########
modules/raft/src/main/java/org/apache/ignite/internal/raft/RetryContext.java:
##########
@@ -69,11 +72,19 @@ class RetryContext {
* Creates a context.
*
* @param targetPeer Target peer to send the request to.
+ * @param originDescription Supplier describing the origin request from
which this one depends, or returning {@code null}
+ * if this request is independent.
* @param requestFactory Factory for creating requests to the target peer.
* @param stopTime Timestamp that denotes the point in time up to which
retry attempts will be made.
*/
- RetryContext(Peer targetPeer, Function<Peer, ? extends NetworkMessage>
requestFactory, long stopTime) {
+ RetryContext(
+ Peer targetPeer,
+ Supplier<String> originDescription,
Review Comment:
Are we sure that we need a supplier here?
--
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: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
