cube-js/cube

Elasticsearch - Result formatting differences

Offen

#2.663 geöffnet am 06.05.2021

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (1.965 Forks)batch import
community supporteddriver:elasticsearchhelp wanted

Repository-Metriken

Stars
 (19.563 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

A recent change to conform with elaastic.co result formatting (to cope with json result) is not compatible with open distro/aws es sql result formatting. In open distro case, json formatting is more like the standard es result format whereas json format in elastic.co is more like a jdbc format.

This is the offending code. https://github.com/cube-js/cube.js/blob/c1e25d7184c5ed63ba083da1cd9cf0f2111b8011/packages/cubejs-elasticsearch-driver/driver/ElasticSearchDriver.js#L86

We perhaps need to split the driver in some way - either entirely or certainly the query and driver.

I am happy to look at PR for this

To Reproduce setting result format to json while using awses/open distro - json result formatting results in an exception

Expected behavior Both open distro and elastic.co should be supported either via single driver or multiple

Version: latest

Contributor Guide

Research-Richtung
Das Problem resultiert aus einer Änderung in cubejs elasticsearch driver, um der Ergebnisformatierung von elastic.co zu entsprechen, wodurch die Kompatibilität mit open distro/aws ES gebrochen wurde. Der problematische Code befindet sich in packages/cubejs elasticsearch driver/driver/ElasticSearchDriver.js um Zeile 86. Die Lösung könnte die Aufteilung des Treibers in zwei Versionen oder die Konfigurierbarkeit der Formatierung beinhalten. Untersuchen Sie die Unterschiede im Ergebnisformat zwischen elastic.co und open distro ES SQL und erwägen Sie das Hinzufügen eines Flags oder eines separaten Treibers.
Tech Stack
javascript
Domain
backenddatabase
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75