googleapis / googleapis/google-cloud-python
change table to target in mutations batcher
オープン
api: bigtable
priority: p3
type: cleanup
- 主要言語
- Python
- スター
- 5.4k
- フォーク
- 1.8k
- 平均マージ
- 3日 4時間
- マージ済み PR(30日)
- 122
説明
The mutation batcher's [first argument is called table](https://github.com/googleapis/python-bigtable/blob/97a019833d82e617769c56761aa5548d3ab896b9/google/cloud/bigtable/data/_async/mutations_batcher.py#L215). Going forward, it should be target, since it can be a table or authorized view
In practice, this change shouldn't break much, since it's typically either constructed using `table.mutations_batcher()`, or `MutationsBatcher(table)`. But it would break users who create a batcher using `MutationsBatcher(table=table)`. So for backwards compatibility reasons, we should ony change this when we're ready for a major version bump
コントリビューションガイド
評価
この issue はまだ評価されていません。