Modify searchResponse and bulkResponse according to the updated spec and remove the error response. (opensearch-project#194)

* Update search response

Signed-off-by: xil <fridalu66@gmail.com>

# Conflicts:
#	CHANGELOG.md

* update

Signed-off-by: xil <fridalu66@gmail.com>

* add `bulkResponse`  to changelog

Signed-off-by: Karen X <karenxyr@gmail.com>

---------

Signed-off-by: xil <fridalu66@gmail.com>
Signed-off-by: Karen X <karenxyr@gmail.com>
Co-authored-by: Karen X <karenxyr@gmail.com>

Author: lucy66hw

CHANGELOG.md

@@ -9,8 +9,8 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 - proto Conversion Tooling support vendor extension `x-protobuf-excluded` ([#192](https://github.com/opensearch-project/opensearch-protobufs/pull/192))
 
 ### Changed
-- update `score` protobuf type ([#179](https://github.com/opensearch-project/opensearch-protobufs/pull/179))
 - Update `RangeQuery` and `NestedQuery` protobuf type ([#196](https://github.com/opensearch-project/opensearch-protobufs/pull/196))
+- Modify `searchResponse` and `bulkResponse` according to the updated spec and remove the error response. ([#194](https://github.com/opensearch-project/opensearch-protobufs/pull/194))
 
 ### Removed
 

protos/schemas/document.proto

@@ -189,31 +189,14 @@ message DeleteOperation {
 
 }
 
-// Bulk response contains the individual results of each operation in the request, returned in the order submitted. The success or failure of an individual operation does not affect other operations in the request.
+// todo: Proposal Spec this name
 message BulkResponse {
-  oneof response {
-    // The bulk success response
-    BulkResponseBody bulk_response_body = 1;
-    // The bulk error response
-    BulkErrorResponse bulk_error_response = 2;
-  }
-}
-
-message BulkErrorResponse {
-  // [optional] The bulk error
-  optional Error error = 1;
-  // [optional] HTTP response status code
-  optional int32 status = 2;
-}
-
-message BulkResponseBody {
-
-  // [optional] If true, one or more of the operations in the bulk request did not complete successfully.
-  optional bool errors = 1;
-  // [optional] Contains the result of each operation in the bulk request, in the order they were submitted.
+  // [required] If true, one or more of the operations in the bulk request did not complete successfully.
+  bool errors = 1;
+  // [required] Contains the result of each operation in the bulk request, in the order they were submitted.
   repeated Item items = 2;
-  // [optional] How long, in milliseconds, it took to process the bulk request.
-  optional int64 took = 3;
+  // [required] How long, in milliseconds, it took to process the bulk request.
+  int64 took = 3;
   // [optional] How long, in milliseconds, it took to process documents through an ingest pipeline
   optional int64 ingest_took = 4;
 

protos/schemas/search.proto

@@ -293,40 +293,6 @@ message TrackHits {
 
 // The response from search request.
 message SearchResponse {
-  oneof response {
-    // The search success response
-    ResponseBody response_body = 1;
-    // The search 4xx error response
-    Error4xxResponseBody error_4xx_response = 2;
-    // The search 5xx error response
-    Error5xxResponse error_5xx_response = 3;
-  }
-}
-
-
-
-// The 4xx error response from search/index-search request.
-message Error4xxResponseBody {
-  // [optional] The search 4xx error response body.
-  Error error = 1;
-  // [optional] The search 4xx error response status code.
-  optional int32 status = 2;
-}
-
-// The 5xx error response from search/index-search request.
-message Error5xxResponse {
-  // [optional] The search 5xx error response status code.
-  optional int32 status_code = 1;
-  // [optional] The search 5xx error content.
-  optional string error = 2;
-  // [optional] The search 5xx error message.
-  optional string message = 3;
-  // [optional] The search 5xx error additional_details.
-  .google.protobuf.Struct additional_details = 4;
-}
-//
-//// The response body from a search/index-search request.
-message ResponseBody {
 
   // [required] Milliseconds it took Elasticsearch to execute the request.
   int64 took = 1;
@@ -368,7 +334,7 @@ message ResponseBody {
   optional bool terminated_early = 13;
   // [optional]
   // todo: not supported yet
-  // map<string, SearchResultSuggestValueInnerArray> suggest = 14;
+  // map<string, SuggestArray> suggest = 14;
 }
 
 message ProcessorExecutionDetail {