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

dex(router): make Frontier position an append-only log #2670

Merged
merged 1 commit into from
Jun 7, 2023

Conversation

erwanor
Copy link
Member

@erwanor erwanor commented Jun 2, 2023

This was suggested by @hdevalence, presumably to avoid churning over the same positions within a same frontier. I don't fully understand the ramifications of that change yet. I will adjust the PR description once I have worked out the consequences that this change has on the DEX engine.

Co-authored-by: hdevalence <hdevalence@penumbralabs.xyz>
@erwanor erwanor force-pushed the append_only_frontier_positions branch from 555eed3 to b0f8f37 Compare June 2, 2023 23:22
@erwanor erwanor temporarily deployed to smoke-test June 2, 2023 23:22 — with GitHub Actions Inactive
@erwanor erwanor temporarily deployed to smoke-test June 7, 2023 14:54 — with GitHub Actions Inactive
@erwanor erwanor merged commit 3a77a12 into main Jun 7, 2023
@erwanor erwanor deleted the append_only_frontier_positions branch June 7, 2023 15:17
@cratelyn cratelyn added the A-dex Area: Relates to the dex label Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-dex Area: Relates to the dex
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants