[GitHub] [hadoop] steveloughran commented on a diff in pull request #4255: HADOOP-18217. ExitUtil synchronized blocks reduced to avoid exit bloc…

2022-07-12 Thread GitBox
steveloughran commented on code in PR #4255: URL: https://github.com/apache/hadoop/pull/4255#discussion_r918827783 ## hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/ExitUtil.java: ## @@ -204,6 +204,20 @@ public static void resetFirstHaltException() {

[GitHub] [hadoop] steveloughran commented on a diff in pull request #4255: HADOOP-18217. ExitUtil synchronized blocks reduced to avoid exit bloc…

2022-07-11 Thread GitBox
steveloughran commented on code in PR #4255: URL: https://github.com/apache/hadoop/pull/4255#discussion_r918162069 ## hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/util/TestExitUtil.java: ## @@ -0,0 +1,122 @@ +/** + * Licensed to the Apache Software

[GitHub] [hadoop] steveloughran commented on a diff in pull request #4255: HADOOP-18217. ExitUtil synchronized blocks reduced to avoid exit bloc…

2022-07-11 Thread GitBox
steveloughran commented on code in PR #4255: URL: https://github.com/apache/hadoop/pull/4255#discussion_r918150727 ## hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/ExitUtil.java: ## @@ -159,138 +163,155 @@ public static void disableSystemHalt() {

[GitHub] [hadoop] steveloughran commented on a diff in pull request #4255: HADOOP-18217. ExitUtil synchronized blocks reduced to avoid exit bloc…

2022-05-30 Thread GitBox
steveloughran commented on code in PR #4255: URL: https://github.com/apache/hadoop/pull/4255#discussion_r884819180 ## hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/ExitUtil.java: ## @@ -159,138 +163,155 @@ public static void disableSystemHalt() {

[GitHub] [hadoop] steveloughran commented on a diff in pull request #4255: HADOOP-18217. ExitUtil synchronized blocks reduced to avoid exit bloc…

2022-05-02 Thread GitBox
steveloughran commented on code in PR #4255: URL: https://github.com/apache/hadoop/pull/4255#discussion_r862765210 ## hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/ExitUtil.java: ## @@ -226,25 +251,48 @@ public static synchronized void