Skip to content

Answer the nearest-neighbour walk as the cursor it is - #36

Merged
estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:feat/nn-cursor-surface
Sep 3, 2026
Merged

Answer the nearest-neighbour walk as the cursor it is#36
estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:feat/nn-cursor-surface

Conversation

@estebanzimanyi

Copy link
Copy Markdown
Member

The catalog gives the R-tree and SP-tree kNN cursors classes, so the object
layer carries RTreeNNCursor and SPNNCursor: a cursor is opened from a tree,
closed when the walk ends, and the three entries sit on the cursor rather than
among the tree's twenty methods. 115 classes and 1342 methods against 113 and
1338, 18 deferred against 22.

TBigint.Values reaches the layer with them, its count declared the way its
definition writes it, so every temporal number answers its distinct values as
an array of them.

next stays deferred on both cursors: it answers the id AND the distance
through two out-parameters, and one body writing both is the same shape the
value-and-time splits need.

The catalog gives the R-tree and SP-tree kNN cursors classes, so the object
layer carries `RTreeNNCursor` and `SPNNCursor`: a cursor is opened from a tree,
closed when the walk ends, and the three entries sit on the cursor rather than
among the tree's twenty methods. 115 classes and 1342 methods against 113 and
1338, 18 deferred against 22.

`TBigint.Values` reaches the layer with them, its count declared the way its
definition writes it, so every temporal number answers its distinct values as
an array of them.

`next` stays deferred on both cursors: it answers the id AND the distance
through two out-parameters, and one body writing both is the same shape the
value-and-time splits need.
@estebanzimanyi
estebanzimanyi merged commit 130bd25 into MobilityDB:main Sep 3, 2026
1 check passed
@estebanzimanyi
estebanzimanyi deleted the feat/nn-cursor-surface branch September 3, 2026 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant