Kontinuation opened a new pull request, #635:
URL: https://github.com/apache/sedona-db/pull/635
## Summary
- Remove the `barrier()` UDF function, which was an optimization barrier
workaround for KNN joins. It had no external consumers (no Python bindings, no
integration tests, no doc references) and is no longer needed since KNN joins
inherently block filter pushdown through extension node semantics.
- Replace the "Optimization Barrier" docs section in `sql-joins.md` with a
"KNN Join Caveats" section that accurately documents:
- **No Filter Pushdown**: KNN joins do not push filters into input tables;
all predicates are post-filters. Notes that query-side pushdown is a valid
future optimization.
- **ST_KNN Predicate Precedence**: `ST_KNN` is always extracted first when
combined with other predicates via `AND`; equivalent examples shown for `ON ...
AND` vs `WHERE` placement.
## Changes
- `docs/reference/sql-joins.md` — Replaced "Optimization Barrier" section
with "KNN Join Caveats"
- `rust/sedona-functions/src/barrier.rs` — Deleted (649 lines)
- `rust/sedona-functions/src/lib.rs` — Removed `mod barrier;`
- `rust/sedona-functions/src/register.rs` — Removed `barrier_udf`
registration
## Testing
- `cargo test -p sedona-functions` — 344 tests pass
- `cargo test -p sedona-spatial-join` — 171 tests pass
- All doc claims verified experimentally via Python SedonaContext
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
