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 小时
- 30 天内合并 PR
- 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 还没有评估数据。