-
Notifications
You must be signed in to change notification settings - Fork 713
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
Split massa db as worker and exports #4045
Merged
Leo-Besancon
merged 13 commits into
testnet_24
from
split_massa_db_as_worker_and_exports
Jun 6, 2023
Merged
Split massa db as worker and exports #4045
Leo-Besancon
merged 13 commits into
testnet_24
from
split_massa_db_as_worker_and_exports
Jun 6, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Leo-Besancon
commented
Jun 6, 2023
- document all added functions
- try in sandbox /simulation/labnet
- unit tests on the added/changed features
- make tests compile
- make tests pass
- add logs allowing easy debugging in case the changes caused problems
- if the API has changed, update the API specification
Still has lifetime issues and some trait methods missing, e.g. for bootstrap
7 tasks
This was
linked to
issues
Jun 6, 2023
modship
approved these changes
Jun 6, 2023
AurelienFT
pushed a commit
that referenced
this pull request
Jun 15, 2023
* 1st step to have massa_db split between exports and worker * Create MassaDBController trait * Implement MassaDBController for MassaDB * Added read function to MassaDBController * Started cleaning up build errors - not finished * Implemented more trait methods, cleaned a bit Still has lifetime issues and some trait methods missing, e.g. for bootstrap * Fixed build errors * Fix after rebase on testnet24 * Fix clippy warnings * Simplified lifetime definitions * Added doc comments * Updated doc comments for get_batch and get_versioning_batch * In pos-exports, put the massa-db-worker as a dev-dependency instead of optional dependancy
AurelienFT
added a commit
that referenced
this pull request
Jun 15, 2023
This reverts commit c1379f2.
damip
added a commit
that referenced
this pull request
Jun 15, 2023
* Update testnet24 * Split massa db as worker and exports (#4045) * 1st step to have massa_db split between exports and worker * Create MassaDBController trait * Implement MassaDBController for MassaDB * Added read function to MassaDBController * Started cleaning up build errors - not finished * Implemented more trait methods, cleaned a bit Still has lifetime issues and some trait methods missing, e.g. for bootstrap * Fixed build errors * Fix after rebase on testnet24 * Fix clippy warnings * Simplified lifetime definitions * Added doc comments * Updated doc comments for get_batch and get_versioning_batch * In pos-exports, put the massa-db-worker as a dev-dependency instead of optional dependancy * Make pub * Make RawMassaDB fileds pub * Clean bootstrap state and versioning cursors (#4053) * Initial rework of get_batch_to_stream and get_versioning_batch_to_stream * Updated write_batch_bootstrap_client and server cursors' logic * secure denunciation pool * correct errors * operation pool refactoring * op pool filtering (no scoring yet) * updates * update executed ops * scoring ops * full compile * optimize * Fix test compilation * better endo expiry * update constants * minor improvements on scoring * Fix tests of pool. * Revert "Clean bootstrap state and versioning cursors (#4053)" This reverts commit 08f7601. * Revert "Make RawMassaDB fileds pub" This reverts commit 730c6e2. * Revert "Make pub" This reverts commit 9e43bbe. * Revert "Split massa db as worker and exports (#4045)" This reverts commit c1379f2. * Use cargo lock from main * Fix grpc compilation * Fix config variable * Test endorsements in protocol * Fix tests * Fix protocol test --------- Co-authored-by: AurelienFT <aurelien.foucault@epitech.eu> Co-authored-by: Leo-Besancon <lb@massa.net> Co-authored-by: Leo-Besancon <leoleo38@live.fr>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.