sanpwc commented on a change in pull request #147:
URL: https://github.com/apache/ignite-3/pull/147#discussion_r639464080



##########
File path: 
modules/raft-client/src/main/java/org/apache/ignite/raft/client/message/ActionRequest.java
##########
@@ -35,19 +35,30 @@
      */
     Command command();
 
+    /**
+     * @return {@code True} for linearizable reading.
+     */
+    boolean readOnlySafe();

Review comment:
       Currently there are no usages of readOnlySafe(). Why we need it? In 
Which cases do you expect to use non-linearizable readings? Besides that it's a 
bit confusing cause readOnlySafe only has sense for read operations, but 
ActrionRequest both covers reads and writes. In other words within given 
abstaction it's possible to create write request with readOnlySafe set to 
false. Do we expect non-linearizable writes?




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to