stoty commented on PR #7081: URL: https://github.com/apache/hadoop/pull/7081#issuecomment-2733666333
My alternate patch tries the new API first, and falls back to the old one, @steveloughran . I have pinged you on that one, so that you can check it out, and maybe discuss it there. Conversion either way has an overhead, as the Callable / PrivilegedExceptionAction conversion and exception re-wrapping has to happen, but IMO anything we do inside those doAs() blocks is heavyweight enough that the overhad is relatively negligible. -- 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]
