[GitHub] [ignite] anton-vinogradov commented on a change in pull request #7069: IGNITE-9913 PR #2

[GitBox <gi...@apache.org>]

anton-vinogradov commented on a change in pull request #7069: IGNITE-9913 PR #2
URL: https://github.com/apache/ignite/pull/7069#discussion_r354150261
 
 

 ##########
 File path: modules/core/src/main/java/org/apache/ignite/internal/processors/cache/ExchangeContext.java
 ##########
 @@ -100,6 +106,13 @@ public boolean fetchAffinityOnJoin() {
         return fetchAffOnJoin;
     }
 
+    /**
+     * @return {@code True} if it's safe to perform PME-free switch.
+     */
+    public boolean exchangeFreeSwitch() {
 
 Review comment:
   Since exchange-free switch blocks upcoming operations during the tx recovery, seems, it's correct to consider it's blocking during this period.
   In other words exchange-free does not means non-blocking, but period is short
   Blocking period can be shortened in future by recovery procedure refactoring.
   Also we, theoretically, can block only affected nodes (we have some cornercases here, so it should be additional fix) 

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services