Remove fail-fast check in NamedForkJoinWorkerThreadFactory constructor. Costs CPU for what should be catchable by tests.

--
MOS_MIGRATED_REVID=139097279

1 files changed, 1 insertions, 3 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/concurrent/NamedForkJoinPool.java b/src/main/java/com/google/devtools/build/lib/concurrent/NamedForkJoinPool.java
index 33a5e86f5b..f63b387d78 100644
--- a/src/main/java/com/google/devtools/build/lib/concurrent/NamedForkJoinPool.java
+++ b/src/main/java/com/google/devtools/build/lib/concurrent/NamedForkJoinPool.java
@@ -50,9 +50,7 @@ public class NamedForkJoinPool extends ForkJoinPool {
     private final String nameFormat;
     private final AtomicLong nextUnusedThreadIndex = new AtomicLong(0L);
 
-    public NamedForkJoinWorkerThreadFactory(String nameFormat) {
-      // Fail fast if nameFormat is bad.
-      String ignored = String.format(nameFormat, 0L);
+    NamedForkJoinWorkerThreadFactory(String nameFormat) {
       this.nameFormat = nameFormat;
     }