-
Notifications
You must be signed in to change notification settings - Fork 58
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
feat: Integrate Negentropy RBSR library for Waku Sync #2426
Comments
my work on this topic can be seen in this PR #2403 |
After doing some digging wrt issue faced in integrating negentropy and consulting nim team, it has been identified that in order to use the negentropy C++ API directly in nim, it would require nim to be compiled as c++ code instead of C(that is being done currently). This is not recommended by nim team as there are limitations and we may run into issues. Note: If there were no templates used in the C++ code, then we could integrate directly into nim by using extern-C, but since negentropy using templates this would not work. Ref Ref Query and pointers:
Hence as suggested, proceeding with writing a C API wrapper for negentropy which can then be used in the Waku-sync code. |
In order to validate this properly we can use the storenode-messages-counter utility. https://grafana.infra.status.im/d/hmX6pJ_Iz/storenode-message-counter?orgId=1&from=now-7d&to=now We will also need to validate that the |
The only left task: |
Closing issue as done. Any new features/reported issues can be raised separately. |
This issue relates to the implementation of a proposed underlying protocol for Waku Sync as set out in waku-org/research#80
One viable technical solution to achieve is to use a RBSR mechanism similar to that implemented in https://github.com/hoytech/negentropy
This issue tracks the work necessary to integrate this library into nwaku and test its suitability for a sync mechanism.
cc @SionoiS @chaitanyaprem
The text was updated successfully, but these errors were encountered: