Hibernate/hibernate-orm
1
0
Fork 0
mirror of https://github.com/hibernate/hibernate-orm synced 2025-02-08 20:24:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

HHH-7748 - EnumType crashes because of unsupported JDBC API in Oracle

Browse Source
This commit is contained in:
Steve Ebersole 2012-11-06 08:26:11 -06:00
parent 1176477e6a
commit 432dab3465
1 changed files with 53 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

58
hibernate-core/src/main/java/org/hibernate/type/EnumType.java
View File

@ -100,12 +100,42 @@ public int hashCode(Object x) throws HibernateException {
@Override @Override
public Object nullSafeGet(ResultSet rs, String[] names, SessionImplementor session, Object owner) throws SQLException { public Object nullSafeGet(ResultSet rs, String[] names, SessionImplementor session, Object owner) throws SQLException {
if ( enumValueMapper == null ) { if ( enumValueMapper == null ) {
guessTypeOfEnumValueMapper( rs.getMetaData().getColumnType( rs.findColumn( names[0] ) ) ); resolveEnumValueMapper( rs, names[0] );
} }
return enumValueMapper.getValue( rs, names ); return enumValueMapper.getValue( rs, names );
} }
private void guessTypeOfEnumValueMapper(int columnType) { private void resolveEnumValueMapper(ResultSet rs, String name) {
if ( enumValueMapper == null ) {
try {
resolveEnumValueMapper( rs.getMetaData().getColumnType( rs.findColumn( name ) ) );
}
catch (Exception e) {
// because some drivers do not implement this
LOG.debugf(
"JDBC driver threw exception calling java.sql.ResultSetMetaData.getColumnType; " +
"using fallback determination [%s] : %s",
enumClass.getName(),
e.getMessage()
);
// peek at the result value to guess type (this is legacy behavior)
try {
Object value = rs.getObject( name );
if ( Number.class.isInstance( value ) ) {
treatAsOrdinal();
}
else {
treatAsNamed();
}
}
catch (SQLException ignore) {
treatAsOrdinal();
}
}
}
}
private void resolveEnumValueMapper(int columnType) {
// fallback for cases where not enough parameter/parameterization information was passed in // fallback for cases where not enough parameter/parameterization information was passed in
if ( isOrdinal( columnType ) ) { if ( isOrdinal( columnType ) ) {
treatAsOrdinal(); treatAsOrdinal();
@ -118,11 +148,29 @@ private void guessTypeOfEnumValueMapper(int columnType) {
@Override @Override
public void nullSafeSet(PreparedStatement st, Object value, int index, SessionImplementor session) throws HibernateException, SQLException { public void nullSafeSet(PreparedStatement st, Object value, int index, SessionImplementor session) throws HibernateException, SQLException {
if ( enumValueMapper == null ) { if ( enumValueMapper == null ) {
guessTypeOfEnumValueMapper( st.getParameterMetaData().getParameterType( index ) ); resolveEnumValueMapper( st, index );
} }
enumValueMapper.setValue( st, (Enum) value, index ); enumValueMapper.setValue( st, (Enum) value, index );
} }
private void resolveEnumValueMapper(PreparedStatement st, int index) {
if ( enumValueMapper == null ) {
try {
resolveEnumValueMapper( st.getParameterMetaData().getParameterType( index ) );
}
catch (Exception e) {
// because some drivers do not implement this
LOG.debugf(
"JDBC driver threw exception calling java.sql.ParameterMetaData#getParameterType; " +
"falling back to ordinal-based enum mapping [%s] : %s",
enumClass.getName(),
e.getMessage()
);
treatAsOrdinal();
}
}
}
@Override @Override
public Object deepCopy(Object value) throws HibernateException { public Object deepCopy(Object value) throws HibernateException {
return value; return value;
@ -153,8 +201,8 @@ public void setParameterValues(Properties parameters) {
final ParameterType reader = (ParameterType) parameters.get( PARAMETER_TYPE ); final ParameterType reader = (ParameterType) parameters.get( PARAMETER_TYPE );
// IMPL NOTE : be protective about not setting enumValueMapper (i.e. calling treatAsNamed/treatAsOrdinal) // IMPL NOTE : be protective about not setting enumValueMapper (i.e. calling treatAsNamed/treatAsOrdinal)
// in cases where we do not have enough information. In such cases the `if` check in nullSafeGet/nullSafeSet // in cases where we do not have enough information. In such cases we do additional checks
// will kick in to query against the JDBC metadata to make that determination. // as part of nullSafeGet/nullSafeSet to query against the JDBC metadata to make the determination.
if ( reader != null ) { if ( reader != null ) {
enumClass = reader.getReturnedClass().asSubclass( Enum.class ); enumClass = reader.getReturnedClass().asSubclass( Enum.class );