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

[DEV-11465] Change spending_by_geography to use Elasticsearch for Subawards #4290

Draft
wants to merge 4 commits into
base: qat
Choose a base branch
from

Conversation

sethstoudenmier
Copy link
Contributor

@sethstoudenmier sethstoudenmier commented Feb 21, 2025

Description:
Updating the /search/spending_by_geography endpoint to use Elasticsearch instead of Postgres.

Technical details:
Changes involve a lot of cleanup of the old Django ORM code and cutting over the Subaward logic to use Elasticsearch. Other changes include many updates to tests so that the Subawards are loaded correctly into Elasticsearch and a minor change to how the New Awards Only logic was previously setup to work when searching for Awards.

Requirements for PR merge:

  1. Unit & integration tests updated
  2. API documentation updated
  3. Necessary PR reviewers:
    • Backend
    • Frontend
    • Operations
    • Domain Expert
  4. Matview impact assessment completed
  5. Frontend impact assessment completed
  6. Data validation completed
  7. Appropriate Operations ticket(s) created
  8. Jira Ticket DEV-11465:
    • Link to this Pull-Request
    • Performance evaluation of affected (API | Script | Download)
    • Before / After data comparison

Area for explaining above N/A when needed:

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