adamsaghy commented on code in PR #6187:
URL: https://github.com/apache/fineract/pull/6187#discussion_r3704864542


##########
fineract-core/src/main/java/org/apache/fineract/infrastructure/core/domain/JdbcSupport.java:
##########
@@ -65,6 +68,23 @@ public static LocalTime getLocalTime(final ResultSet rs, 
final String columnName
         return localTime;
     }
 
+    public static OffsetTime getOffsetTime(final ResultSet rs, final String 
columnName) throws SQLException {

Review Comment:
   I dont think we need this many options... the field is stored as 
`OffsetTime` and fetched as `OffsetTime`. No need for anything else but reading 
as OffsetTime...



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to