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.

Sign up for GitHub

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

Jump to bottom

TimeColumnLocalTimeMapper incorrectly uses "getNano()" #70

Open

davidemassarenti-optio3 opened this issue Mar 22, 2018 · 1 comment

davidemassarenti-optio3 commented Mar 22, 2018 •

edited

Loading

jadira/usertype.core/src/main/java/org/jadira/usertype/dateandtime/threeten/columnmapper/TimeColumnLocalTimeMapper.java

Line 58 in d55d023

final Time time = new Time(localDateTime.getNano());

getNano() only returns the nanosecond portion of the time.

This should use something like Time.valueOf(localDateTime.toLocalTime())

Dropwizard 1.3.0 took a dependency on the 7.0.0.CR1 release, which means all uses of LocalTime in Hibernate will truncate time.

rafaelbrizola commented Oct 1, 2019

I'm having the same problem here. Basically, a column of type Time is unusable.

rafaelbrizola mentioned this issue

Fixes TimeColumnLocalTimeMapper #84

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment