4paradigm / 4paradigm/OpenMLDB

support execute multi sql in ExecuteSQL

Offen
#2,016 2 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @vagetablechicken Auf GitHub ansehen
enhancement storage-engine
Vorherrschende Sprache
C++
Sterne
1.7k
Forks
331
Ø Merge
12 T. 12 Std.
Gemergte PRs (30 T.)
1

Beschreibung

https://github.com/4paradigm/OpenMLDB/blob/06255ca9620adf0e0033411984081dab6906a79f/src/sdk/sql_cluster_router.cc#L2308

When we do `execute("sql1;sql2")`, only sql1 will be executed, and no warning about sql2.

We should support execute multi sql in `ExecuteSQL`

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Der Issue verweist auf Zeile 2308 in sql_cluster_router.cc, wo ExecuteSQL SQL-Zeichenfolgen verarbeitet. Untersuche zunächst diese Funktion, um zu sehen, wie sie SQL parst und ausführt. Suche nach vorhandener Parsing-Logik oder einer Aufteilung an Semikolons. Prüfe, ob es Tests für ExecuteSQL gibt, um das erwartete Verhalten zu verstehen. 'Done' bedeutet, dass die Funktion alle Anweisungen in einer durch Semikolons getrennten Zeichenfolge ausführt und möglicherweise Warnungen oder Fehler für jede zurückgibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
backend, databases
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.