[hibernate#856] Support property JDBC_TIME_ZONE for time types

We cannot apply a timezone to a time without a date, this probably the best we can do without throwing an exception.
DavideD · Jun 14, 2021 · e0d10f5 · e0d10f5
1 parent 5587b8a
commit e0d10f5
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 15 deletions.
diff --git a/...tive-core/src/main/java/org/hibernate/reactive/adaptor/impl/PreparedStatementAdaptor.java b/...tive-core/src/main/java/org/hibernate/reactive/adaptor/impl/PreparedStatementAdaptor.java
@@ -157,6 +157,11 @@ public void setTime(int parameterIndex, Time x) {
 		put( parameterIndex, x.toLocalTime() );
 	}
 
+	@Override
+	public void setTime(int parameterIndex, Time x, Calendar cal) {
+		put( parameterIndex, x.toLocalTime() );
+	}
+
 	@Override
 	public void setTimestamp(int parameterIndex, Timestamp x) {
 		put( parameterIndex, x.toLocalDateTime() );
@@ -242,11 +247,6 @@ public void setDate(int parameterIndex, Date x, Calendar cal) {
 		put( parameterIndex, x.toLocalDate() );
 	}
 
-	@Override
-	public void setTime(int parameterIndex, Time x, Calendar cal) {
-		throw new UnsupportedOperationException();
-	}
-
 	@Override
 	public void setNull(int parameterIndex, int sqlType, String typeName) {
 		throw new UnsupportedOperationException();

diff --git a/...ate-reactive-core/src/main/java/org/hibernate/reactive/adaptor/impl/ResultSetAdaptor.java b/...ate-reactive-core/src/main/java/org/hibernate/reactive/adaptor/impl/ResultSetAdaptor.java
@@ -151,6 +151,12 @@ public Time getTime(int columnIndex) {
 		return (wasNull=localTime==null) ? null : Time.valueOf(localTime);
 	}
 
+	@Override
+	public Time getTime(int columnIndex, Calendar cal) {
+		LocalTime localTime = row.getLocalTime(columnIndex);
+		return (wasNull=localTime==null) ? null : Time.valueOf(localTime);
+	}
+
 	@Override
 	public Timestamp getTimestamp(int columnIndex) {
 		LocalDateTime localDateTime = row.getLocalDateTime(columnIndex);
@@ -251,6 +257,12 @@ public Time getTime(String columnLabel) {
 		return (wasNull=localTime==null) ? null : Time.valueOf(localTime);
 	}
 
+	@Override
+	public Time getTime(String columnLabel, Calendar cal) {
+		LocalTime localTime = row.getLocalTime( columnLabel );
+		return ( wasNull = localTime == null ) ? null : Time.valueOf( localTime );
+	}
+
 	@Override
 	public Timestamp getTimestamp(String columnLabel) {
 		Object rawValue = row.getValue(columnLabel);
@@ -711,16 +723,6 @@ public Date getDate(String columnLabel, Calendar cal) {
 		throw new UnsupportedOperationException();
 	}
 
-	@Override
-	public Time getTime(int columnIndex, Calendar cal) {
-		throw new UnsupportedOperationException();
-	}
-
-	@Override
-	public Time getTime(String columnLabel, Calendar cal) {
-		throw new UnsupportedOperationException();
-	}
-
 	@Override
 	public Timestamp getTimestamp(int columnIndex, Calendar cal) {
 		throw new UnsupportedOperationException();