feat(vortex-bench): support vortex native geo types into SpatialBench, and wire into benchmark orchestrator#8623
Conversation
Signed-off-by: Nemo Yu <zyu379@wisc.edu>
Signed-off-by: Nemo Yu <zyu379@wisc.edu>
Merging this PR will degrade performance by 11.93%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | bitwise_not_vortex_buffer_mut[128] |
215.3 ns | 244.4 ns | -11.93% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing nemo/duckdb-native-geometry (69a4f90) with nemo/geo-multipolygin (a3c046b)
Footnotes
-
4 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. ↩
Summary
Wires vortex geo native format into SpatialBench, and wire into the
vx-bench/bench-orchestratorpipeline so it can be run end-to-end like the other benchmarks.Running command:
Limitation
BENCHMARK_ENGINES = { SPATIALBENCH: {DUCKDB} }.Performance
SF=1.0
SF=10
Takeaways: for now we do not have only pushdown to execute predicate in vortex kernel, so geo native types pay the tax of converting to geo-binary (talk to duckdb) but no gains, so in SF=10 vortex-native is less efficient compared with vortex-wkb.