[ https://issues.apache.org/jira/browse/HADOOP-19526?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17941583#comment-17941583 ]
ASF GitHub Bot commented on HADOOP-19526: ----------------------------------------- Hexiaoqiao commented on code in PR #7567: URL: https://github.com/apache/hadoop/pull/7567#discussion_r2031242280 ########## hadoop-tools/hadoop-distcp/src/test/java/org/apache/hadoop/tools/TestExternalCall.java: ########## @@ -66,7 +67,11 @@ private static Configuration getConf() { public void setup() { securityManager = System.getSecurityManager(); Review Comment: `System.getSecurityManager()` will also be one Deprecate APIs based on JEP-411, Would you mind to process it together. Thanks. > Skip tests in Hadoop common that depend on SecurityManager if the JVM does > not support it > ----------------------------------------------------------------------------------------- > > Key: HADOOP-19526 > URL: https://issues.apache.org/jira/browse/HADOOP-19526 > Project: Hadoop Common > Issue Type: Improvement > Reporter: Istvan Toth > Assignee: Istvan Toth > Priority: Major > Labels: pull-request-available > > TestExternalCall fails when SecurityManager cannot be set, we need to skip it > on thos JVMs. > TestGridmixSubmission has already been rewritten to use ExitUtil, we just > need to remove the leftover SecurityManager calls. -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org