Skip to content

[ENG-9696] timeseries search#94

Merged
aaxelb merged 11 commits into
CenterForOpenScience:mainfrom
aaxelb:9696-timeseries-search
Mar 24, 2026
Merged

[ENG-9696] timeseries search#94
aaxelb merged 11 commits into
CenterForOpenScience:mainfrom
aaxelb:9696-timeseries-search

Conversation

@aaxelb

@aaxelb aaxelb commented Mar 17, 2026

Copy link
Copy Markdown
Collaborator
  • fix searching across timeseries indexes in base class elasticsearch_metrics.imps.elastic8.DjelmeRecordtype
    • FooRecord.search() now returns a Search that queries all indexes for the type
    • add FooRecord.search_timeseries_range(from_when, until_when), which returns a Search filtered to the given range (either dates or a "timeparts" tuple of ints), using index-name wildcard to query (roughly) only the relevant timeseries indexes
  • fix bug handling DJELME_BACKENDS setting
  • add/improve tests, docs, annotations...

@aaxelb aaxelb force-pushed the 9696-timeseries-search branch from cb9b646 to 96955a2 Compare March 17, 2026 20:37
@aaxelb aaxelb force-pushed the 9696-timeseries-search branch from 96955a2 to d667702 Compare March 17, 2026 20:37
@aaxelb aaxelb changed the title [wip][ENG-9696] timeseries search [ENG-9696] timeseries search Mar 19, 2026
@aaxelb aaxelb marked this pull request as ready for review March 19, 2026 20:04
@aaxelb aaxelb requested a review from felliott March 19, 2026 20:04

@felliott felliott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

@aaxelb aaxelb merged commit 28a048c into CenterForOpenScience:main Mar 24, 2026
32 checks passed
@aaxelb aaxelb deleted the 9696-timeseries-search branch March 24, 2026 14:30
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.

2 participants