Skip to content

Commit 04037cd

Browse files
Add toString() to MultiSearchRequest, MultiGetRequest and CreateIndexRequest classes. (#12314)
fix #12144 Signed-off-by: Guillaume Alvarez <7364145+guillaume-alvarez@users.noreply.github.com>
1 parent 4bcdc67 commit 04037cd

File tree

7 files changed

+80
-0
lines changed

7 files changed

+80
-0
lines changed

CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
88
- Add support for dependencies in plugin descriptor properties with semver range ([#11441](https://github.com/opensearch-project/OpenSearch/pull/11441))
99
- Add community_id ingest processor ([#12121](https://github.com/opensearch-project/OpenSearch/pull/12121))
1010
- Introduce query level setting `index.query.max_nested_depth` limiting nested queries ([#3268](https://github.com/opensearch-project/OpenSearch/issues/3268)
11+
- Add toString methods to MultiSearchRequest, MultiGetRequest and CreateIndexRequest ([#12163](https://github.com/opensearch-project/OpenSearch/pull/12163))
1112

1213
### Dependencies
1314
- Bump `com.squareup.okio:okio` from 3.7.0 to 3.8.0 ([#12290](https://github.com/opensearch-project/OpenSearch/pull/12290))

server/src/main/java/org/opensearch/action/admin/indices/create/CreateIndexRequest.java

+21
Original file line numberDiff line numberDiff line change
@@ -601,4 +601,25 @@ public void writeTo(StreamOutput out) throws IOException {
601601
}
602602
waitForActiveShards.writeTo(out);
603603
}
604+
605+
@Override
606+
public String toString() {
607+
return "CreateIndexRequest{"
608+
+ "cause='"
609+
+ cause
610+
+ '\''
611+
+ ", index='"
612+
+ index
613+
+ '\''
614+
+ ", settings="
615+
+ settings
616+
+ ", mappings='"
617+
+ mappings
618+
+ '\''
619+
+ ", aliases="
620+
+ aliases
621+
+ ", waitForActiveShards="
622+
+ waitForActiveShards
623+
+ '}';
624+
}
604625
}

server/src/main/java/org/opensearch/action/get/MultiGetRequest.java

+15
Original file line numberDiff line numberDiff line change
@@ -590,4 +590,19 @@ public XContentBuilder toXContent(XContentBuilder builder, Params params) throws
590590
return builder;
591591
}
592592

593+
@Override
594+
public String toString() {
595+
return "MultiGetRequest{"
596+
+ "preference='"
597+
+ preference
598+
+ '\''
599+
+ ", realtime="
600+
+ realtime
601+
+ ", refresh="
602+
+ refresh
603+
+ ", items="
604+
+ items
605+
+ '}';
606+
}
607+
593608
}

server/src/main/java/org/opensearch/action/search/MultiSearchRequest.java

+12
Original file line numberDiff line numberDiff line change
@@ -398,4 +398,16 @@ public boolean shouldCancelChildrenOnCancellation() {
398398
}
399399
};
400400
}
401+
402+
@Override
403+
public String toString() {
404+
return "MultiSearchRequest{"
405+
+ "maxConcurrentSearchRequests="
406+
+ maxConcurrentSearchRequests
407+
+ ", requests="
408+
+ requests
409+
+ ", indicesOptions="
410+
+ indicesOptions
411+
+ '}';
412+
}
401413
}

server/src/test/java/org/opensearch/action/admin/indices/create/CreateIndexRequestTests.java

+15
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@
5050
import java.util.Map;
5151
import java.util.Set;
5252

53+
import static org.hamcrest.CoreMatchers.containsString;
5354
import static org.hamcrest.CoreMatchers.equalTo;
5455

5556
public class CreateIndexRequestTests extends OpenSearchTestCase {
@@ -150,6 +151,20 @@ public void testSettingsType() throws IOException {
150151
assertThat(e.getMessage(), equalTo("key [settings] must be an object"));
151152
}
152153

154+
public void testToString() throws IOException {
155+
CreateIndexRequest request = new CreateIndexRequest("foo");
156+
String mapping = JsonXContent.contentBuilder()
157+
.startObject()
158+
.startObject(MapperService.SINGLE_MAPPING_NAME)
159+
.endObject()
160+
.endObject()
161+
.toString();
162+
request.mapping(mapping);
163+
164+
assertThat(request.toString(), containsString("index='foo'"));
165+
assertThat(request.toString(), containsString("mappings='{\"_doc\":{}}'"));
166+
}
167+
153168
public static void assertMappingsEqual(Map<String, String> expected, Map<String, String> actual) throws IOException {
154169
assertEquals(expected.keySet(), actual.keySet());
155170

server/src/test/java/org/opensearch/action/get/MultiGetRequestTests.java

+8
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232

3333
package org.opensearch.action.get;
3434

35+
import org.opensearch.action.get.MultiGetRequest.Item;
3536
import org.opensearch.common.xcontent.XContentFactory;
3637
import org.opensearch.common.xcontent.XContentType;
3738
import org.opensearch.core.common.ParsingException;
@@ -141,6 +142,13 @@ public void testXContentSerialization() throws IOException {
141142
}
142143
}
143144

145+
public void testToString() {
146+
MultiGetRequest req = createTestInstance();
147+
for (Item items : req.getItems()) {
148+
assertThat(req.toString(), containsString(items.toString()));
149+
}
150+
}
151+
144152
private MultiGetRequest createTestInstance() {
145153
int numItems = randomIntBetween(0, 128);
146154
MultiGetRequest request = new MultiGetRequest();

server/src/test/java/org/opensearch/action/search/MultiSearchRequestTests.java

+8
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,7 @@
7373
import static java.util.Collections.singletonList;
7474
import static org.opensearch.search.RandomSearchRequestGenerator.randomSearchRequest;
7575
import static org.opensearch.test.EqualsHashCodeTestUtils.checkEqualsAndHashCode;
76+
import static org.hamcrest.Matchers.containsString;
7677
import static org.hamcrest.Matchers.equalTo;
7778
import static org.hamcrest.Matchers.instanceOf;
7879
import static org.hamcrest.Matchers.is;
@@ -565,6 +566,13 @@ public void testEqualsAndHashcode() {
565566
checkEqualsAndHashCode(createMultiSearchRequest(), MultiSearchRequestTests::copyRequest, MultiSearchRequestTests::mutate);
566567
}
567568

569+
public void testToString() {
570+
MultiSearchRequest req = createMultiSearchRequest();
571+
for (SearchRequest subReq : req.requests()) {
572+
assertThat(req.toString(), containsString(subReq.toString()));
573+
}
574+
}
575+
568576
private static MultiSearchRequest mutate(MultiSearchRequest searchRequest) throws IOException {
569577
MultiSearchRequest mutation = copyRequest(searchRequest);
570578
List<CheckedRunnable<IOException>> mutators = new ArrayList<>();

0 commit comments

Comments
 (0)