-
Notifications
You must be signed in to change notification settings - Fork 18
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
Implement Mempool #29
Comments
See https://github.com/ava-labs/avalanchego/pull/1958/files#diff-fc671b8b3c30328876f4984afaaf43b548dd28cc886934a95a0fbcdb6788f1ae for an example of the new |
I'm interested in contributing (at least have a try) to this ticket. Before I start, I have a question regarding the directory structure for the implementation. In avalanchego, the relevant code is located in I'm asking because I noticed that the I'd like to ensure I'm placing my contributions in the correct directories. |
Thanks for looking into this. In avalanchego historically, each vm has their own mempool implementation, leading to a lot of code duplication. We are working to provide a common library for APIs like mempool (see ava-labs/avalanchego#1958 for an example of a mempool API called I think for now the best thing to do would be to implement a We can put it in https://github.com/ava-labs/avalanche-rs/tree/main/core/network/src under a |
I have some stuff ongoing over here : https://github.com/Sanghren/avalanche-rs/tree/add_gossipable. Not sure about everything I did though. |
That's great. I would open a PR and then we can take a look. It can be a draft PR if not's not fully ready. |
Opened a draft PR here : #45 |
Port the Mempool type and API from avalanchego. See https://pkg.go.dev/github.com/ava-labs/[email protected] for package-level documentation.
The text was updated successfully, but these errors were encountered: