Exclusion Advent-ure: constant arguments (#5808)

palantir · Dec 6, 2021 · deb412a · deb412a
1 parent b50c0ab
commit deb412a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/...st-utils/src/main/java/com/palantir/atlasdb/containers/InterruptibleFileLogCollector.java b/...st-utils/src/main/java/com/palantir/atlasdb/containers/InterruptibleFileLogCollector.java
@@ -31,7 +31,6 @@
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 import java.util.concurrent.TimeUnit;
-import org.apache.commons.lang3.Validate;
 
 @SuppressFBWarnings("SLF4J_ILLEGAL_PASSED_CLASS")
 public class InterruptibleFileLogCollector implements LogCollector {
@@ -46,7 +45,11 @@ public class InterruptibleFileLogCollector implements LogCollector {
     public InterruptibleFileLogCollector(File logDirectory) {
         Preconditions.checkArgument(!logDirectory.isFile(), "Log directory cannot be a file");
         if (!logDirectory.exists()) {
-            Validate.isTrue(logDirectory.mkdirs(), "Error making log directory: " + logDirectory.getAbsolutePath());
+            boolean successfullyCreated = logDirectory.mkdirs();
+            Preconditions.checkState(
+                    successfullyCreated,
+                    "Error making log directory",
+                    SafeArg.of("logDirectory", logDirectory.getAbsolutePath()));
         }
         this.logDirectory = logDirectory;
     }

diff --git a/build.gradle b/build.gradle
@@ -123,8 +123,7 @@ allprojects {
                 'StaticAssignmentInConstructor',
                 'StrictUnusedVariable',
                 'ThreadPriorityCheck',
-                'TooManyArguments',
-                'ValidateConstantMessage'
+                'TooManyArguments'
     }
 }