influxdata / influxdata/influxdb-client-java
InfluxQLQueryApi.queryJson should not transform all values to String
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Java
- Sterne
- 477
- Forks
- 129
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Proposal:
It would be nice if influxQLQueryApi.queryJson() can preserve field types instead of converting everything to Strings.
ValueExtractor should expose an API with raw types.
Current behavior:
Unless a ValueExtractor is present, every values get transformed to String even of the InfluxDB Json result contains a non string value.
Desired behavior:
Unless a value extractor is specified, the JSON result should preserve original response types as much as possible
Alternatives considered:
For now, provide a value extractor that parse each String values.
It works but doing twice the job: for a number value it get read by Gson as Number then the client converts it to String then manually converted to back to a number.
Use case:
All use cases using influxQLQueryApi.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne bei InfluxQLQueryApi.queryJson und verfolge, wie ValueExtractor Werte aus der InfluxDB-JSON-Antwort verarbeitet. Prüfe das Verhalten der bestehenden Query-API und, falls vorhanden, die Tests, um festzustellen, wo Werte in String konvertiert werden. Als erledigt gilt, wenn queryJson standardmäßig Antworttypen beibehält, die keine Strings sind, und weiterhin einen explizit angegebenen ValueExtractor unterstützt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java, json
- Bereich
- api, databases
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100