Skip to content

feat: add support for auto-configure of reactive API#244

Merged
bednar merged 2 commits intomasterfrom
spring-refactoring
Jul 26, 2021
Merged

feat: add support for auto-configure of reactive API#244
bednar merged 2 commits intomasterfrom
spring-refactoring

Conversation

@bednar
Copy link
Copy Markdown
Contributor

@bednar bednar commented Jul 13, 2021 •
edited
Loading

Closes #243
Closes #237

Proposed Changes

This PR adds support for auto-configure InfluxDBClientReactive for SpringBoot and also change configuration prefix from spring.influx2 to influx according to Spring Docs:

image

Checklist

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

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 13, 2021 •
edited
Loading

Codecov Report

Merging #244 (ca78d5f) into master (643d3da) will increase coverage by 0.09%.
The diff coverage is 100.00%.

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

@@             Coverage Diff              @@
##             master     #244      +/-   ##
============================================
+ Coverage     89.14%   89.24%   +0.09%     
- Complexity      442      447       +5     
============================================
  Files           143      143              
  Lines          5759     5756       -3     
  Branches        271      270       -1     
============================================
+ Hits           5134     5137       +3     
+ Misses          546      540       -6     
  Partials         79       79
Impacted Files Coverage Δ
...lth/InfluxDB2HealthIndicatorAutoConfiguration.java 100.00% <ø> (+100.00%) ⬆️
...om/influxdb/spring/influx/InfluxDB2Properties.java 71.42% <ø> (ø)
...uxdb/spring/influx/InfluxDB2AutoConfiguration.java 95.83% <100.00%> (+0.59%) ⬆️
...om/influxdb/client/flux/FluxConnectionOptions.java 91.83% <0.00%> (-0.17%) ⬇️
...ava/com/influxdb/query/internal/FluxCsvParser.java 97.24% <0.00%> (-0.06%) ⬇️
...ava/com/influxdb/client/InfluxDBClientOptions.java 96.32% <0.00%> (-0.06%) ⬇️
...fluxdb/spring/health/InfluxDB2HealthIndicator.java 26.66% <0.00%> (+26.66%) ⬆️

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 643d3da...a2f3bd0. Read the comment docs.

@bednar bednar marked this pull request as ready for review July 13, 2021 08:24
@bednar bednar requested a review from rhajek July 13, 2021 08:24
bednar added 2 commits July 26, 2021 14:09
@bednar
feat: add support for auto-configure of reactive API
af8da42 
- breaking change: rename property prefix from `spring.influx2` to `influx`
@bednar
docs: update CHANGELOG.md
a2f3bd0
@bednar bednar force-pushed the spring-refactoring branch from ca78d5f to a2f3bd0 Compare July 26, 2021 12:28
@bednar bednar merged commit 2b3e3f6 into master Jul 26, 2021
@bednar bednar deleted the spring-refactoring branch July 26, 2021 12:37
@bednar bednar added this to the 3.1.0 milestone Jul 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@rhajek rhajek rhajek approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3.1.0

Development

Successfully merging this pull request may close these issues.

Spring integration is using the spring namespace Expose reactive Api for Spring Boot Starter

3 participants

@bednar @codecov-commenter @rhajek