Support for extensive `COUNT(*)`
Open
- Dominant language
- Rust
- Stars
- 1.6k
- Forks
- 145
- PR merge metrics
- No merged PRs in 30d
Description
Currently, supporting COUNT(*) as followed
```rust
Ok(Expression::Literal(Field::Null))
```
because we'll also have to think about how to count nulls, creating this issue to get back on this
Contributor guide
Research direction
The issue only identifies the Rust fallback expression for COUNT(*); no file, test, or entry point is named. First locate the COUNT(*) handling and clarify the intended null-counting semantics, then define completion as correctly supporting COUNT(*) with tests for the relevant cases.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust, sql
- Domain
- databases
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100