ClickHouse: tuple element access followed by a field changes meaning after round-trip
Maintainer thường phản hồi trong vòng 2 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 78/100
Hướng nghiên cứu
Bắt đầu với triển khai Display của dialect ClickHouse và target fuzz_parse_roundtrip, sử dụng reproduction SELECT t.1.name làm trường hợp khởi đầu. So sánh SQL được in ra với chuỗi truy cập có khoảng trắng như mong đợi và xác minh rằng việc parse lại tạo ra cùng một AST; src/Parsers/Lexer.cpp cung cấp ngữ cảnh để phân giải sự mơ hồ của lexer.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Description
With ClickHouseDialect, a tuple element access followed by a named field prints without spaces around the second dot. The printed SQL parses back as a different query, with no error.
Reproduction
use sqlparser::dialect::ClickHouseDialect;
use sqlparser::parser::Parser;
let sql = "SELECT t.1.name";
let ast = Parser::parse_sql(&ClickHouseDialect {}, sql).unwrap();
let printed = ast[0].to_string();
println!("{printed}"); // SELECT t . 1.name
let reparsed = Parser::parse_sql(&ClickHouseDialect {}, &printed).unwrap();
println!("{}", reparsed[0]); // SELECT t . 1. AS name
| Step | Result |
|---|---|
| Input | SELECT t.1.name |
| Parsed | CompoundFieldAccess { root: t, access_chain: [Dot(1), Dot(name)] } (correct) |
| Printed | SELECT t . 1.name |
| Re-parsed | SELECT t . 1. AS name: the number 1. with alias name |
Expected
The printed output keeps the access chain, e.g. SELECT t . 1 . name, and parses back to the same AST.
Context
ClickHouse's lexer tokenizes t.1.name as t . 1 . name (see the "chained tuple access" disambiguation in src/Parsers/Lexer.cpp), so the initial parse is correct and only Display is wrong.
Follow-up to #2574 (tuple element access). Found with the fuzz_parse_roundtrip fuzz target.
- Ngôn ngữ chính
- Rust
- Star
- 3.5k
- Fork
- 774
- Merge trung bình
- 3 ngày 23 giờ
- Pull request đã merge (30 ngày)
- 53
Chuẩn bị môi trường
Chúng tôi chưa kiểm tra các tệp thiết lập môi trường của dự án này. Hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của apache/datafusion-sqlparser-rs
-
ClickHouse
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
apache/datafusion-sqlparser-rs#2597 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Hive
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
apache/datafusion-sqlparser-rs#2589 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
apache/datafusion-sqlparser-rs#2461 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 73/100
apache/datafusion-sqlparser-rs#2407 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Add SQL/PGQ keywordsĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
apache/datafusion-sqlparser-rs#2393 ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của apache/datafusion-sqlparser-rs
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 3 ngày
-
state:triage-needed
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Automattic/harper#4503 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
tobi/walgit#84 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày