arpitjain099 opened a new pull request, #65739:
URL: https://github.com/apache/doris/pull/65739
### What problem does this PR solve?
Small tidy-ups I noticed while reading through the authorization plugin path
(`AccessControllerManager` and the Ranger controller) for the OpenFGA work.
Two things:
- In `AccessControllerManager.loadAccessControllerPlugins()` the two
log/exception strings for the classpath loader say "Authentication Plugin
Factories", but this method loads `AccessControllerFactory` instances
(authorization). The directory loader a few lines down already logs "Access
Controller Plugin Factory", so this just makes the classpath branch consistent
and stops the messages from pointing a reader at the wrong subsystem. The class
javadoc also called the manager the entry point of "privilege authentication";
corrected to "authorization".
- `RangerDorisAccessController` still carries a leftover `public static void
main` marked "For test only" that constructs a controller and prints to stdout.
It is dead code (nothing references it, and it is not a real test) so I removed
it.
No behavior change, log wording and a dead method only.
### Release note
None
### Check List (For Author)
- Test
- [x] No need to test or manual test. Explain why:
- [x] This is a refactor/code format and no logic has been changed.
- Behavior changed:
- [x] No.
- Does this need documentation?
- [x] No.
I'll keep looking at where I can help Doris support different authentication
and authorization systems as I go.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
