diff options
| author | Christian Grothoff <christian@grothoff.org> | 2020-03-19 19:33:20 +0100 |
|---|---|---|
| committer | Christian Grothoff <christian@grothoff.org> | 2020-03-19 19:33:20 +0100 |
| commit | f76e7c46e6609d39d5b28cb489a946833ca58bfb (patch) | |
| tree | f111ba1e0249c45374248e85fb4d7ca096e542b3 /src/testing/testing_api_cmd_deposit.c | |
| parent | 5e98c0652a6a03a7ce70405fa53adf706412b24b (diff) | |
| download | exchange-f76e7c46e6609d39d5b28cb489a946833ca58bfb.tar.gz exchange-f76e7c46e6609d39d5b28cb489a946833ca58bfb.tar.bz2 exchange-f76e7c46e6609d39d5b28cb489a946833ca58bfb.zip | |
make do_retry not retry forever in tests/benchmark
Diffstat (limited to 'src/testing/testing_api_cmd_deposit.c')
| -rw-r--r-- | src/testing/testing_api_cmd_deposit.c | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/src/testing/testing_api_cmd_deposit.c b/src/testing/testing_api_cmd_deposit.c index 573c68b9c..6781568a1 100644 --- a/src/testing/testing_api_cmd_deposit.c +++ b/src/testing/testing_api_cmd_deposit.c @@ -30,6 +30,12 @@ /** + * How often do we retry before giving up? + */ +#define NUM_RETRIES 5 + + +/** * State for a "deposit" CMD. */ struct DepositState @@ -104,9 +110,9 @@ struct DepositState unsigned int expected_response_code; /** - * Should we retry on (transient) failures? + * How often should we retry on (transient) failures? */ - int do_retry; + unsigned int do_retry; /** * Set to #GNUNET_YES if the /deposit succeeded @@ -184,8 +190,9 @@ deposit_cb (void *cls, ds->dh = NULL; if (ds->expected_response_code != http_status) { - if (GNUNET_YES == ds->do_retry) + if (0 != ds->do_retry) { + ds->do_retry--; if ( (0 == http_status) || (TALER_EC_DB_COMMIT_FAILED_ON_RETRY == ec) || (MHD_HTTP_INTERNAL_SERVER_ERROR == http_status) ) @@ -559,7 +566,7 @@ TALER_TESTING_cmd_deposit_with_retry (struct TALER_TESTING_Command cmd) GNUNET_assert (&deposit_run == cmd.run); ds = cmd.cls; - ds->do_retry = GNUNET_YES; + ds->do_retry = NUM_RETRIES; return cmd; } |