exceptionfactory edited a comment on pull request #4712: URL: https://github.com/apache/nifi/pull/4712#issuecomment-740983198
@bbende Thanks for the review and feedback. The `resetHDFSResources()` method handles Kerberos setup options prior to calling `getFileSystemAsUser()`, which includes calling `preProcessConfiguration()` that subclasses can override to adjust the `Configuration`. Existing subclasses do not appear to do anything that would influence the file system type, so as long as that doesn't seem like a concern, changing the behavior to fail validation instead of throwing the AccessDeniedException during scheduling sounds like a better approach. ---------------------------------------------------------------- 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]
