kevdoran commented on a change in pull request #232: NIFIREG-292 Add DB impls of UserGroupProvider and AccessPolicyProvider URL: https://github.com/apache/nifi-registry/pull/232#discussion_r335623504
########## File path: nifi-registry-core/nifi-registry-framework/src/main/java/org/apache/nifi/registry/security/authorization/AbstractPolicyBasedAuthorizer.java ########## @@ -269,10 +272,13 @@ public final synchronized User updateUser(final User user) throws AuthorizationA /** * Deletes the user with the given id. * + * @deprecated use {@link #deleteUser(User)} instead + * * @param userIdentifier the identifier of the user to delete * @return the user that was deleted, or null if no matching user was found * @throws AuthorizationAccessException if there was an unexpected error performing the operation */ + @Deprecated Review comment: Just curious, why are this deprecated? ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services