Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Support for
ST_DWITHIN
was originally developed as part of the work onST_DISTANCE
for a few reasons:_search
query forgeo_distance
. Conceptuallygeo_distance
is much closer toST_DWITHIN
than it is toST_DISTANCE
, because the distance value is strongly coupled to the function (as an argument).However it was argued that we do not really need to implement
ST_DWITHIN
for a few reasons:ST_DISTANCE() < X
to a call toST_INTERSECTS(Circle(X))
, which already supports lucene pushdown.ST_DISTANCE
itself, and do not really need a second function.This PR captures the work done for
ST_DWITHIN
if we decide to implement it later.