[Bug]: Support ALTER VIEW ... REFRESH syntax added in Trino 477
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- sql, typescript
- Ambito
- databases
Direzione di ricerca
Apri TrinoSql.g4 e leggi la regola statement, insieme alle definizioni esistenti di view-alteration e delle keyword. Aggiungi il supporto per la forma ALTER VIEW view_name REFRESH, quindi verifica che il parser Trino accetti l'istruzione senza errori di sintassi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Version
v4.5.1
Which SQL?
Trino
What happened?
Currently, the Trino SQL parser implementation in this repository is based on an older version of Trino SQL grammar. As specified in the grammar file comment:
* current version 450(https://github.com/DTStack/dt-sql-parser/blob/main/src/grammar/trino/TrinoSql.g4#L18C4-L18C23)
Because of this, the parser lacks support for several new DDL statements introduced in later versions of Trino. Specifically, trying to validate or parse the ALTER VIEW ... REFRESH statement results in a syntax error.
TrinoSql.g4 grammar can be updated to support the ALTER VIEW view_name REFRESH statement.
According to the official Trino 477 Release Notes:
- Add support for ALTER VIEW ... REFRESH. (#25906) (https://trino.io/docs/current/release/release-477.html)
Proposed Grammar Changes (TrinoSql.g4):
- Add a new alternative rule to the
statementblock:
statement
: ...
| KW_ALTER KW_VIEW from=viewRef KW_REFRESH # refreshView
| ...
;
(Alternatively, it can be named alterView if extracted into a separate rule).
- Ensure
REFRESHis registered as a keyword/lexer token if it's not fully global.
Additional context
This feature is highly required for correct parsing and auto-completion validation when working with Hive or Memory connectors in modern Trino clusters (v477+).
- Lingua principale
- TypeScript
- Stelle
- 394
- Fork
- 108
- Merge medio
- 5m
- PR unite (30g)
- 1
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di DTStack/dt-sql-parser
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
DTStack/dt-sql-parser#490 · 1 commento ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
DTStack/dt-sql-parser#489 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
DTStack/dt-sql-parser#488 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
DTStack/dt-sql-parser#463 ·
-
enhancement improvement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
DTStack/dt-sql-parser#445 · 2 commenti ·
Tutte le issue di DTStack/dt-sql-parser
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
safetrustcr/dApp-SafeTrust#426 ·
-
area:workflow bug ready-for-agent
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
fil-donadoni/tolaria#4409 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
Fission-AI/OpenSpec#1960 ·
-
Add dependabot Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
corsairdev/corsair#1764 ·