-
Notifications
You must be signed in to change notification settings - Fork 130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: mapping Number
types to POJO query output
#414
Conversation
client-core/src/main/java/com/influxdb/query/internal/FluxResultMapper.java
Show resolved
Hide resolved
client-core/src/main/java/com/influxdb/query/internal/FluxResultMapper.java
Show resolved
Hide resolved
client-core/src/main/java/com/influxdb/query/internal/FluxResultMapper.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bednar thanks for your PR, please add also mapping for short and float types.
240924b
to
9712c87
Compare
Codecov ReportBase: 88.22% // Head: 88.32% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #414 +/- ##
============================================
+ Coverage 88.22% 88.32% +0.10%
+ Complexity 777 775 -2
============================================
Files 172 172
Lines 6987 6987
Branches 375 375
============================================
+ Hits 6164 6171 +7
+ Misses 712 703 -9
- Partials 111 113 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
d2eb05e
to
63ac968
Compare
Closes #410
Proposed Changes
Fixed mapping aggregated results (changed type) from InfluxDB.
Checklist
mvn test
completes successfully