[async REST] Remove guards for async rest transport and async client generation once async REST support is GA

オープン 初心者向け
#16,286 コメント 3 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
68/100
issue の種類
リファクタリング
明瞭さ
明確に書かれている
活発さ
静か
技術スタック
python
領域
build-system, tooling

調査の方向性

まず rest_asyncio.py.j2 と async_client.py.j2 を読み、生成されたコード周辺の実験的機能ガードに注目してください。GAPICs 向けのジェネレーターの出力を調査し、非同期 REST サポートが GA になった時点で rest_asyncio.py と async_client.py が無条件に生成されることを確認してください。

索引モデルが issue の本文から書いたものです。

説明

priority: p2 type: feature request

Once support for async REST transport goes from experimental -> GA,
we need to remove the following conditions that guard the generation of the async rest transport and the async client:

from rest_asyncio.py.j2:

{% set rest_async_io_enabled = api.all_library_settings[api.naming.proto_package].python_settings.experimental_features.rest_async_io_enabled %}
{% if rest_async_io_enabled %}
..code
{% endif %}{# if rest_async_io_enabled #}

from async_client.py.j2:

{% set rest_async_io_enabled = api.all_library_settings[api.naming.proto_package].python_settings.experimental_features.rest_async_io_enabled %}
{% set is_grpc_transport = True if 'grpc' in opts.transport else False %}
{% if rest_async_io_enabled or is_grpc_transport %}
..code
{% endif %}{# if rest_async_io_enabled and is_grpc_transport #}

Making this change wil generate the relevant files i.e rest_asyncio.py and async_client.py unconditionally for GAPICs.

主要言語
Python
スター
5.4k
フォーク
1.8k
平均マージ
1日 17時間
マージ済み PR（30日）
93

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

googleapis/google-cloud-python のほかの issue

googleapis/google-cloud-python の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。