Eventual-Inc
Daft
Blog
Docs
Changelog
Blog
Docs
Changelog
Overview
Branches
Benchmarks
Runs
Performance History
Latest Results
removing pinning
chris/test-fixes-3.10
49 minutes ago
revert support for variable_shape_tensor
canonical-tensor
1 hour ago
fixes
chris/metrics-node-plan-id-and-base-counters
1 hour ago
skip if conda version not present
chris/test-fixes-3.10
1 hour ago
don't use threshold in distributed operator
oh/flight_shuffle_dev
2 hours ago
fixes
chris/test-fixes-3.10
3 hours ago
fix flight pre shuffle merge implementation
oh/flight_shuffle_dev
3 hours ago
fix(sql): reject duplicate CTE names in WITH clause (#6331) ## Changes Made Duplicate CTE names in a single `WITH` clause were silently shadowed — `HashMap::insert` in `plan_ctes()` overwrote the first definition without any error, returning incorrect query results. **Root cause:** `Bindings::insert()` delegates to `HashMap::insert()`, which silently replaces existing keys. The return value (`Option<T>`) was discarded, so the second CTE definition quietly replaced the first. **Fix:** Added a pre-insert check in `plan_ctes()` (`src/daft-sql/src/planner.rs`). If `bound_ctes` already contains the CTE name, an `InvalidOperation` error is raised before insertion. ```sql -- Previously: silently returned df2's result -- Now: raises "Duplicate CTE name: 'cte'" WITH cte AS (SELECT * FROM df1), cte AS (SELECT * FROM df2) SELECT * FROM cte ``` **Tests:** Added 11 test cases in `tests/sql/test_cte_duplicate_names.py` covering: - Duplicate names raise error with correct message - Single / multiple distinct CTEs still work (regression) - Case sensitivity (`cte` vs `CTE` are distinct) - Duplicates among 3+ CTEs, with different queries / column aliases - CTE name reuse across separate `daft.sql()` calls ## Related Issues Closes #6325
main
4 hours ago
Active Branches
feat(datatypes): canocial tensor representation for daft tensors
last run
1 hour ago
#6342
CodSpeed Performance Gauge
0%
feat(metrics): add node_plan_id for distributed stats attribution
last run
1 hour ago
#6332
CodSpeed Performance Gauge
0%
feat: implement ray-based pre-shuffle-merge for flight shuffle
last run
2 hours ago
#6306
CodSpeed Performance Gauge
0%
© 2026 CodSpeed Technology
Home
Terms
Privacy
Docs