Skip to content
This repository was archived by the owner on Mar 4, 2026. It is now read-only.

patch: ensure that the client pool consistently uses grpc clients after transitioning from rest#1807

Merged
MarkDuckworth merged 5 commits intomainfrom
markduckworth/preferRest-simple-transition
Jan 3, 2023
Merged

patch: ensure that the client pool consistently uses grpc clients after transitioning from rest#1807
MarkDuckworth merged 5 commits intomainfrom
markduckworth/preferRest-simple-transition

Conversation

@MarkDuckworth
Copy link
Copy Markdown
Contributor

@MarkDuckworth MarkDuckworth commented Dec 20, 2022

patch: Ensure that the client pool consistently uses gRPC clients after transitioning from REST. Previously there were some edge cases where the client pool may create a REST client after it had transitioned to gRPC. Additionally, API docs were added for preferRest.

BEGIN_COMMIT_OVERRIDE
fix: Ensure that the client pool consistently uses gRPC clients after transitioning from REST
END_COMMIT_OVERRIDE

…, so that the client pool consistently uses grpc after the transition, rather than occasionally reverting back to REST.
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. api: firestore Issues related to the googleapis/nodejs-firestore API. labels Dec 20, 2022
@MarkDuckworth MarkDuckworth marked this pull request as ready for review December 20, 2022 17:29
@MarkDuckworth MarkDuckworth requested review from a team December 20, 2022 17:29
Copy link
Copy Markdown
Contributor

@ehsannas ehsannas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the test coverage!

Comment thread dev/test/pool.ts
@MarkDuckworth MarkDuckworth added the automerge Merge the pull request once unit tests and other checks pass. label Jan 3, 2023
@ehsannas ehsannas assigned MarkDuckworth and unassigned ehsannas Jan 3, 2023
@MarkDuckworth MarkDuckworth merged commit 3068361 into main Jan 3, 2023
@MarkDuckworth MarkDuckworth deleted the markduckworth/preferRest-simple-transition branch January 3, 2023 18:41
@gcf-merge-on-green gcf-merge-on-green Bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jan 3, 2023
@MarkDuckworth MarkDuckworth added the release-please:force-run To run release-please label Jan 9, 2023
@release-please release-please Bot removed the release-please:force-run To run release-please label Jan 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: firestore Issues related to the googleapis/nodejs-firestore API. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants