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.

Sign up for GitHub

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

Jump to bottom

[SERVICES-2603] Refactor ES integration to use events index #1506

Open
wants to merge 7 commits into
base: development
Choose a base branch
from
Open

[SERVICES-2603] Refactor ES integration to use events index #1506

wants to merge 7 commits into from

Conversation

mad2sm0key
Copy link
Contributor

@mad2sm0key mad2sm0key commented Oct 11, 2024 •
edited
Loading

Reasoning

  • replace queries on the ES logs index with queries on the new events index providing flat SC events
  • use sdk-nestjs-elastic package for all Elastic Search queries

Proposed Changes

  • refactor token swap count compute method to use events index
  • refactor pair swap count compute method to use events index
  • refactor governance user vote compute method to use events index
  • refactor Logs processor cron job to use events index
  • remove Elastic Service helper

How to test

  • N/A

@mad2sm0key mad2sm0key marked this pull request as ready for review October 11, 2024 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cfaur09 cfaur09 cfaur09 approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mad2sm0key @cfaur09