sanpwc commented on code in PR #6053:
URL: https://github.com/apache/ignite-3/pull/6053#discussion_r2166036602


##########
modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/impl/MetaStorageManagerImpl.java:
##########
@@ -1116,7 +1116,7 @@ private void onSafeTimeAdvanced(HybridTimestamp time) {
     /**

Review Comment:
   I guess we also call resetPeers in MetaStorageManagerImpl, we either should 
implement retry logic here or create a  todo. Or which might be slightly better 
idea, move retry logic to the raft level, like we do in common sendWithRetry in 
order not to duplicate retry code.



-- 
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]

Reply via email to