search

influxdata / influxdb-java

Java client for InfluxDB
MIT License
1.17k stars 476 forks source link

Support generic POJO super classes #980

Closed eranl closed 7 months ago

eranl commented 7 months ago

Also, simplify type checking.

Should I add this to the manual?

codecov-commenter commented 7 months ago

Codecov Report

Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (e93949f) 60.98% compared to head (66f3075) 61.05%. Report is 1 commits behind head on master.

:exclamation: Current head 66f3075 differs from pull request most recent head 2394d26. Consider uploading reports for the commit 2394d26 to get more accurate results

Files Patch % Lines
...n/java/org/influxdb/impl/InfluxDBResultMapper.java 90.74% 3 Missing and 2 partials :warning:
src/main/java/org/influxdb/impl/TypeMapper.java 75.00% 2 Missing and 2 partials :warning:

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

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #980 +/- ## ============================================ + Coverage 60.98% 61.05% +0.07% - Complexity 435 437 +2 ============================================ Files 70 71 +1 Lines 2568 2583 +15 Branches 276 278 +2 ============================================ + Hits 1566 1577 +11 - Misses 932 933 +1 - Partials 70 73 +3 ```

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

majst01 commented 7 months ago

Support for Generics is for sure worth mentioning in the manual.

eranl commented 7 months ago

Support for Generics is for sure worth mentioning in the manual.

Done