Skip to content

Conversation

jianoaix
Copy link
Contributor

@jianoaix jianoaix commented Apr 21, 2025

Why are these changes needed?

Responses have pre-joined both requests and response so the clients won't need to make additional (potentially a lot!) lookups to dataapi.

Checks

  • I've made sure the tests are passing. Note that there might be a few flaky tests, in that case, please comment that they are not relevant.
  • I've checked the new test coverage and the coverage percentage didn't drop.
  • Testing Strategy
    • Unit tests
    • Integration tests
    • This PR is not tested :(

@jianoaix jianoaix requested review from dmanc and ian-shim April 21, 2025 22:50
@jianoaix jianoaix changed the title feat: make dispersal feed based on responses not requests perf: make dispersal feed based on responses not requests Apr 22, 2025
@jianoaix jianoaix merged commit 02c0788 into Layr-Labs:master Apr 22, 2025
16 checks passed
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.

2 participants