Skip to content

chore: use async HTTP call in batching writer#280

Merged
bednar merged 3 commits intomasterfrom
chore/async
Nov 22, 2021
Merged

chore: use async HTTP call in batching writer#280
bednar merged 3 commits intomasterfrom
chore/async

Conversation

@bednar
Copy link
Copy Markdown
Contributor

@bednar bednar commented Nov 15, 2021

Proposed Changes

Use asynchronous HTTP calls in batching writer to achieve better performance in a multithreading environment.

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • mvn test completes successfully
  • Commit messages are in semantic format
  • Sign CLA (if not already signed)

@bednar bednar added the enhancement New feature or request label Nov 15, 2021
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 15, 2021

Codecov Report

Merging #280 (b6bf1ee) into master (798ff29) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head b6bf1ee differs from pull request most recent head 0d64bf6. Consider uploading reports for the commit 0d64bf6 to get more accurate results
Impacted file tree graph

@@            Coverage Diff            @@
##             master     #280   +/-   ##
=========================================
  Coverage     88.37%   88.37%           
  Complexity      473      473           
=========================================
  Files           148      148           
  Lines          5865     5868    +3     
  Branches        276      276           
=========================================
+ Hits           5183     5186    +3     
  Misses          601      601           
  Partials         81       81           
Impacted Files Coverage Δ
.../influxdb/client/internal/AbstractWriteClient.java 91.38% <100.00%> (+0.12%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 798ff29...0d64bf6. Read the comment docs.

@bednar bednar requested a review from rhajek November 15, 2021 10:55
@bednar bednar marked this pull request as ready for review November 15, 2021 10:55
@bednar bednar merged commit e0d5c42 into master Nov 22, 2021
@bednar bednar deleted the chore/async branch November 22, 2021 06:53
@bednar bednar added this to the 3.5.0 milestone Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants