Commit 65f337d
authored
perf: Use bulk-NULL builder in
## Which issue does this PR close?
- Closes #21844.
## Rationale for this change
`uuid()` never emits NULLs, so this is a straightforward win.
Benchmarks:
```
uuid (1024 rows): 21.618 µs → 20.133 µs, −6.90% (`p < 0.05`)
```
## What changes are included in this PR?
* `GenericStringBuilder` -> `GenericStringArrayBuilder` in `uuid()`
implementation
## Are these changes tested?
Yes.
## Are there any user-facing changes?
No.uuid (#21845)1 parent 6f1040b commit 65f337d
1 file changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
20 | | - | |
21 | 20 | | |
22 | 21 | | |
23 | 22 | | |
24 | 23 | | |
25 | 24 | | |
| 25 | + | |
26 | 26 | | |
27 | 27 | | |
28 | 28 | | |
| |||
87 | 87 | | |
88 | 88 | | |
89 | 89 | | |
90 | | - | |
| 90 | + | |
91 | 91 | | |
92 | 92 | | |
93 | 93 | | |
| |||
101 | 101 | | |
102 | 102 | | |
103 | 103 | | |
104 | | - | |
| 104 | + | |
105 | 105 | | |
106 | 106 | | |
107 | 107 | | |
| |||
0 commit comments