influxdata / influxdb-java

Java client for InfluxDB
MIT License
1.19k stars 478 forks source link

Fix time column with precision > ms #973

Closed eranl closed 12 months ago

eranl commented 12 months ago

When @TimeColumn is used with timeUnit larger than a millisecond, saving the pojo would result in a wrong timestamp. This change fixes that.

codecov-commenter commented 12 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ba9bfeb) 61.28% compared to head (b95938e) 61.27%.

:exclamation: Current head b95938e differs from pull request most recent head 3a2cf97. Consider uploading reports for the commit 3a2cf97 to get more accurate results

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

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #973 +/- ## ============================================ - Coverage 61.28% 61.27% -0.02% Complexity 450 450 ============================================ Files 70 70 Lines 2591 2590 -1 Branches 277 277 ============================================ - Hits 1588 1587 -1 Misses 932 932 Partials 71 71 ```

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

majst01 commented 12 months ago

somehow related to #182