Skip to content

Commit eae9511

Browse files
committedFeb 13, 2024·
Remove unused codepath
Signed-off-by: Peter Nied <petern@amazon.com>
1 parent cba189e commit eae9511

File tree

3 files changed

+3
-15
lines changed

3 files changed

+3
-15
lines changed
 

‎server/src/internalClusterTest/java/org/opensearch/action/admin/indices/view/ViewTestBase.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ protected List<String> listViewNames() {
5757
}
5858

5959
protected SearchResponse searchView(final String viewName) throws Exception {
60-
final SearchViewAction.Request request = SearchViewAction.createRequestWith(viewName, new SearchRequest());
60+
final SearchViewAction.Request request = new SearchViewAction.Request(viewName, new SearchRequest());
6161
final SearchResponse response = client().searchView(request).actionGet();
6262
return response;
6363
}

‎server/src/main/java/org/opensearch/action/admin/indices/view/SearchViewAction.java

-12
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,8 @@
1616
import org.opensearch.action.support.HandledTransportAction;
1717
import org.opensearch.common.annotation.ExperimentalApi;
1818
import org.opensearch.common.inject.Inject;
19-
import org.opensearch.common.io.stream.BytesStreamOutput;
2019
import org.opensearch.core.action.ActionListener;
2120
import org.opensearch.core.common.Strings;
22-
import org.opensearch.core.common.io.stream.BytesStreamInput;
2321
import org.opensearch.core.common.io.stream.StreamInput;
2422
import org.opensearch.core.common.io.stream.StreamOutput;
2523
import org.opensearch.tasks.Task;
@@ -42,16 +40,6 @@ private SearchViewAction() {
4240
super(NAME, SearchResponse::new);
4341
}
4442

45-
/** Given a search request, creates a ViewSearchRequest */
46-
public static Request createRequestWith(final String view, final SearchRequest searchRequest) throws IOException {
47-
final BytesStreamOutput savedSearchRequest = new BytesStreamOutput();
48-
searchRequest.writeTo(savedSearchRequest);
49-
savedSearchRequest.writeString(view);
50-
51-
final BytesStreamInput input = new BytesStreamInput(savedSearchRequest.bytes().toBytesRef().bytes);
52-
return new Request(input);
53-
}
54-
5543
/**
5644
* Wraps the functionality of search requests and tailors for what is available
5745
* when searching through views

‎server/src/test/java/org/opensearch/action/admin/indices/view/SearchViewRequestTests.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -30,14 +30,14 @@ protected Writeable.Reader<SearchViewAction.Request> instanceReader() {
3030
@Override
3131
protected SearchViewAction.Request createTestInstance() {
3232
try {
33-
return SearchViewAction.createRequestWith(randomAlphaOfLength(8), new SearchRequest());
33+
return new SearchViewAction.Request(randomAlphaOfLength(8), new SearchRequest());
3434
} catch (final Exception e) {
3535
throw new RuntimeException(e);
3636
}
3737
}
3838

3939
public void testValidateRequest() throws IOException {
40-
final SearchViewAction.Request request = SearchViewAction.createRequestWith("my-view", new SearchRequest());
40+
final SearchViewAction.Request request = new SearchViewAction.Request("my-view", new SearchRequest());
4141
MatcherAssert.assertThat(request.validate(), nullValue());
4242
}
4343

0 commit comments

Comments
 (0)
Please sign in to comment.