googleapis / googleapis/google-cloud-python

change table to target in mutations batcher

Ouverte
#15,301 1 commentaire 0 réactions 1 personne assignée Réclamée par @daniel-sanche Voir sur GitHub
api: bigtable priority: p3 type: cleanup
Langage dominant
Python
Étoiles
5.4k
Forks
1.8k
Merge moyen
3 j 4 h
PR mergées (30 j)
122

Description

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

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.