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

dev: batch RPC requests linked to storage queries #940

Open
greged93 opened this issue Apr 9, 2024 · 2 comments
Open

dev: batch RPC requests linked to storage queries #940

greged93 opened this issue Apr 9, 2024 · 2 comments
Labels
blocked Blocked right now for external reasons enhancement Enhancement of the code, not introducing new features. timebox-1day Exploratory issue: Do not spend more than a day on it to assess doability.

Comments

@greged93
Copy link
Collaborator

greged93 commented Apr 9, 2024 •
edited
Loading

Describe the enhancement request
The json RPC standard allows to send a batch of request (see link and link). This would allow us to batch the requests to the node for storage requests in the context of a call for example.

Describe Preferred Solution

  • Verify batch request is available on Juno.
  • Verify it is possible to build and send a batch request in Rust.
  • Explore the different hotpoints on our code where batching would improve the latency.
  • Implement a wrapper type in order to perform batch requests to a starknet json rpc.
  • Batch calls where possible

Links
@greged93 greged93 added enhancement Enhancement of the code, not introducing new features. timebox-1day Exploratory issue: Do not spend more than a day on it to assess doability. labels Apr 9, 2024
@hhamud
Copy link
Contributor

hhamud commented Jun 28, 2024

can I take this one? @greged93 @tcoratger

@greged93 greged93 added the blocked Blocked right now for external reasons label Jun 28, 2024
@greged93
Copy link
Collaborator Author

greged93 commented Jun 28, 2024 •
edited
Loading

hey @hhamud this is quite a complicated issue so probably better not if you don't have a lot of experience with our stack. This is actually still blocked for now anyway, waiting for this xJonathanLEI/starknet-rs#600 PR on starknet-rs.

anukkrit149 pushed a commit to karnotxyz/kakarot-rpc that referenced this issue Aug 9, 2024
@ClementWalter
Add EVM serialization (kkrt-labs#995)
b0c62ee 
Time spent on this PR: 0.5

## Pull request type

Please check the type of change your PR introduces:

- [ ] Bugfix
- [x] Feature
- [ ] Code style update (formatting, renaming)
- [ ] Refactoring (no functional changes, no api changes)
- [ ] Build related changes
- [ ] Documentation content changes
- [ ] Other (please describe):

## What is the current behavior?

Not possible to return `model.EVM` from a test

Resolves kkrt-labs#940 

## What is the new behavior?

Serialize model.EVM + included struct + fix when returning struct and
not struct*

<!-- Reviewable:start -->
- - -
This change is [<img src="https://reviewable.io/review_button.svg"
height="34" align="absmiddle"
alt="Reviewable"/>](https://reviewable.io/reviews/kkrt-labs/kakarot/995)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
blocked Blocked right now for external reasons enhancement Enhancement of the code, not introducing new features. timebox-1day Exploratory issue: Do not spend more than a day on it to assess doability.
Projects
Kakarot's Project Management
Status: 🆕 Backlog
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hhamud @greged93