Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: update comments on protos #509

Merged
merged 2 commits into from
Mar 1, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 28 additions & 22 deletions protos/google/pubsub/v1/pubsub.proto
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ option java_package = "com.google.pubsub.v1";
option php_namespace = "Google\\Cloud\\PubSub\\V1";
option ruby_package = "Google::Cloud::PubSub::V1";


// The service that an application uses to manipulate topics, and to send
// messages to a topic.
service Publisher {
Expand Down Expand Up @@ -79,7 +78,8 @@ service Publisher {
}

// Lists the names of the subscriptions on this topic.
rpc ListTopicSubscriptions(ListTopicSubscriptionsRequest) returns (ListTopicSubscriptionsResponse) {
rpc ListTopicSubscriptions(ListTopicSubscriptionsRequest)
returns (ListTopicSubscriptionsResponse) {
option (google.api.http) = {
get: "/v1/{topic=projects/*/topics/*}/subscriptions"
};
Expand All @@ -94,7 +94,8 @@ service Publisher {
// <b>BETA:</b> This feature is part of a beta release. This API might be
// changed in backward-incompatible ways and is not recommended for production
// use. It is not subject to any SLA or deprecation policy.
rpc ListTopicSnapshots(ListTopicSnapshotsRequest) returns (ListTopicSnapshotsResponse) {
rpc ListTopicSnapshots(ListTopicSnapshotsRequest)
returns (ListTopicSnapshotsResponse) {
option (google.api.http) = {
get: "/v1/{topic=projects/*/topics/*}/snapshots"
};
Expand Down Expand Up @@ -125,9 +126,10 @@ service Subscriber {
// If the name is not provided in the request, the server will assign a random
// name for this subscription on the same project as the topic, conforming
// to the
// [resource name format](https://cloud.google.com/pubsub/docs/admin#resource_names).
// The generated name is populated in the returned Subscription object.
// Note that for REST API requests, you must specify a name in the request.
// [resource name
// format](https://cloud.google.com/pubsub/docs/admin#resource_names). The
// generated name is populated in the returned Subscription object. Note that
// for REST API requests, you must specify a name in the request.
rpc CreateSubscription(Subscription) returns (Subscription) {
option (google.api.http) = {
put: "/v1/{name=projects/*/subscriptions/*}"
Expand All @@ -152,7 +154,8 @@ service Subscriber {
}

// Lists matching subscriptions.
rpc ListSubscriptions(ListSubscriptionsRequest) returns (ListSubscriptionsResponse) {
rpc ListSubscriptions(ListSubscriptionsRequest)
returns (ListSubscriptionsResponse) {
option (google.api.http) = {
get: "/v1/{project=projects/*}/subscriptions"
};
Expand All @@ -163,7 +166,8 @@ service Subscriber {
// `NOT_FOUND`. After a subscription is deleted, a new one may be created with
// the same name, but the new one has no association with the old
// subscription or its topic unless the same topic is specified.
rpc DeleteSubscription(DeleteSubscriptionRequest) returns (google.protobuf.Empty) {
rpc DeleteSubscription(DeleteSubscriptionRequest)
returns (google.protobuf.Empty) {
option (google.api.http) = {
delete: "/v1/{subscription=projects/*/subscriptions/*}"
};
Expand All @@ -174,7 +178,8 @@ service Subscriber {
// subscriber, or to make the message available for redelivery if the
// processing was interrupted. Note that this does not modify the
// subscription-level `ackDeadlineSeconds` used for subsequent messages.
rpc ModifyAckDeadline(ModifyAckDeadlineRequest) returns (google.protobuf.Empty) {
rpc ModifyAckDeadline(ModifyAckDeadlineRequest)
returns (google.protobuf.Empty) {
option (google.api.http) = {
post: "/v1/{subscription=projects/*/subscriptions/*}:modifyAckDeadline"
body: "*"
Expand Down Expand Up @@ -212,16 +217,17 @@ service Subscriber {
// reassign server-side resources, in which case, the client should
// re-establish the stream. Flow control can be achieved by configuring the
// underlying RPC channel.
rpc StreamingPull(stream StreamingPullRequest) returns (stream StreamingPullResponse) {
}
rpc StreamingPull(stream StreamingPullRequest)
returns (stream StreamingPullResponse) {}

// Modifies the `PushConfig` for a specified subscription.
//
// This may be used to change a push subscription to a pull one (signified by
// an empty `PushConfig`) or vice versa, or change the endpoint URL and other
// attributes of a push subscription. Messages will accumulate for delivery
// continuously through the call regardless of changes to the `PushConfig`.
rpc ModifyPushConfig(ModifyPushConfigRequest) returns (google.protobuf.Empty) {
rpc ModifyPushConfig(ModifyPushConfigRequest)
returns (google.protobuf.Empty) {
option (google.api.http) = {
post: "/v1/{subscription=projects/*/subscriptions/*}:modifyPushConfig"
body: "*"
Expand Down Expand Up @@ -275,9 +281,10 @@ service Subscriber {
// the request, the server will assign a random
// name for this snapshot on the same project as the subscription, conforming
// to the
// [resource name format](https://cloud.google.com/pubsub/docs/admin#resource_names).
// The generated name is populated in the returned Snapshot object. Note that
// for REST API requests, you must specify a name in the request.
// [resource name
// format](https://cloud.google.com/pubsub/docs/admin#resource_names). The
// generated name is populated in the returned Snapshot object. Note that for
// REST API requests, you must specify a name in the request.
rpc CreateSnapshot(CreateSnapshotRequest) returns (Snapshot) {
option (google.api.http) = {
put: "/v1/{name=projects/*/snapshots/*}"
Expand Down Expand Up @@ -576,7 +583,8 @@ message Subscription {
// messages are not expunged from the subscription's backlog, even if they are
// acknowledged, until they fall out of the `message_retention_duration`
// window. This must be true if you would like to
// <a href="https://cloud.google.com/pubsub/docs/replay-overview#seek_to_a_time">
// <a
// href="https://cloud.google.com/pubsub/docs/replay-overview#seek_to_a_time">
// Seek to a timestamp</a>.
// <br><br>
// <b>BETA:</b> This feature is part of a beta release. This API might be
Expand Down Expand Up @@ -872,9 +880,9 @@ message StreamingPullResponse {
}

// Request for the `CreateSnapshot` method.<br><br>
// <b>BETA:</b> This feature is part of a beta release. This API might be changed in
// backward-incompatible ways and is not recommended for production use.
// It is not subject to any SLA or deprecation policy.
// <b>BETA:</b> This feature is part of a beta release. This API might be
// changed in backward-incompatible ways and is not recommended for production
// use. It is not subject to any SLA or deprecation policy.
message CreateSnapshotRequest {
// Optional user-provided name for this snapshot.
// If the name is not provided in the request, the server will assign a random
Expand Down Expand Up @@ -1028,6 +1036,4 @@ message SeekRequest {
}

// Response for the `Seek` method (this response is empty).
message SeekResponse {

}
message SeekResponse {}
9 changes: 5 additions & 4 deletions src/v1/doc/google/pubsub/v1/doc_pubsub.js
Original file line number Diff line number Diff line change
Expand Up @@ -378,7 +378,8 @@ const DeleteTopicRequest = {
* messages are not expunged from the subscription's backlog, even if they are
* acknowledged, until they fall out of the `message_retention_duration`
* window. This must be true if you would like to
* <a href="https://cloud.google.com/pubsub/docs/replay-overview#seek_to_a_time">
* <a
* href="https://cloud.google.com/pubsub/docs/replay-overview#seek_to_a_time">
* Seek to a timestamp</a>.
* <br><br>
* <b>BETA:</b> This feature is part of a beta release. This API might be
Expand Down Expand Up @@ -820,9 +821,9 @@ const StreamingPullResponse = {

/**
* Request for the `CreateSnapshot` method.<br><br>
* <b>BETA:</b> This feature is part of a beta release. This API might be changed in
* backward-incompatible ways and is not recommended for production use.
* It is not subject to any SLA or deprecation policy.
* <b>BETA:</b> This feature is part of a beta release. This API might be
* changed in backward-incompatible ways and is not recommended for production
* use. It is not subject to any SLA or deprecation policy.
*
* @property {string} name
* Optional user-provided name for this snapshot.
Expand Down
17 changes: 10 additions & 7 deletions src/v1/subscriber_client.js
Original file line number Diff line number Diff line change
Expand Up @@ -291,9 +291,10 @@ class SubscriberClient {
* If the name is not provided in the request, the server will assign a random
* name for this subscription on the same project as the topic, conforming
* to the
* [resource name format](https://cloud.google.com/pubsub/docs/admin#resource_names).
* The generated name is populated in the returned Subscription object.
* Note that for REST API requests, you must specify a name in the request.
* [resource name
* format](https://cloud.google.com/pubsub/docs/admin#resource_names). The
* generated name is populated in the returned Subscription object. Note that
* for REST API requests, you must specify a name in the request.
*
* @param {Object} request
* The request object that will be sent.
Expand Down Expand Up @@ -341,7 +342,8 @@ class SubscriberClient {
* messages are not expunged from the subscription's backlog, even if they are
* acknowledged, until they fall out of the `message_retention_duration`
* window. This must be true if you would like to
* <a href="https://cloud.google.com/pubsub/docs/replay-overview#seek_to_a_time">
* <a
* href="https://cloud.google.com/pubsub/docs/replay-overview#seek_to_a_time">
* Seek to a timestamp</a>.
* <br><br>
* <b>BETA:</b> This feature is part of a beta release. This API might be
Expand Down Expand Up @@ -1180,9 +1182,10 @@ class SubscriberClient {
* the request, the server will assign a random
* name for this snapshot on the same project as the subscription, conforming
* to the
* [resource name format](https://cloud.google.com/pubsub/docs/admin#resource_names).
* The generated name is populated in the returned Snapshot object. Note that
* for REST API requests, you must specify a name in the request.
* [resource name
* format](https://cloud.google.com/pubsub/docs/admin#resource_names). The
* generated name is populated in the returned Snapshot object. Note that for
* REST API requests, you must specify a name in the request.
*
* @param {Object} request
* The request object that will be sent.
Expand Down
6 changes: 3 additions & 3 deletions synth.metadata
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"updateTime": "2019-02-28T12:18:55.555938Z",
"updateTime": "2019-03-01T12:18:24.808590Z",
"sources": [
{
"generator": {
Expand All @@ -12,8 +12,8 @@
"git": {
"name": "googleapis",
"remote": "https://github.com/googleapis/googleapis.git",
"sha": "9c769d3a0e67e4df9b9e8eee480124c2700a7e6c",
"internalRef": "235997788"
"sha": "41d72d444fbe445f4da89e13be02078734fb7875",
"internalRef": "236230004"
}
},
{
Expand Down