Fusionnée le 6 octobre 2021. Retirée le 6 octobre 2026. Pendant cinq ans jour pour jour, une ligne de débogage a imprimé les labels de dataset BigQuery en niveau warning, chaque fois que le SDK Python d'Apache Beam préparait un dataset temporaire.
_LOGGER.warning("### Labels: %s", str(self.bigquery_dataset_labels))Pourquoi elle ressemblait à un oubli
- ### n'apparaît dans aucun autre appel de log parmi les 1 380 fichiers Python du SDK
- str() est redondant avec %s
- C'était le seul appel en warning du module, et il rapportait une valeur fournie par l'appelant lui-même
git blame situait la ligne dans la pull request qui ajoutait le support de l'API BigQuery Read, fusionnée en octobre 2021. Elle n'avait pas bougé depuis.
Pourquoi on pouvait la retirer
Aucun test n'y faisait référence : la seule assertion de log du paquet vise un message de debug sur un autre chemin. Le logger du module reste utilisé à neuf endroits, rien ne devenait inutilisé. Le formateur épinglé par le projet ne produisait aucun diff. Et au cas où les mainteneurs voudraient garder la valeur visible, j'ai proposé de la passer en debug plutôt que de la supprimer.
Ce que j'en retiens
Une pull request d'une ligne doit quand même gagner un argument. Le diff ne peut pas le faire, la description s'en charge : d'où vient la ligne, pourquoi c'est du bruit, ce qui en dépend. Ici, la réponse à la dernière question était rien, et le dire preuves à l'appui l'a fait fusionner.
Ayoub Hammoudi, Ingénieur logiciel, full-stack et plateformes IA. github.com/Ayoubhm07
