googleapis / googleapis/google-cloud-python
BidiRpc missing timeout and retry arguments
- 主要语言
- Python
- 星标
- 5.4k
- 派生
- 1.8k
- 平均合并
- 3 天 4 小时
- 30 天内合并 PR
- 122
描述
**Is your feature request related to a problem? Please describe.**
Most (all?) generated methods contain the following arguments:
```
retry: google.api_core.retry.Retry = google.api_core.gapic_v1.method.DEFAULT,
timeout: float = None,
metadata: Sequence[Tuple[str, str]] = (),
```
I'd like to be able to pass these through from the method I'm wrapping. Currently, this is possible with `metadata`. See:
https://github.com/googleapis/python-api-core/blob/40f52bf1100cf56b7f9af267d210b8a72fc34f08/google/api_core/bidi.py#L279
But it does not appear to be possible with other arguments such as `timeout` and `retry`.
**Describe the solution you'd like**
Add `retry` and `timeout` as optional arguments to `BidiRpc`.
Perhaps if `retry` and `timeout` aren't available in older generated clients, some additional logic will be required to prevent breaking them? I recall they've been around for quite some time, though.
**Describe alternatives you've considered**
Perhaps an arbitrary `**kwargs` dictionary for extra arguments to pass to the `_start_rpc` method?
**Additional context**
* https://github.com/googleapis/python-bigquery-storage/pull/278
* https://github.com/googleapis/google-cloud-python/issues/16450 -- I suspect this feature will need to be implemented before the generator can consider using BidiRpc by default.
贡献指南
评估
这个 Issue 还没有评估数据。