Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Sep 26, 2023. It is now read-only.
This repository was archived by the owner on Sep 26, 2023. It is now read-only.

[gax] wrap non-retryable RPCs in the same retryable callable machinery #1327

@noahdietz

Description

@noahdietz

Today, those methods that are configured at the client-level to be non-retryable would not benefit from some ApiCallContext improvements, like supporting withRetrySettings and withRetryableCodes added in #1238, and a proposal to add a new logical timeout option. This is because the callable for such methods is not wrapped in the same context-aware machinery that retryable methods are enabling such features.

We should consider wrapping the non-retryable RPCs in the same callable machinery that would enable retries. Since the retry settings would prevent retries, there shouldn't be a behavioral issue there.

Metadata

Metadata

Assignees

No one assigned

    Labels

    type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions