Mmuzaf commented on a change in pull request #9240:
URL: https://github.com/apache/ignite/pull/9240#discussion_r680548702



##########
File path: 
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/SnapshotRestoreProcess.java
##########
@@ -86,6 +87,12 @@
     /** Reject operation message. */
     private static final String OP_REJECT_MSG = "Cache group restore operation 
was rejected. ";
 
+    /** Snapshot restore operation finish message. */
+    private static final String OP_FINISHED_MSG = "Successfully restored cache 
group(s) from the snapshot [reqId=%s]";

Review comment:
       Let's use the right word order Subject + Verb + Object: `Cache groups 
have been successfully restored from snapshot`.
   Do we need a request Id? I'd rather use the cache group names or remove it 
at all.




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