fimugdha commented on code in PR #1171: URL: https://github.com/apache/ranger/pull/1171#discussion_r3915394000
########## unixauthservice/src/main/java/org/apache/ranger/authentication/UnixAuthenticationService.java: ########## Review Comment: @pradeepagrawal8184 , Thanks for this review. Earlier the Unix Auth used to be enabled always using the argument `-enableUnixAuth`, the main thread remains alive inside `startService()` and the finally block never runs, therefore the curator leader election kept running. With this PR fix it caused a regression on leader selection when Unix Auth is disabled. Updated the fix and tested this fix on a HA setup cluster with both unix auth enabled & disabled. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
