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

Hendelse filter backend opensearch #1807

Merged
merged 11 commits into from
Dec 4, 2024
Merged

Conversation

slovrid
Copy link
Contributor

@slovrid slovrid commented Dec 3, 2024

Describe your changes

Denne PR-en opprettar eit nytt felt i OpenSearch kalt "utgatt_varsel" og sikrar at vi oppdaterer dette med data vi har lagra ifm. nytt hendelsesfilter-topic, både ved enkelt-/hovedindeksering og ved behandling av meldingar.

Ein detalj som er verdt å merke seg er at vi kan ha lagra fleire hendelsar på brukaren, men det er berre den eldste av desse vi er interessert i ifm. indeksering.

Støtte for filtrering og statustal kjem i eigen PR.

Trello ticket number and link

TC-819

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.

@slovrid slovrid requested a review from a team as a code owner December 3, 2024 15:24
@slovrid
Copy link
Contributor Author

slovrid commented Dec 4, 2024

Note to self: "eldste" er tvetydig

@slovrid
Copy link
Contributor Author

slovrid commented Dec 4, 2024

Ny note to self: pga. nylig identifisert edge-case er det lurast å alltid oppdatere OpenSearch med resultatet fra getEldste.

Edge-caset er: dersom det kjem ei hending som pdd. ligg lagra i DB og er den eldste men oppdateringa gjer at den ikkje lenger er eldst, så kan vi ikkje ha ein sjekk som er if (eldsteHendelse.id == hendelse.id) { //oppdater opensearch }. TL;DR: vi bør alltid oppdatere opensearch for å vere på den trygge sida.

Copy link

sonarqubecloud bot commented Dec 4, 2024

@slovrid slovrid merged commit d3c702b into dev Dec 4, 2024
11 checks passed
@slovrid slovrid deleted the hendelse-filter-backend-opensearch branch December 4, 2024 11:27
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