Skip to content

Commit 88f8ef5

Browse files
Save O(1s) of CPU time in FieldSortIT (#118146) (#118241)
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. Co-authored-by: Dimitris Rempapis <[email protected]>
1 parent bc2bd59 commit 88f8ef5

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)