Repository navigation
refactor(spatial): make bounding-box statistics explicit - #10837
HarukiMoriarty wants to merge 2 commits into
Conversation
56f5964 to
a5e57e7
Compare
Merging this PR will degrade performance by 10.95%2 warnings
Warning Please fix the performance issues or acknowledge them on CodSpeed. Performance Changes
Tip Investigate this regression by commenting Comparing Footnotes
|
a5e57e7 to
2b4dec5
Compare
Remove automatic zone-stat discovery and document explicit file and zone configuration. Let spatial pruning rules match nested field paths so footer AABB statistics can prune table files, and cover opt-in statistics and pruning with integration tests. Signed-off-by: Nemo Yu <[email protected]>
2b4dec5 to
aaea379
Compare
595bf9a to
aaea379
Compare
Signed-off-by: Robert Kruszewski <[email protected]>
Summary
Make spatial bounding-box statistics an explicit writer choice. Initializing
vortex-spatialstill registers geometry types,GeometryAabb, and spatial pruning rules, but no longer adds AABB statistics to zone maps automatically. Writers select file bounds withinclude_nested_statistics().with_file_statistics(...)and zone bounds throughZonedLayoutOptions::aggregate_fns.Also fix spatial pruning for table files: the existing rules only recognized geometry columns at the expression root, so predicates such as
ST_Intersects(geom, constant)could not use footer bounds for a struct field. The shared matcher now accepts direct and nested field paths while retaining the column-versus-literal restriction.Changes
zone_stat_defaulthooks and registry discovery fromvortex-arrayand the zoned writer. Built-in zone defaults remain unchanged.ST_Intersectsand orderedST_Distancefilters use the spatial rules; registering another compute function does not automatically enable pruning for it.Validation
git diff --cached --checkpassed.developat2e1b05b84.