apache / apache/datafusion-sqlparser-rs

Improving support for `CREATE PROCEDURE`

Offen
#1,902 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Rust
Sterne
3.5k
Forks
772
Ø Merge
4 T. 9 Std.
Gemergte PRs (30 T.)
17

Beschreibung

sqlparser-rs currently doesn't have great support for `CREATE PROCEDURE` statements, leading to parser errors when I pass in valid SQL. I'd like to work on improving this (I've already started on [my fork](https://github.com/ZacJW/sqlparser-rs/tree/dev)).

What's the project's ideal vision for how this would be handled? Stored procedures are the kind of thing that can be very RDBMS specific. Do you just want the AST to cover what's in the standard, but the dialects to accept but discard additional options?

The first fix which will be ready to go once I've written tests is #1901, support for what PostgreSQL calls argmode and the standard calls parameter mode.

## PRs

- #1901
- #1903

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, die bestehende Arbeit im dev-Branch des Forks des Autors sowie die verlinkten Issues und PRs #1901 und #1903 zu prüfen. Kläre das beabsichtigte Verhalten des Projekts für AST und Dialekte bei CREATE PROCEDURE und füge anschließend Tests hinzu, die die vereinbarte Unterstützung abdecken, sodass gültiges SQL wie erwartet geparst wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
rust, sql
Bereich
compilers, databases
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

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