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

Adding namespace to dataset class. by dhermes · Pull Request #934 · googleapis/google-cloud-python · GitHub

Adding namespace to dataset class. - #934

Merged
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:add-name-space-to-dataset
Jun 23, 2015
Merged

Adding namespace to dataset class.#934
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:add-name-space-to-dataset

Conversation

dhermes commented Jun 22, 2015

Copy link
Copy Markdown
Contributor

This does not add support for namespace in _implicit_environ.

Related to #902.


As a result of #861 we will likely be removing _implicit_environ which makes this the actual resolution of #902.

/cc @jonparrott

This does not add support for namespace in `_implicit_environ`.

Related to googleapis#902.
dhermes added the api: datastore Issues related to the Datastore API. label Jun 22, 2015
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 22, 2015

dhermes commented Jun 22, 2015

Copy link
Copy Markdown
Contributor Author

FWIW I checked where namespace is used in the .proto API. It is only used in PartitionId (along with dataset_id) and the PartitionId message type is only used in Key and RunQueryRequest.

Accordingly, the only places we support namespace is:

  • Connection.run_query(dataset_id, query_pb, namespace=...)
  • Key(..., namespace=...)
  • Query(..., namespace=...)

Copy link
Copy Markdown
Contributor

Cool, as long as it's an option on the client class when that happens I'm satisfied.

tseaver commented Jun 23, 2015

Copy link
Copy Markdown
Contributor

LGTM. I plan to rename datastore.dataset.Dataset -> datastore.client.Client after this merges.

dhermes commented Jun 23, 2015

Copy link
Copy Markdown
Contributor Author

OK SGTM

dhermes added a commit that referenced this pull request Jun 23, 2015
dhermes merged commit 88f7c83 into googleapis:master Jun 23, 2015
dhermes deleted the add-name-space-to-dataset branch June 23, 2015 20:18
dhermes mentioned this pull request Jul 10, 2015
parthea pushed a commit that referenced this pull request Nov 22, 2025
* chore(deps): update all dependencies

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Mar 2, 2026
* chore(deps): update all dependencies

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Mar 6, 2026
* fix: guard imports against unsupported pyarrow versions

* add unit tests

* fix pytype

* second try at fixing pytype
parthea pushed a commit that referenced this pull request Mar 6, 2026
parthea pushed a commit that referenced this pull request Mar 9, 2026
* fix: rst retry for txn

* rst changes and tests

* fix

* rst stream comment changes

* lint

* lint
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: datastore Issues related to the Datastore API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL