duckdb: vortex_version() macro - #9787
Conversation
Merging this PR will improve performance by 13.45%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | Simulation | allocate_drop_arrow[0] |
456.9 ns | 402.7 ns | +13.45% |
Tip
Curious why performance improved? Comment @codspeedbot explain why performance improved on this PR, or directly use the CodSpeed MCP with your agent.
Comparing myrrc/duckdb-vortex-versoin (a0565ea) with develop (2a5b1a7)
Footnotes
-
218 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
joseph-isaacs
left a comment
There was a problem hiding this comment.
This is not the vortex version.
It might be a good idea to add a enabled editions func for the writer?
Signed-off-by: Mikhail Kot <mikhail@spiraldb.com>
294c384 to
8da8c7b
Compare
|
CI changes in duckdb-vortex to come after we merge this. |
Add vortex_version() duckdb function macro returning Vortex tag this was built from