MySQL: compound time units (`YEAR_MONTH`, `DAY_HOUR`...) fail in `EXTRACT` and `INTERVAL`

未关闭
  • 哪里不对？
#2,693 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

  • #2700 来自 @princejain756 —— 已关闭，未合并

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
35/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
停滞
技术栈
rust, sql
领域
backend, databases

调研方向

Start with the MySQL dialect handling for EXTRACT date/time fields and INTERVAL units, then inspect the parser tests for those expressions. Add coverage for the 11 listed compound units in both contexts; done means the supplied examples and tests parse successfully. An open linked pull request (#2700) already addresses this issue.

由索引模型根据 Issue 内容生成。

描述

MySQL

MySQL and MariaDB accept compound time units in EXTRACT and in INTERVAL expressions (DATE_ADD, DATE_SUB, + INTERVAL...). MySqlDialect rejects them, in 0.63.0 and on main:

use sqlparser::dialect::MySqlDialect;
use sqlparser::parser::Parser;

for sql in [
    "SELECT EXTRACT(YEAR_MONTH FROM d) FROM t",
    "SELECT DATE_ADD(d, INTERVAL '1-2' YEAR_MONTH) FROM t",
    "SELECT d + INTERVAL '1 2' DAY_HOUR FROM t",
] {
    println!("{:?}", Parser::parse_sql(&MySqlDialect {}, sql));
}

Err(ParserError("Expected: date/time field, found: YEAR_MONTH at Line: 1, Column: 16"))
Err(ParserError("INTERVAL requires a unit after the literal value at Line: 1, Column: 35"))
Err(ParserError("INTERVAL requires a unit after the literal value at Line: 1, Column: 27"))

The simple units (DAY, MONTH, QUARTER...) parse fine.

The compound units are the same 11 in MySQL and MariaDB:

SECOND_MICROSECOND, MINUTE_MICROSECOND, MINUTE_SECOND, HOUR_MICROSECOND, HOUR_SECOND, HOUR_MINUTE, DAY_MICROSECOND, DAY_SECOND, DAY_MINUTE, DAY_HOUR, YEAR_MONTH.

Thx,

主要语言
Rust
星标
3.5k
派生
805
平均合并
2 天 3 小时
30 天内合并 PR
116

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 有 Pull Request 模板
  • 没有贡献指南

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

apache/datafusion-sqlparser-rs 的其他 Issue

查看 apache/datafusion-sqlparser-rs 的全部 Issue

相似的 Issue

更多 Rust Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。