Skip to content

Commit 7ffac3b

Browse files
Save O(1s) of CPU time in FieldSortIT (#118146)
I see this `toString` take ~2s of hot CPU time in some test runs which isn't entirely surprising. Rather than optimize this in some form, just dropping the string here which aligns the thing with other tests anyway.
1 parent b2b8e3f commit 7ffac3b

File tree

1 file changed

+0
-1
lines changed
  • server/src/internalClusterTest/java/org/elasticsearch/search/sort

1 file changed

+0
-1
lines changed

server/src/internalClusterTest/java/org/elasticsearch/search/sort/FieldSortIT.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,6 @@ public void testIssue6614() throws InterruptedException {
202202
response -> {
203203
for (int j = 0; j < response.getHits().getHits().length; j++) {
204204
assertThat(
205-
response.toString() + "\n vs. \n" + allDocsResponse.toString(),
206205
response.getHits().getHits()[j].getId(),
207206
equalTo(allDocsResponse.getHits().getHits()[j].getId())
208207
);

0 commit comments

Comments
 (0)