parse_drop silently discards TEMPORARY and PERSISTENT for most object types
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
Start in src/parser/mod.rs at Parser::parse_drop, where TEMPORARY and PERSISTENT are consumed before the object type is read. Use the existing Statement::Drop and DROP SECRET branches, which keep these flags, as the model. Done means DROP TEMPORARY FUNCTION keeps its flag in a dialect that supports it, and the other object types reject the keyword with a parse error. Check the dialect tests for MySQL, Generic and DuckDB.
由索引模型根据 Issue 内容生成。
描述
parse_drop consumes TEMPORARY (MySQL, Generic, DuckDB) and PERSISTENT (DuckDB) before it reads the object type. Only Statement::Drop and DROP SECRET keep these flags. Every other branch drops them, so the statement parses without error but loses the keyword when printed:
| Input | Dialect | Output |
|---|---|---|
DROP TEMPORARY FUNCTION f |
Generic | DROP FUNCTION f |
DROP TEMPORARY TRIGGER tr ON t |
Generic | DROP TRIGGER tr ON t |
DROP TEMPORARY PROCEDURE p |
Generic | DROP PROCEDURE p |
DROP PERSISTENT TABLE t |
DuckDB | DROP TABLE t |
The same applies to POLICY, CONNECTOR, DOMAIN, EXTENSION and OPERATOR.
In most cases the input isn't valid SQL anyway, so it should fail instead of being accepted. DROP TEMPORARY FUNCTION is real syntax in Spark/Databricks and Hive, so there the parser drops meaningful information.
Expected
Reject TEMPORARY/PERSISTENT for object types that don't support them, and keep the flag where a dialect does support it, e.g. DROP TEMPORARY FUNCTION.
Code
src/parser/mod.rs, Parser::parse_drop
Spotted while reviewing #2434
- 主要语言
- Rust
- 星标
- 3.5k
- 派生
- 805
- 平均合并
- 2 天 3 小时
- 30 天内合并 PR
- 116
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 没有贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
apache/datafusion-sqlparser-rs 的其他 Issue
-
Redshift
难度 2/5 1-3 小时 新手友好度 82/100
apache/datafusion-sqlparser-rs#2959 ·
维护者通常 1 天内回复
-
Redshift
难度 2/5 1-3 小时 新手友好度 68/100
apache/datafusion-sqlparser-rs#2958 ·
维护者通常 1 天内回复
-
PostgreSQL
难度 2/5 1-3 小时 新手友好度 76/100
apache/datafusion-sqlparser-rs#2862 ·
维护者通常 1 天内回复
-
PostgreSQL
难度 2/5 1-3 小时 新手友好度 75/100
apache/datafusion-sqlparser-rs#2861 ·
维护者通常 1 天内回复
-
PostgreSQL
难度 2/5 1-3 小时 新手友好度 77/100
apache/datafusion-sqlparser-rs#2858 ·
维护者通常 1 天内回复
查看 apache/datafusion-sqlparser-rs 的全部 Issue
相似的 Issue
-
area:schema documentation
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 82/100
ElectronicWave/Elemental#252 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 78/100
dekobon/big-code-analysis#1887 ·
维护者通常 1 天内回复
-
Shell中文符号输入异常未关闭bug
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 82/100
维护者通常 1 天内回复