[Bug]: Support ALTER VIEW ... REFRESH syntax added in Trino 477
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- sql, typescript
- Área
- databases
Línea de trabajo
Abre TrinoSql.g4 y lee la regla statement, además de las definiciones existentes de view-alteration y de palabras clave. Añade compatibilidad con la forma ALTER VIEW view_name REFRESH y, después, verifica que el parser de Trino acepte la instrucción sin un error de sintaxis.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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+).
- Lenguaje dominante
- TypeScript
- Estrellas
- 393
- Forks
- 108
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de DTStack/dt-sql-parser
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
DTStack/dt-sql-parser#490 · 1 comentario ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
DTStack/dt-sql-parser#489 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
DTStack/dt-sql-parser#488 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
DTStack/dt-sql-parser#463 ·
-
enhancement improvement
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
DTStack/dt-sql-parser#445 · 2 comentarios ·
Todos los issues de DTStack/dt-sql-parser
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
aiko-chan-ai/DiscordBotClient#380 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
vercel/ai-elements#507 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 Medio día Aptitud para principiantes 84/100
anaclumos/qa-interns#148 · 1 comentario ·
Los mantenedores suelen responder en 1 día