Skip to content

feat(bindings): temporal time-domain predicates (topological + position)#42

Closed
estebanzimanyi wants to merge 1 commit intofeat/tnumber-numeric-functionsfrom
feat/temporal-time-domain-predicates
Closed

feat(bindings): temporal time-domain predicates (topological + position)#42
estebanzimanyi wants to merge 1 commit intofeat/tnumber-numeric-functionsfrom
feat/temporal-time-domain-predicates

Conversation

@estebanzimanyi
Copy link
Copy Markdown
Member

Summary

Combines temporal × {Temporal, tstzspan, tstzspanset, timestamptz} time-domain predicates: 96 topological registrations + 32 time-position registrations. 214 lines, all routed through the same TempTempBoolPred / TempBoxBoolPred<Span> / BoxTempBoolPred<Span> helpers.

This PR consolidates two previously stacked PRs that both targeted the same templated dispatch surface (Temporal × span/spanset) but with different op-sets — splitting added review burden without independent decision points.

Coverage

Op set Surface
Topological (@>, <@, &&, `- -`)
Position (before, after, overbefore, overafter) Same surface; named functions because DuckDB's parser does not accept the MobilityDB <<#, #>>, &<#, #&> operator forms

Test plan

  • All 128 registrations load
  • Smoke tests for representative entries in each surface cell
  • Full local suite passes net of pre-existing TZ-environment failures

Replaces

@estebanzimanyi estebanzimanyi force-pushed the feat/temporal-time-domain-predicates branch from c4513da to cf03319 Compare April 25, 2026 15:16
@estebanzimanyi estebanzimanyi changed the base branch from feat/tnumber-distance to feat/tnumber-numeric-functions April 25, 2026 15:17
@estebanzimanyi estebanzimanyi force-pushed the feat/tnumber-numeric-functions branch from 672f31b to 02fe2f2 Compare April 27, 2026 12:55
Combines temporal × {Temporal, tstzspan, tstzspanset, timestamptz} time-
domain predicates:

- Topological: @>, <@, &&, -|- (96 registrations across all temporal
  base types and time-shape pairs)
- Time-position: before, after, overbefore, overafter (named functions)

All wrappers route through TempTempBoolPred / TempBoxBoolPred<Span> /
BoxTempBoolPred<Span> with single-line lambdas dispatching to MEOS
*_temporal_temporal / *_tstzspan_temporal / *_temporal_tstzspan etc.
@estebanzimanyi estebanzimanyi force-pushed the feat/temporal-time-domain-predicates branch from cf03319 to 152ec24 Compare April 27, 2026 12:55
@estebanzimanyi
Copy link
Copy Markdown
Member Author

Superseded — content landed in main as commit 0d949bb when PR #39 was merged with base=main (which brought the entire stack along). This PR's branch is now behind main and merging it would delete code, so closing.

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