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

tests: turn on LIMIT_TX_QUEUE_SOURCE_ACCOUNT by default and rewrite test cases to support it #3787

Merged
merged 4 commits into from
Jul 15, 2023

Conversation

marta-lokhova
Copy link
Contributor

@marta-lokhova marta-lokhova commented Jun 20, 2023

Resolves the second bullet in #3812

Note that this change creates a stricter relationship between the desired tx rate and number of accounts created. This should be reflected in https://github.com/stellar/supercluster as well

@marta-lokhova marta-lokhova force-pushed the loadgen_source_accounts branch from 4c646bc to b2e0b69 Compare June 21, 2023 04:02
@marta-lokhova marta-lokhova changed the title Loadgen: generate load with unique source accounts tests: generate load with unique source accounts and cleanup test suite Jun 27, 2023
@marta-lokhova marta-lokhova force-pushed the loadgen_source_accounts branch from 8fe0e80 to bea62e5 Compare June 27, 2023 23:38
@marta-lokhova marta-lokhova changed the title tests: generate load with unique source accounts and cleanup test suite tests: turn on LIMIT_TX_QUEUE_SOURCE_ACCOUNT by default and rewrite test cases to support it Jul 11, 2023
@marta-lokhova marta-lokhova force-pushed the loadgen_source_accounts branch from b4cc169 to f19247b Compare July 11, 2023 21:04
@marta-lokhova marta-lokhova requested a review from sisuresh July 11, 2023 23:33
src/transactions/test/MergeTests.cpp Outdated Show resolved Hide resolved
src/transactions/test/PaymentTests.cpp Outdated Show resolved Hide resolved
@marta-lokhova marta-lokhova force-pushed the loadgen_source_accounts branch from b90b48f to 3f679a1 Compare July 14, 2023 22:30
@marta-lokhova
Copy link
Contributor Author

@sisuresh I needed to re-generate tx meta, but this is done now. r+ if it the change looks good?

@sisuresh
Copy link
Contributor

r+ 3f679a1

@latobarita latobarita merged commit b385714 into stellar:master Jul 15, 2023
@marta-lokhova marta-lokhova self-assigned this Jul 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants