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

pindexer: Implement indexing of arb executions #4754

Merged
merged 5 commits into from
Jul 24, 2024
Merged

Conversation

cronokirby
Copy link
Contributor

Merge #4753 first.

Describe your changes

This implements indexing for arb executions in pindexer.

Issue ticket number and link

Closes #4736.

Checklist before requesting a review

  • If this code contains consensus-breaking changes, I have added the "consensus-breaking" label. Otherwise, I declare my belief that there are not consensus-breaking changes, for the following reason:

    indexing changes only

@vacekj vacekj changed the title pindexer: Implement indexing of arb executions pindexer: Implement indexing of arb executions and lp updates Jul 24, 2024
Base automatically changed from pindexer-dex-scaffolding to main July 24, 2024 17:19
@cronokirby cronokirby changed the title pindexer: Implement indexing of arb executions and lp updates pindexer: Implement indexing of arb executions Jul 24, 2024
@conorsch conorsch self-requested a review July 24, 2024 17:23
Copy link
Contributor

@conorsch conorsch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#4753 has landed; approving this optimistically, to unblock ongoing pindexer work.

Copy link
Contributor

@aubrika aubrika left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aubrika aubrika merged commit 53b8338 into main Jul 24, 2024
13 checks passed
@aubrika aubrika deleted the 4636-pindexer-arbs branch July 24, 2024 19:32
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.

pindexer: dex: implement /api/arbs related events and tables
4 participants