databricks / databricks-sql-nodejs

Databricks SQL Connector for Node.js
Apache License 2.0
24 stars 32 forks source link

[PECO-729] Improve retry behavior #230

Closed kravets-levko closed 6 months ago

kravets-levko commented 7 months ago

PECO-729

Should be easier (hopefully) to review commit by commit

Note: this PR doesn't include logic for retry on network errors. That part will be covered in a follow-up

codecov-commenter commented 7 months ago

Codecov Report

Attention: Patch coverage is 96.66667% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 93.19%. Comparing base (17112c7) to head (7303b9a).

Files Patch % Lines
lib/connection/connections/ThriftHttpConnection.ts 95.23% 1 Missing :warning:
lib/hive/Commands/BaseCommand.ts 66.66% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #230 +/- ## ========================================== + Coverage 93.09% 93.19% +0.09% ========================================== Files 62 63 +1 Lines 1478 1513 +35 Branches 256 262 +6 ========================================== + Hits 1376 1410 +34 - Misses 40 41 +1 Partials 62 62 ```

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