Merge lp:~massimo-siani/mariadb-java-client/CONJ-86 into lp:mariadb-java-client
Proposed by
Massimo Siani
Status: | Needs review |
---|---|
Proposed branch: | lp:~massimo-siani/mariadb-java-client/CONJ-86 |
Merge into: | lp:mariadb-java-client |
Diff against target: |
136 lines (+106/-1) 2 files modified
src/main/java/org/mariadb/jdbc/internal/common/AbstractValueObject.java (+13/-1) src/test/java/org/mariadb/jdbc/TimestampTest.java (+93/-0) |
To merge this branch: | bzr merge lp:~massimo-siani/mariadb-java-client/CONJ-86 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Maria-captains | Pending | ||
Review via email: mp+224263@code.launchpad.net |
Commit message
Fix for CONJ-86
Description of the change
CONJ-86:
It seems that the getTimestamp of the ResultSet class use the default client timestamp for time conversions. This works in most cases, but when Daylight Saving Time is used, it fails at certain times.
Proposed solution: if an exception is thrown while parsing the timestamp, and no explicit calendar has been set, try with UTC. Otherwise, works as before (throws the exception).
To post a comment you must log in.
Unmerged revisions
- 513. By Massimo Siani
-
Merge changes from trunk and retest
- 512. By Massimo Siani
-
If a calendar has been given, do not try with UTC
- 511. By Massimo Siani
-
Fix for CONJ-86