Eventual-Inc/Daft - CodSpeed

Daft

Blog Docs Changelog

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

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

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

0%

feat(metrics): add node_plan_id for distributed stats attribution

last run

1 hour ago

0%

feat: implement ray-based pre-shuffle-merge for flight shuffle

last run

2 hours ago

0%

© 2026 CodSpeed Technology

Home Terms Privacy Docs