forked from mit-dci/opencbdc-tx
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Td/values clean #9
Open
darbha-ram
wants to merge
15
commits into
trunk
Choose a base branch
from
td/values
base: trunk
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
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
Signed-off-by: Sam Stuewe <[email protected]>
Signed-off-by: James Lovejoy <[email protected]>
Co-authored-by: Sam Stuewe <[email protected]> Signed-off-by: James Lovejoy <[email protected]>
…atomizer integration tests Signed-off-by: James Lovejoy <[email protected]>
Signed-off-by: James Lovejoy <[email protected]>
Co-authored-by: Sam Stuewe <[email protected]> Signed-off-by: James Lovejoy <[email protected]>
Signed-off-by: James Lovejoy <[email protected]>
Do not skip audit if there was no previous audit running Signed-off-by: James Lovejoy <[email protected]>
Signed-off-by: Sam Stuewe <[email protected]>
Right now, the values implementation is quite a bit more bloated than it needs to be; the input side of a transaction only needs to be the UHS ids (just the hashes), not the full UHS element. Because the benchmarks were largely written against the cryptographic commitment code (for which this optimization has an outsized effect), they assumed this would be the case. For now, instead of implementing this optimization in the values solution, we just ensure the benchmarks behave correctly. In the long-run, if this branch should stay around, we should correctly minimize the input-side of compact transactions. Signed-off-by: Sam Stuewe <[email protected]>
Should awaken the TC Signed-off-by: Sam Stuewe <[email protected]>
Removes blocking SA-validation in the shards, depending on the coordinator to faithfully handle it. Signed-off-by: Sam Stuewe <[email protected]>
This sets up the 2PC load-generator to read the config values the Test Controller already supports* to limit throughput to a set amount, optionally increasing it according to a definable schedule. Among other benefits, this allows the tester to constrain load-generators directly rather than by relying on the preseed count to artifically reduce their load. This also includes the relevant config changes to enable respected per-loadgen log-levels. Signed-off-by: Sam Stuewe <[email protected]>
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.
No description provided.