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-2655] Migrate code from mx-exchange-analytics-indexer-repo #1515

Open
wants to merge 3 commits into
base: feat/analytics-indexer
Choose a base branch
from
Open

[SERVICES-2655] Migrate code from mx-exchange-analytics-indexer-repo #1515

wants to merge 3 commits into from

Conversation

mad2sm0key
Copy link
Contributor

@mad2sm0key mad2sm0key commented Oct 23, 2024 •
edited
Loading

Reasoning

  • provide a way to recompute historical analytics persisted in TimescaleDB using data from Elastic Search

Proposed Changes

  • compute historical prices and locked values for tokens and pairs by processing events generated by SCs. Pairs state is maintained in memory at processing time, and USD values are computed using the data from the WEGLD/USDC pair
  • by keeping pair state in memory, the computation can be performed in a synchronous manner. Getting the initial state, fetching the events from ES and persisting computed data are still performed asynchronously

How to test

  • N/A

@mad2sm0key
SERVICES-2655: migrate and refactor code from analytics-indexer repo
0081f4e 
- rename services and entities
- use ES events index when querying
- make all code relying on in-memory data run in a synchronous manner
@mad2sm0key
SERVICES-2655: enable ssl for timescaledb module
d91fe0b
@mad2sm0key
SERVICES-2655: make price discover indexing synchronous
9f15b96 
- add 1 day caching to values fetched durring init operations
@mad2sm0key mad2sm0key marked this pull request as ready for review October 24, 2024 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gabrielmatei gabrielmatei gabrielmatei 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 @gabrielmatei