influxdata/telegraf

postgresql_extensible: query.tagvalue is strangely a CSV rather than a TOML array

Open

#13.730 geöffnet am 7. Aug. 2023

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Go (4.161 Forks)batch import
feature requesthelp wantedsize/m

Repository-Metriken

Stars
 (9.892 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 2T 12h) (136 gemergte PRs in 30 T)

Beschreibung

Use Case

When creating custom queries tagvalue is widely used to define result columns to be used as tag. Given that whole config file is a TOML syntax, use of CSV string format for tagvalue feels less natural than what TOML array would be.

Expected behavior

Supporting 2 types for the same config param can be messy, so I propose deprecate tagvalue and introduce tagvalues parameter with array type.

Actual behavior

tagvalue is a CSV string.

Additional info

No response

Contributor Guide

Research-Richtung
Fügen Sie einen neuen Konfigurationsparameter `tagvalues` als TOML Array im postgresql extensible Eingabe Plugin hinzu, veralten Sie `tagvalue` unter Beibehaltung der Abwärtskompatibilität, aktualisieren Sie Dokumentation und Tests.
Tech Stack
go
Domain
backend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GoTOML
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65