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

feat: optimize tick liquidity fetching through indexer #469

Merged
merged 23 commits into from
Oct 20, 2023
Merged

feat: optimize tick liquidity fetching through indexer #469

merged 23 commits into from
Oct 20, 2023

Conversation

dib542
Copy link
Collaborator

@dib542 dib542 commented Oct 13, 2023 •
edited
Loading

This PR uses the new pair liquidity long-polling route from the indexer to request real-time up-to-date liquidity data from the chain

This PR:

  • ensures the liquidity data for a token pair is from the same chain height when displayed to end users
  • updates with partial liquidity responses
    • which reduces requires data download size by orders of magnitude for each new block update
    • these partial responses are not diffs, they represent new reserves at each given tickIndex of a token pair, so the updates do not lose resolution due to cumulative math operations on rounded numbers
  • updates as soon as new data is available: using new long-polling mechanism

dib542 added 17 commits October 13, 2023 00:57
@dib542
feat: switch tick liquidity data source from chain RPC to Indexer
2d17427
@dib542
refactor: refine type conditions for if blocks
e4edcff
@dib542
feat: allow both height indicators to be used
546459b
@dib542
feat: allow tick liquidity updates to be applied over known state
f181ba0
@dib542
feat: switch block range logic from headers to query params / res body
ba9c115
@dib542
feat: add long-running request cancellation
5822de9
@dib542
feat: prevent excessive cancelled progressive update requests on mount
46dfd58
@dib542
feat: add custom control for efficiently caching liquidity state
1b1bdb3
@dib542
feat: ensure cache persists from time last known to be fresh (unmount)
ab50f47
@dib542
refactor: reduce cache write by only writing when unmounting
425cd55 
    - the cached data should only be needed after unmounting
@dib542
refactor: reduce cache writes by avoiding writing what was just read
37fdedb
@dib542
fix: cached results were unexpected without full URL path in cache
fc4362a
@dib542
refactor: abstract out fetch API cache check
3ff79ae
@dib542
fix: calculate estimated cached update total pagination
6820492
@dib542
fix: shift computationally intensive steps for caching out of unmount
a70dad6
@dib542
docs: improve comments
749f788
@dib542
feat: switch from querying tick liquidity halves to pair liquidity
235ac3b 
    - this helps ensure that the liquidity data is always from the
      same chain height, because the data is from the same request
@dib542 dib542 linked an issue Oct 13, 2023 that may be closed by this pull request
Optimize tick liquidity fetching through indexer #452
Closed
@github-actions
Copy link

Preview URL: https://7e4af258-5586221d--duality-xyz.netlify.app
Logs: https://app.netlify.com/sites/duality-xyz/deploys/653296cf033eb119c069f40f

@dib542 dib542 merged commit de75c25 into main Oct 20, 2023
2 checks passed
@dib542 dib542 deleted the 452-optimize-tick-liquidity-fetching-through-indexer branch October 20, 2023 15:53
dib542 pushed a commit that referenced this pull request Oct 20, 2023
@semantic-release-bot
chore(release): 0.3.33 [skip ci]
8cac588 
## [0.3.33](v0.3.32...v0.3.33) (2023-10-20)

### Features

* optimize tick liquidity fetching through indexer ([#469](#469)) ([de75c25](de75c25))
@dib542
Copy link
Collaborator Author

dib542 commented Oct 20, 2023

🎉 This PR is included in version 0.3.33 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
deploy preview released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Optimize tick liquidity fetching through indexer
1 participant
@dib542