FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add GAX-based _SubscriberAPI. by tseaver · Pull Request #1795 · googleapis/google-cloud-python · GitHub

Add GAX-based _SubscriberAPI. - #1795

Merged
tseaver merged 3 commits into
googleapis:masterfrom
tseaver:pubsub-wrap_gax_subscriber_api
May 16, 2016
Merged

Add GAX-based _SubscriberAPI.#1795
tseaver merged 3 commits into
googleapis:masterfrom
tseaver:pubsub-wrap_gax_subscriber_api

Conversation

tseaver commented May 13, 2016
edited
Loading

Copy link
Copy Markdown
Contributor

Uses #1764 as a base.

tseaver added the api: pubsub Issues related to the Pub/Sub API. label May 13, 2016
googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 13, 2016

tseaver commented May 16, 2016

Copy link
Copy Markdown
Contributor Author

@dhermes rebased after merging #1794 to master. PTAL

Comment thread gcloud/pubsub/_gax.py
"""List subscriptions for the project associated with this API.

See:
https://cloud.google.com/pubsub/reference/rest/v1/projects.subscriptions/list

This comment was marked as spam.

This comment was marked as spam.

dhermes commented May 16, 2016

Copy link
Copy Markdown
Contributor

LGTM. Please fix the small nits, but don't need to check with me once fixed.

tseaver merged commit bb26745 into googleapis:master May 16, 2016
tseaver deleted the pubsub-wrap_gax_subscriber_api branch May 16, 2016 15:54
tseaver mentioned this pull request May 16, 2016
parthea pushed a commit that referenced this pull request Nov 26, 2025
* update secrets

* 1

* 2
parthea pushed a commit that referenced this pull request Apr 1, 2026
* test: avoid exact float comparison in `test_apply_lambda`

* use by_row=False in apply_simple_udf too
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: pubsub Issues related to the Pub/Sub API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL