Separate GlobalParams into a separate message (opensearch-project#207)

* separate GlobalParams into a separate message

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

* Update CHANGELOG.md

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

* Update CHANGELOG.md

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

* renumber

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

---------

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

Author: karenyrx

CHANGELOG.md

@@ -8,6 +8,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 
 ### Changed
 - Modify `searchResponse` and `bulkResponse` according to the updated spec and remove the error response. ([#194](https://github.com/opensearch-project/opensearch-protobufs/pull/194))
+- Use separate `GlobalParams` message ([#207](https://github.com/opensearch-project/opensearch-protobufs/pull/207))
 
 ### Removed
 

protos/schemas/common.proto

@@ -10,6 +10,16 @@ option go_package = "github.com/opensearch-project/opensearch-protobufs/go/opens
 
 import "google/protobuf/struct.proto";
 
+// TODO: not supported yet in server
+message GlobalParams {
+  // [optional] Whether to return human-readable values for statistics.
+  optional bool human = 1;
+  // [optional] Whether to include the stack trace of returned errors.
+  optional bool error_trace = 2;
+  // [optional] A comma-separated list of filters used to filter the response. Use wildcards to match any field or part of a field's name. To exclude fields, use `-`.
+  repeated string filter_path = 3;
+}
+
 message WaitForActiveShards {
 
   oneof wait_for_active_shards {

protos/schemas/document.proto

@@ -40,15 +40,8 @@ message BulkRequest {
   optional WaitForActiveShards wait_for_active_shards = 11;
   // [required] The request body contains create, delete, index, and update actions and their associated source data
   repeated BulkRequestBody request_body = 12;
-  // [optional] Whether to return human-readable values for statistics.
-  optional bool human = 13;
-  // [optional] Whether to include the stack trace of returned errors.
-  // TODO not supported yet
-  optional bool error_trace = 14;
-  // [optional] The URL-encoded request definition. Useful for libraries that do not accept a request body for non-POST requests.
-  optional string source = 15;
-  // [optional] A comma-separated list of filters used to filter the response. Use wildcards to match any field or part of a field's name. To exclude fields, use `-`.
-  repeated string filter_path = 16;
+  // [optional] Global parameters
+  optional GlobalParams global_params = 13;
 }
 
 message BulkRequestBody {

protos/schemas/search.proto

@@ -135,15 +135,8 @@ message SearchRequest {
   optional bool version = 48;
   // [optional] Search Request body
   SearchRequestBody request_body = 49;
-  // [optional] Whether to return human-readable values for statistics.
-  optional bool human = 50;
-  // [optional] Whether to include the stack trace of returned errors.
-  // todo: not supported yet.
-  optional bool error_trace = 52;
-  // [optional] The URL-encoded request definition. Useful for libraries that do not accept a request body for non-POST requests.
-  optional string source = 53;
-  // [optional] A comma-separated list of filters used to filter the response. Use wildcards to match any field or part of a field's name. To exclude fields, use `-`.
-  repeated string filter_path = 54;
+  // [optional] Global parameters
+  optional GlobalParams global_params = 50;
 }
 
 message FloatMap {