stoty commented on PR #7892: URL: https://github.com/apache/hadoop/pull/7892#issuecomment-3359676121
> wow, this is a big patch. but we need it, don't we? Yes. IMO the Java language development process has really dropped the ball here. Changing the Subject APIs while providing replacements is one thing, but breaking the whole Java authorization model was a big mistake. Strictly speaking we don't need to convert EVERY Thread instance, especially in the non-secure tests, but figuring out the cases where we can safely drop the Subject / UGI from the new Thread would be super hard and error-prone (and also fragile in case authenticated operations are added later there). > > only looked at it briefly -will need to go through the full change before approval but I don't see any reason to block the change so far -- 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]
