nizhikov commented on a change in pull request #8892:
URL: https://github.com/apache/ignite/pull/8892#discussion_r617632764



##########
File path: 
modules/core/src/main/java/org/apache/ignite/internal/processors/authentication/IgniteAuthenticationProcessor.java
##########
@@ -136,8 +161,8 @@
     /** Node activate future. */
     private final GridFutureAdapter<Void> activateFut = new 
GridFutureAdapter<>();
 
-    /** Validate error. */
-    private String validateErr;
+    /** User management operations. */
+    private final EnumSet<SecurityPermission> userOps = 
EnumSet.of(CREATE_USER, DROP_USER, ALTER_USER);

Review comment:
       This set used only one time so let's remove it and use explicit check 
like 
   
   `if (op == CREATE_USER || op == ALTER_USER || op == DROP_USER)`




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