From 920a21e55cbc8fcd4098c1d65a393c02d13d0245 Mon Sep 17 00:00:00 2001 From: Jason Tedor Date: Sun, 7 Aug 2016 12:50:15 -0400 Subject: [PATCH] Fix parsing in test set max number of threads This commit fixes a test bug in EvilJNANativesTests#testSetMaximumNumberOfThreads. Namely, the test was not checking whether or not the value from /proc/self/limits was equal to "unlimited" before attempting to parse as a long. This commit fixes that error. --- .../java/org/elasticsearch/bootstrap/EvilJNANativesTests.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qa/evil-tests/src/test/java/org/elasticsearch/bootstrap/EvilJNANativesTests.java b/qa/evil-tests/src/test/java/org/elasticsearch/bootstrap/EvilJNANativesTests.java index 1ef5bdfcfae..069f7e1aeff 100644 --- a/qa/evil-tests/src/test/java/org/elasticsearch/bootstrap/EvilJNANativesTests.java +++ b/qa/evil-tests/src/test/java/org/elasticsearch/bootstrap/EvilJNANativesTests.java @@ -40,7 +40,7 @@ public class EvilJNANativesTests extends ESTestCase { for (String line : lines) { if (line != null && line.startsWith("Max processes")) { final String[] fields = line.split("\\s+"); - final long limit = Long.parseLong(fields[2]); + final long limit = "unlimited".equals(fields[2]) ? JNACLibrary.RLIM_INFINITY : Long.parseLong(fields[2]); assertThat(JNANatives.MAX_NUMBER_OF_THREADS, equalTo(limit)); return; }