Skip to content

Commit 71f4b36

Browse files
committed
Removed response name from QueryRequest.
It was added for future use, but it was decided to not include the expected response type as part of the query request. The name of the query must be sufficient to route it to a proper destination.
1 parent 605ad35 commit 71f4b36

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/main/proto/query.proto

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -133,8 +133,6 @@ message QueryRequest {
133133
/* The Name of the Component dispatching the query */
134134
string component_name = 9;
135135

136-
/* the name of the response the query expects */
137-
string response_name = 10;
138136
}
139137

140138
/* Message that represents the Response to a Query */
@@ -248,8 +246,10 @@ message SubscriptionQueryRequest {
248246

249247
/* Requests the initial result of a subscription query to be sent. This should always be done after opening the
250248
subscription query itself, to remove concurrency conflicts with Update messages.
249+
250+
Note that this request is deprecated. It is recommended to retrieve the initial request using a regular query.
251251
*/
252-
SubscriptionQuery get_initial_result = 3;
252+
SubscriptionQuery get_initial_result = 3 [deprecated = true];
253253

254254
/* Allows the Server to provide additional Updates to be sent. Only the `number_of_permits` field needs to be
255255
set on this message.

0 commit comments

Comments
 (0)