Skip to content
This repository has been archived by the owner on Aug 13, 2024. It is now read-only.

fix: InfluxDB client read/write timeout settings #163

Merged
merged 3 commits into from
Apr 22, 2024
Merged

Conversation

alespour
Copy link
Contributor

@alespour alespour commented Apr 19, 2024

Closes #158

Proposed Changes

Uses max connection timeout also as read/write timeout in InfluxDB client.

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • Tests pass
  • Commit messages are conventional

@codecov-commenter
Copy link

Codecov Report

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

Project coverage is 34.51%. Comparing base (eb50e63) to head (f1a8dd0).

Files Patch % Lines
...n/java/org/influxdata/nifi/util/InfluxDBUtils.java 66.66% 2 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #163      +/-   ##
============================================
+ Coverage     34.15%   34.51%   +0.35%     
+ Complexity      131       64      -67     
============================================
  Files            27       27              
  Lines          2029     2034       +5     
  Branches        146      146              
============================================
+ Hits            693      702       +9     
+ Misses         1308     1302       -6     
- Partials         28       30       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@alespour alespour marked this pull request as ready for review April 19, 2024 14:48
@alespour alespour requested review from bednar and karel-rehor and removed request for bednar April 22, 2024 06:59
Copy link
Contributor

@bednar bednar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@alespour alespour merged commit 7a74bc9 into master Apr 22, 2024
10 checks passed
@alespour alespour deleted the fix/query-timeout branch April 22, 2024 07:36
@bednar bednar added this to the 1.28.0 milestone Apr 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GetInfluxDatabase_2 10 seconds timeout problem
3 participants