Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Index Occurrences/Sightings in OpenSearch #1010

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

naknomum
Copy link
Member

@naknomum naknomum commented Feb 4, 2025

Index Sightings (Occurrences) in OpenSearch

PR fixes #911

Based on occurrenceSearch.jsp, added all fields necessary to query on items in each section:

  • Location (lat/lon)
  • Date filters
  • Collected data fields
  • Metadata filters
  • search api has not been altered to support this, as there is no current need; can be added when usage dictates what requirements are

@codecov-commenter
Copy link

codecov-commenter commented Feb 4, 2025

Codecov Report

Attention: Patch coverage is 0% with 67 lines in your changes missing coverage. Please review.

Project coverage is 0.13%. Comparing base (5b1695e) to head (1e8b20e).

Files with missing lines Patch % Lines
src/main/java/org/ecocean/Occurrence.java 0.00% 66 Missing ⚠️
src/main/java/org/ecocean/OpenSearch.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              main   #1010      +/-   ##
==========================================
- Coverage     0.13%   0.13%   -0.01%     
  Complexity      29      29              
==========================================
  Files          580     580              
  Lines        64015   64082      +67     
  Branches     11090   11100      +10     
==========================================
  Hits            89      89              
- Misses       63907   63974      +67     
  Partials        19      19              
Flag Coverage Δ
backend 0.13% <0.00%> (-0.01%) ⬇️
frontend 0.13% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@naknomum naknomum marked this pull request as ready for review February 7, 2025 19:41
@vkirkl vkirkl added this to the 10.7.0 milestone Feb 11, 2025
@naknomum naknomum marked this pull request as draft February 11, 2025 22:34
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.

Sightings indexed by Opensearch
3 participants