-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
List operations do not return nextPageToken which prevents proper paging #665
Labels
api: cloudresourcemanager
Issues related to the Resource Manager API.
api: storage
Issues related to the Cloud Storage API.
type: bug
Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Comments
@mderka do you want to assign yourself this issue and fix it in BigQuery, Storage, and ResourceManager? |
ajkannan
added
type: bug
Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
api: bigquery
Issues related to the BigQuery API.
api: storage
Issues related to the Cloud Storage API.
api: cloudresourcemanager
Issues related to the Resource Manager API.
labels
Feb 25, 2016
Nice catch!! |
I will fix it tomorrow. |
ajkannan
pushed a commit
to ajkannan/gcloud-java
that referenced
this issue
Mar 4, 2016
Debugged parsing for options. Fixed context for the server URLs and regular expression parsing. Fixed paging. Improved error detection for missing zones vs. non-existent changes.
github-actions bot
pushed a commit
to suztomo/google-cloud-java
that referenced
this issue
Jun 29, 2022
- [ ] Regenerate this pull request now. Committer: @parthea PiperOrigin-RevId: 407854413 Source-Link: googleapis/googleapis@f81b7bd Source-Link: https://github.com/googleapis/googleapis-gen/commit/1f0cfde5514fae024c673b6798174ab6727cb0a5 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMWYwY2ZkZTU1MTRmYWUwMjRjNjczYjY3OTgxNzRhYjY3MjdjYjBhNSJ9
github-actions bot
pushed a commit
to suztomo/google-cloud-java
that referenced
this issue
Jun 29, 2022
🤖 I have created a release \*beep\* \*boop\* --- ## [1.2.0](https://www.github.com/googleapis/java-resourcemanager/compare/v1.1.4...v1.2.0) (2021-12-03) ### Features * next release from main branch is 1.2.0 ([googleapis#659](https://www.github.com/googleapis/java-resourcemanager/issues/659)) ([8af7d6d](https://www.github.com/googleapis/java-resourcemanager/commit/8af7d6de5eb05cf3c08c52e13e4ef47a51fb7d36)) ### Bug Fixes * **java:** java 17 dependency arguments ([googleapis#1266](https://www.github.com/googleapis/java-resourcemanager/issues/1266)) ([googleapis#656](https://www.github.com/googleapis/java-resourcemanager/issues/656)) ([2bb306a](https://www.github.com/googleapis/java-resourcemanager/commit/2bb306a23319c415666f72ff8f793385c5a7a3f0)) ### Documentation * fix docstring formatting ([googleapis#665](https://www.github.com/googleapis/java-resourcemanager/issues/665)) ([cb79f5f](https://www.github.com/googleapis/java-resourcemanager/commit/cb79f5f49b2d3941c698c11c2e0177e89a44e153)) * fix docstring formatting ([googleapis#673](https://www.github.com/googleapis/java-resourcemanager/issues/673)) ([5e371b9](https://www.github.com/googleapis/java-resourcemanager/commit/5e371b93873b361e5d9b54a29c50bc91d63e39b3)) ### Dependencies * update dependency com.google.apis:google-api-services-cloudresourcemanager to v1-rev20211017-1.32.1 ([googleapis#654](https://www.github.com/googleapis/java-resourcemanager/issues/654)) ([d489a3e](https://www.github.com/googleapis/java-resourcemanager/commit/d489a3e486113d75caa419e5fe0ab182764f7b49)) * update dependency com.google.apis:google-api-services-cloudresourcemanager to v1-rev20211107-1.32.1 ([googleapis#670](https://www.github.com/googleapis/java-resourcemanager/issues/670)) ([c648624](https://www.github.com/googleapis/java-resourcemanager/commit/c648624456d10518971d3a8ad2ef897a8f520576)) * update dependency com.google.cloud:google-cloud-shared-dependencies to v2.5.0 ([googleapis#672](https://www.github.com/googleapis/java-resourcemanager/issues/672)) ([c07eaef](https://www.github.com/googleapis/java-resourcemanager/commit/c07eaef86911a560e51b85b726d26c1f82e9cbf1)) * update dependency com.google.cloud:google-cloud-shared-dependencies to v2.5.1 ([googleapis#678](https://www.github.com/googleapis/java-resourcemanager/issues/678)) ([3bfbff5](https://www.github.com/googleapis/java-resourcemanager/commit/3bfbff573edbf1e11472f8eba130470a0c9d7841)) * update dependency com.google.errorprone:error_prone_annotations to v2.10.0 ([googleapis#663](https://www.github.com/googleapis/java-resourcemanager/issues/663)) ([ed01782](https://www.github.com/googleapis/java-resourcemanager/commit/ed0178221930b4c3796d993e467c2481170ec863)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
github-actions bot
pushed a commit
that referenced
this issue
Jul 14, 2022
… v2.6.0 (#665) * chore(deps): update dependency com.google.cloud:google-cloud-redis to v2.6.0 * 🦉 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>
github-actions bot
pushed a commit
that referenced
this issue
Aug 18, 2022
… to v2.3.2 (#665) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [com.google.cloud:google-cloud-memcache](https://togithub.com/googleapis/java-memcache) | `2.3.0` -> `2.3.2` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-memcache/2.3.2/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-memcache/2.3.2/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-memcache/2.3.2/compatibility-slim/2.3.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-memcache/2.3.2/confidence-slim/2.3.0)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Renovate will not automatically rebase this PR, because other commits have been found. 🔕 **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox. ⚠ **Warning**: custom changes will be lost. --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-memcache). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4xNTMuMCIsInVwZGF0ZWRJblZlciI6IjMyLjE1My4wIn0=-->
github-actions bot
pushed a commit
that referenced
this issue
Sep 15, 2022
…1575) (#665) Source-Link: googleapis/synthtool@2e9ac19 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:8175681a918181d306d9c370d3262f16b4c724cc73d74111b7d42fc985ca7f93
suztomo
pushed a commit
that referenced
this issue
Feb 1, 2023
🤖 I have created a release *beep* *boop* --- ### Updating meta-information for bleeding-edge SNAPSHOT release. --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
api: cloudresourcemanager
Issues related to the Resource Manager API.
api: storage
Issues related to the Cloud Storage API.
type: bug
Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
When implementing integration tests for DNS, I noticed that if one restricts the fields to be retrieved for a list operation,
iterateAll()
does not work properly. The following test would always fail:This is because the field selector silently prevents the service from returning the
nextPageToken
:builder.append("rrsets(").append(DnsRecordField.selector(fields)).append(')');
instead of
builder.append("nextPageToken,rrsets(").append(DnsRecordField.selector(fields)).append(')');
I recall seeing the same handling in resource manage. Having discussed this with @ajkannan, this is likely a bug. The same bug may be present in other parts of the library when a combination of listing and field restriction is possible.
The text was updated successfully, but these errors were encountered: