vesoft-inc/nebula

Parameter supported in more fields like ones in `INSERT`

Offen

#3.990 geöffnet am 08.03.2022

Auf GitHub ansehenContributor Guide
 (4 Kommentare) (1 Reaktion) (0 zugewiesene Personen)C++ (1.144 Forks)batch import
help wantedtype/enhancement

Repository-Metriken

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

Beschreibung

A user from slack (and also from my side) would consider it's nice to support so ;).

(root@nebula) [basketballplayer]> :param t => "player1000"

(root@nebula) [basketballplayer]> insert vertex player(name,age) values $t:("foo bar", 20);
[ERROR (-1004)]: SyntaxError: Parameter is not supported in vid near `$t'

Contributor Guide

Research-Richtung
Untersuche den Parser Code für INSERT Anweisungen im Nebula Quellcodebaum (wahrscheinlich unter src/parser). Verstehe, wie Parameter (z.B. $t) derzeit in anderen Teilen der Abfrage (wie values) unterstützt werden, aber nicht im VID Feld. Überprüfe die Grammatikregel für INSERT und identifiziere, wo die Parameterunterstützung für die Vertex ID hinzugefügt werden muss. Prüfe vorhandene Kommentare oder Diskussionen im Issue auf zusätzliche Einschränkungen. Schlage eine Änderung vor, um Parameter in der VID Position zuzulassen, und stelle sicher, dass die Typüberprüfung und Fehlermeldungen konsistent behandelt werden.
Tech Stack
cpp
Domain
databasebackend
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.
1-2 Tage
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
C++Graph database concepts
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40