influxdata/telegraf

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

Open

#13.730 aperta il 7 ago 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Go (4161 fork)batch import
feature requesthelp wantedsize/m

Metriche repository

Star
 (9892 star)
Metriche merge PR
 (Merge medio 2g 12h) (136 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Aggiungi un nuovo parametro di configurazione `tagvalues` come array TOML nel plugin di input postgresql extensible, depreca `tagvalue` mantenendo la compatibilità all'indietro, aggiorna documentazione e test.
Tech stack
go
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GoTOML
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65