search

XRPLF / clio

An XRP Ledger API Server
https://xrpl.org
ISC License
55 stars 47 forks source link

Improve etl check #1465

Closed cindyyan317 closed 1 week ago

cindyyan317 commented 1 week ago

When there was an ETL not working and allow_no_etl is false, Clio would fail to start. This PR improved the behavior. As long as there is one ETL working, Clio should be able to start.

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 25.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 67.78%. Comparing base (37cd79c) to head (7483320).

:exclamation: Current head 7483320 differs from pull request most recent head fd0a31d

Please upload reports for the commit fd0a31d to get more accurate results.

Files Patch % Lines
src/etl/LoadBalancer.cpp 25.00% 1 Missing and 2 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #1465 +/- ## =========================================== - Coverage 67.81% 67.78% -0.03% =========================================== Files 233 233 Lines 9416 9418 +2 Branches 5251 5253 +2 =========================================== - Hits 6385 6384 -1 - Misses 1653 1654 +1 - Partials 1378 1380 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.