nizhikov commented on code in PR #11970:
URL: https://github.com/apache/ignite/pull/11970#discussion_r2041960016


##########
modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/DmlStatementsProcessor.java:
##########
@@ -95,62 +89,14 @@ public ModifyingEntryProcessor(Object val, 
IgniteInClosure<MutableEntry<Object,
         }
     }
 
-    /** Dummy anonymous class to advance RMV anonymous value to 5. */
-    private static final Runnable DUMMY_1 = new Runnable() {
-        @Override public void run() {
-            // No-op.
-        }
-    };
-
-    /** Dummy anonymous class to advance RMV anonymous value to 5. */
-    private static final Runnable DUMMY_2 = new Runnable() {
-        @Override public void run() {
-            // No-op.
-        }
-    };
-
-    /** Dummy anonymous class to advance RMV anonymous value to 5. */
-    private static final Runnable DUMMY_3 = new Runnable() {
-        @Override public void run() {
-            // No-op.
-        }
-    };
-
-    /** Remove updater for compatibility with < 2.7.0. Must not be moved 
around to keep at anonymous position 4. */
-    private static final IgniteInClosure<MutableEntry<Object, Object>> RMV_OLD 
=
-        new IgniteInClosure<MutableEntry<Object, Object>>() {
+    /** Remove updater. */
+    public static final IgniteInClosure<MutableEntry<Object, Object>> RMV =

Review Comment:
   Let's make this class named and static to overcome future issues with RU 
that was solved with the `DUMMY` runnables.



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

Reply via email to