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

C-Binding for contantine library #195

Open
matkt opened this issue Jun 3, 2024 · 4 comments
Open

C-Binding for contantine library #195

matkt opened this issue Jun 3, 2024 · 4 comments
Assignees
Labels
good first issue Good for newcomers mainnet

Comments

@matkt
Copy link
Contributor

matkt commented Jun 3, 2024

create C binding for verkle in https://github.com/mratsim/constantine and try to integrate it into Besu.
This library seems to be promising regarding performance they have. We also want to do some benchmark between rust-verkle and constantine when it will be integrated into besu.

update : this C-binding should be implemented in https://github.com/hyperledger/besu-native (see the rust-verkle implementation here https://github.com/hyperledger/besu-native/tree/main/ipa-multipoint)

when it will be implemented in besu-native we need to integrate it into besu (see https://github.com/hyperledger/besu/tree/verkle/ethereum/verkletrie/src/main/java/org/hyperledger/besu/ethereum/verkletrie)

tag @agnxsh

@matkt matkt added good first issue Good for newcomers mainnet labels Jun 3, 2024
@NickSneo
Copy link
Contributor

Hey @matkt can I pick this ticket?

@garyschulte
Copy link
Contributor

This should probably be in besu-native repo, since what we are looking for is a jni implementation of constantine for verkle.

@NickSneo it would be helpful to coordinate on discord

@matkt
Copy link
Contributor Author

matkt commented Jun 18, 2024

@NickSneo could you ask your question again in the besu-verkle channel in the hyperledger besu discord ? like that we will be able to coordinate https://discord.com/channels/905194001349627914/1207729310534737920

I also updated the description in order to talk about besu native

@macfarla
Copy link
Contributor

I think we can transfer this issue to besu-native

PR is already drafted

@macfarla macfarla transferred this issue from hyperledger/besu Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers mainnet
Projects
None yet
Development

No branches or pull requests

4 participants