Closed austin3dickey closed 1 year ago
Oh, I forgot to comment on this. Thanks for the pragmatic approach. Let's do this, and learn.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
benchmarks/_benchmark.py | 4 | 8 | 50.0% | ||
<!-- | Total: | 4 | 8 | 50.0% | --> |
Totals | |
---|---|
Change from base Build 5137100215: | -0.2% |
Covered Lines: | 1526 |
Relevant Lines: | 1748 |
This PR addresses https://github.com/voltrondata-labs/arrow-benchmarks-ci/issues/121 by overwriting the
publish()
method that most benchmarks use to post benchmark results to Conbench. It now should use the newbenchclients.ConbenchClient
, which has more reasonable retrying logic.