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.

Sign up for GitHub

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

Jump to bottom

Updating documentation to reflect bulk datastore API changes. #506

Merged
merged 3 commits into from
Jan 8, 2015
Merged

Updating documentation to reflect bulk datastore API changes. #506

merged 3 commits into from
Jan 8, 2015

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Jan 8, 2015

Address final part of #477 but does not resolve cyclic import (I am planning to resolve after this is merged).

Also fixes broken builds caused by :metho: -> :method: -> :meth: confusion:
https://travis-ci.org/GoogleCloudPlatform/gcloud-python/builds/46255178
https://travis-ci.org/GoogleCloudPlatform/gcloud-python/builds/46255166
...

@tseaver I noticed that the signature and :raises: statement for Query.__init__ is in the class docstring rather than that of __init__. Is there a reason for that? Is there a preference for one over the other?

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 073478a on dhermes:fix-477-part9-docs-fix into * on GoogleCloudPlatform:master*.

tseaver
tseaver reviewed Jan 8, 2015
View reviewed changes
gcloud/connection.py
in method arguments,
however they should be capable
of returning advanced types.
Subclasses should understand only the basic types in method arguments,

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@dhermes
Removing docstring lines with only empty '>>>'.
48bcddc 
Found via "git grep -e '>>>$'".
tseaver
tseaver reviewed Jan 8, 2015
View reviewed changes
gcloud/datastore/key.py
:raises: `ValueError` if the current key is not partial or if
`id_or_name` is not a string or integer.
:raises: :class:`ValueError` if the current key is not partial or if
``id_or_name`` is not a string or integer.

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 48bcddc on dhermes:fix-477-part9-docs-fix into * on GoogleCloudPlatform:master*.

tseaver
tseaver reviewed Jan 8, 2015
View reviewed changes
gcloud/datastore/connection.py
if status != '200':
message = ('Request failed with status code %s. '
'Error was: %s' % (status, content))
raise six.moves.http_client.HTTPException(message)

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@dhermes
Ditching :class: reference for builtins.
8b950cf 
Making best attempt at being consistent (i.e. string, not str,
boolean, not bool, etc.).

Also adding a block for `id_or_name` in Key.completed_key.
@tseaver tseaver mentioned this pull request Jan 8, 2015
Closed
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 8b950cf on dhermes:fix-477-part9-docs-fix into * on GoogleCloudPlatform:master*.

@dhermes
Copy link
Contributor Author

dhermes commented Jan 8, 2015

@tseaver Any lingering issues here?

@dhermes dhermes added api: datastore Issues related to the Datastore API. documentation labels Jan 8, 2015
@tseaver
Copy link
Contributor

tseaver commented Jan 8, 2015

LGTM.

dhermes added a commit that referenced this pull request Jan 8, 2015
@dhermes
Merge pull request #506 from dhermes/fix-477-part9-docs-fix
f577b30 
Updating documentation to reflect bulk datastore API changes.
@dhermes dhermes merged commit f577b30 into googleapis:master Jan 8, 2015
@dhermes dhermes deleted the fix-477-part9-docs-fix branch January 8, 2015 02:56
@dhermes dhermes mentioned this pull request Jan 13, 2015
Closed
parthea pushed a commit that referenced this pull request Aug 15, 2023
@gcf-owl-bot
feat: Added persist_parameter_changes field from query_params to Ma…
b1765c0 
…tchIntentRequest (#506)

* feat: Added persist_parameter_changes field from `query_params` to MatchIntentRequest

PiperOrigin-RevId: 513016804

Source-Link: googleapis/googleapis@388f408

Source-Link: googleapis/googleapis-gen@99ede80
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiOTllZGU4MGQzZDE2N2NjMzk2YTZiYTg1ZTBiYTk0OTVhYmI2MDk3ZiJ9

* 🦉 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 Sep 22, 2023
@release-please
chore(main): release 3.12.2 (#506)
6416a28 
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Sep 22, 2023
@yoshi-automation @busunkim96
feat: add support for streaming automl action recognition in v1p3beta…
75c6c3a 
…1; make 'features' a positional param for annotate_video in betas (#31)

* Integrate Python GAPIC Microgenerator in googleapis.
This PR uses using documentai as an example.
Depends on googleapis/gapic-generator-python#402

PiperOrigin-RevId: 309824146

Source-Author: Google APIs <noreply@google.com>
Source-Date: Mon May 4 15:06:44 2020 -0700
Source-Repo: googleapis/googleapis
Source-Sha: e0f9d9e1f9de890db765be46f45ca8490723e3eb
Source-Link: googleapis/googleapis@e0f9d9e

* chore(python): create thread_safety doc (#517)

This will be included in index.rst files. See https://github.com/googleapis/python-bigtable/pull/26/files

Source-Author: Bu Sun Kim <8822365+busunkim96@users.noreply.github.com>
Source-Date: Tue May 5 10:06:52 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: 09c48461232ce929c34386259eb59018ad2d8eef
Source-Link: googleapis/synthtool@09c4846

* chore(python): remove extra #!/bin/bash (#538)

Co-authored-by: Jeffrey Rennie <rennie@google.com>

Source-Author: Bu Sun Kim <8822365+busunkim96@users.noreply.github.com>
Source-Date: Tue May 12 09:51:12 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: 7482e79a82e353248769d819788adc1213e8c207
Source-Link: googleapis/synthtool@7482e79

* chore!: enable gapicv2 for monitoring/v3 API

This introduces the following breakages upon code regeneration, which are due to the `TimeSeriesQueryLanguageCondition` type being removed from `alert.proto` in a preceding change.

```
[ERROR] 6011: com.google.monitoring.v3.AlertPolicy$Condition: Field CONDITION_TIME_SERIES_QUERY_LANGUAGE_FIELD_NUMBER has been removed, but it was previously a constant
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$Condition: Method 'public com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageCondition getConditionTimeSeriesQueryLanguage()' has been removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$Condition: Method 'public com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageConditionOrBuilder getConditionTimeSeriesQueryLanguageOrBuilder()' has been removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$Condition: Method 'public boolean hasConditionTimeSeriesQueryLanguage()' has been removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$Condition$Builder: Method 'public com.google.monitoring.v3.AlertPolicy$Condition$Builder clearConditionTimeSeriesQueryLanguage()' has been removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$Condition$Builder: Method 'public com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageCondition getConditionTimeSeriesQueryLanguage()' has been removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$Condition$Builder: Method 'public com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageCondition$Builder getConditionTimeSeriesQueryLanguageBuilder()' has been removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$Condition$Builder: Method 'public com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageConditionOrBuilder getConditionTimeSeriesQueryLanguageOrBuilder()' has been removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$Condition$Builder: Method 'public boolean hasConditionTimeSeriesQueryLanguage()' has been removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$Condition$Builder: Method 'public com.google.monitoring.v3.AlertPolicy$Condition$Builder mergeConditionTimeSeriesQueryLanguage(com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageCondition)' has been removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$Condition$Builder: Method 'public com.google.monitoring.v3.AlertPolicy$Condition$Builder setConditionTimeSeriesQueryLanguage(com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageCondition)' has been removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$Condition$Builder: Method 'public com.google.monitoring.v3.AlertPolicy$Condition$Builder setConditionTimeSeriesQueryLanguage(com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageCondition$Builder)' has been removed
[ERROR] 6001: com.google.monitoring.v3.AlertPolicy$Condition$ConditionCase: Removed field CONDITION_TIME_SERIES_QUERY_LANGUAGE
[ERROR] 8001: com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageCondition: Class com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageCondition removed
[ERROR] 8001: com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageCondition$Builder: Class com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageCondition$Builder removed
[ERROR] 8001: com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageConditionOrBuilder: Class com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageConditionOrBuilder removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$ConditionOrBuilder: Method 'public com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageCondition getConditionTimeSeriesQueryLanguage()' has been removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$ConditionOrBuilder: Method 'public com.google.monitoring.v3.AlertPolicy$Condition$TimeSeriesQueryLanguageConditionOrBuilder getConditionTimeSeriesQueryLanguageOrBuilder()' has been removed
[ERROR] 7002: com.google.monitoring.v3.AlertPolicy$ConditionOrBuilder: Method 'public boolean hasConditionTimeSeriesQueryLanguage()' has been removed
```

Committer: @miraleung
PiperOrigin-RevId: 311581003

Source-Author: Google APIs <noreply@google.com>
Source-Date: Thu May 14 12:13:51 2020 -0700
Source-Repo: googleapis/googleapis
Source-Sha: 123873dc7e2be35252d172205f67a375e70e9747
Source-Link: googleapis/googleapis@123873d

* fix: restore GAPIC v2 retry configs

Restores GAPIC v2 retry config overrides that were
mistakenly removed during migration.

PiperOrigin-RevId: 312088359

Source-Author: Google APIs <noreply@google.com>
Source-Date: Mon May 18 08:57:12 2020 -0700
Source-Repo: googleapis/googleapis
Source-Sha: 5a90d467aa65e7f038f87585e8fbb45d74475e7c
Source-Link: googleapis/googleapis@5a90d46

* feat(v1p3beta1): added support for streaming automl action recognition.

PiperOrigin-RevId: 312101156

Source-Author: Google APIs <noreply@google.com>
Source-Date: Mon May 18 10:04:40 2020 -0700
Source-Repo: googleapis/googleapis
Source-Sha: d1a9f02fd4fb263bae0383b4a5af0bbef33753d6
Source-Link: googleapis/googleapis@d1a9f02

* fix: point artman at gapic v1 for monitoring/v3 API

Committer: @miraleung
PiperOrigin-RevId: 312104480

Source-Author: Google APIs <noreply@google.com>
Source-Date: Mon May 18 10:19:39 2020 -0700
Source-Repo: googleapis/googleapis
Source-Sha: e94ad376529fabdfd2fce00407926a0cefd38b9e
Source-Link: googleapis/googleapis@e94ad37

* Dialogflow weekly v2 library update:
- Minor comment updates.

PiperOrigin-RevId: 312123588

Source-Author: Google APIs <noreply@google.com>
Source-Date: Mon May 18 11:49:18 2020 -0700
Source-Repo: googleapis/googleapis
Source-Sha: cd3ce2651c3921670217e664303976cdf76e9fe2
Source-Link: googleapis/googleapis@cd3ce26

* chore(python): remove one remaining extra bash line (#565)

To avoid generating bogus PRs

Co-authored-by: Jeffrey Rennie <rennie@google.com>

Source-Author: Carlos de la Guardia <cguardia@yahoo.com>
Source-Date: Tue May 19 11:57:19 2020 -0500
Source-Repo: googleapis/synthtool
Source-Sha: d2364eb80b840a36136c8ce12f1c6efabcc9600e
Source-Link: googleapis/synthtool@d2364eb

* Upgrade protoc-docs-plugin version to 0.7.0 to fix unstable docstrings.

PiperOrigin-RevId: 312689208

Source-Author: Google APIs <noreply@google.com>
Source-Date: Thu May 21 10:00:47 2020 -0700
Source-Repo: googleapis/googleapis
Source-Sha: dec3204175104cef49bf21d685d5517caaf0058f
Source-Link: googleapis/googleapis@dec3204

* bazel: update protobuf, rules_go, gazelle, and gapic-generator-go versions
- protobuf v3.12.1
- rules_go v0.23.0
- gazelle v0.21.0
- gapic-generator-go v0.14.1

PiperOrigin-RevId: 313460921

Source-Author: Google APIs <noreply@google.com>
Source-Date: Wed May 27 14:10:16 2020 -0700
Source-Repo: googleapis/googleapis
Source-Sha: c4e37010d74071851ff24121f522e802231ac86e
Source-Link: googleapis/googleapis@c4e3701

* Use correct resource type for DetachSubscriptionRequest

PiperOrigin-RevId: 313488995

Source-Author: Google APIs <noreply@google.com>
Source-Date: Wed May 27 16:45:32 2020 -0700
Source-Repo: googleapis/googleapis
Source-Sha: eafa840ceec23b44a5c21670288107c661252711
Source-Link: googleapis/googleapis@eafa840

* docs: update python docs template (#576)

* docs: update python docs template

* sphinx change

Source-Author: kolea2 <45548808+kolea2@users.noreply.github.com>
Source-Date: Wed May 27 20:44:34 2020 -0400
Source-Repo: googleapis/synthtool
Source-Sha: 71b8a272549c06b5768d00fa48d3ae990e871bec
Source-Link: googleapis/synthtool@71b8a27

* feat: add templates for python samples projects (#506)

These templates will be used for templates in python-docs-samples
and in Python client libraries. The README generation code is a modified
version of
https://github.com/GoogleCloudPlatform/python-docs-samples/tree/master/scripts/readme-gen.

Co-authored-by: Kurtis Van Gent <31518063+kurtisvg@users.noreply.github.com>

Source-Author: Bu Sun Kim <8822365+busunkim96@users.noreply.github.com>
Source-Date: Thu May 28 14:39:58 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: ffe10407ee2f261c799fb0d01bf32a8abc67ed1e
Source-Link: googleapis/synthtool@ffe1040

* chore: update default ignores for Python libraries (#586)

Source-Author: Bu Sun Kim <8822365+busunkim96@users.noreply.github.com>
Source-Date: Thu May 28 14:43:37 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: e99975b6b49827b8720f0a885e218dbdb67849ca
Source-Link: googleapis/synthtool@e99975b

* feat: allow custom python versions in noxfile (#585)

Libraries on the microgenerator support a smaller range of Python versions (3.6+).

Source-Author: Bu Sun Kim <8822365+busunkim96@users.noreply.github.com>
Source-Date: Thu May 28 18:22:04 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: 4e1d2cb79b02d7496b1452f91c518630c207145e
Source-Link: googleapis/synthtool@4e1d2cb

* docs: fix docs

Co-authored-by: Bu Sun Kim <busunkim@google.com>
parthea pushed a commit that referenced this pull request Oct 21, 2023
@gcf-owl-bot
feat: add a new searchable field kmsKeys (#506)
d3f0a11 
- [ ] Regenerate this pull request now.

fix: deprecate searchable field kmsKey

PiperOrigin-RevId: 479103182

Source-Link: https://togithub.com/googleapis/googleapis/commit/7b5a467b978ff2dde6cd34717ebad5728d05f2bb

Source-Link: https://togithub.com/googleapis/googleapis-gen/commit/417aefa0fb633d0f171a607779e9f0e19c8b8b7f
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNDE3YWVmYTBmYjYzM2QwZjE3MWE2MDc3NzllOWYwZTE5YzhiOGI3ZiJ9
parthea pushed a commit that referenced this pull request Oct 21, 2023
@renovate-bot
chore(deps): update dependency google-cloud-monitoring to v2.11.3 (#506)
d4a0d25
parthea pushed a commit that referenced this pull request Oct 21, 2023
@release-please
chore(main): release 3.11.3 (#506)
444125b 
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Oct 21, 2023
@gcf-owl-bot
docs: Minor formatting (#506)
7ec4c47 
* docs: Minor formatting
chore: Update gapic-generator-python to v1.11.5
build: Update rules_python to 0.24.0

PiperOrigin-RevId: 563436317

Source-Link: googleapis/googleapis@42fd37b

Source-Link: googleapis/googleapis-gen@280264c
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMjgwMjY0Y2EwMmZiOTMxNmI0MjM3YTk2ZDBhZjFhMjM0M2E4MWE1NiJ9

* 🦉 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
api: datastore Issues related to the Datastore API.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dhermes @coveralls @tseaver