共计 3466 个字符,预计需要花费 9 分钟才能阅读完成。
说来惭愧,干了 10 来年程序员,还没有给开源做过任何奉献,以前只晓得嘎嘎写,出了问题嘎嘎改,素来没想过提个 PR 去修复他,最近碰到个问题,发现挺简略的,就顺手提了个 PR 过来。
问题
问题挺简略的,就是在应用 mybatis 和 ShardingSphere 的时候,有人在 model 类应用了 OffsetDateTime
这个工夫类型,发现会报错。
Caused by: java.lang.ClassCastException: class java.sql.Timestamp cannot be cast to class java.time.OffsetDateTime (java.sql.Timestamp is in module java.sql of loader 'platform'; java.time.OffsetDateTime is in module java.base of loader 'bootstrap')
at org.apache.ibatis.type.OffsetDateTimeTypeHandler.getNullableResult(OffsetDateTimeTypeHandler.java:38)
at org.apache.ibatis.type.OffsetDateTimeTypeHandler.getNullableResult(OffsetDateTimeTypeHandler.java:28)
at org.apache.ibatis.type.BaseTypeHandler.getResult(BaseTypeHandler.java:85)
... 99 more
这就是一个简略的类型转换的异样,于是跟着源码看了下,先看到 BaseTypeHandler#getResult
这个办法,实际上就是依据列名返回查问后果。
依据调用关系,找到了 OffsetDateTimeTypeHandler
实现类。
发现最终会调用 rs.getObject()
这个办法,那么其实这个办法会最终走到由 ShardingSphere 实现的 getObject
办法中。
看到这里的时候其实曾经明确了为啥会报错了,Shardingsphere 只判断了几个 LocalDateTime
等类型,对于这个比拟非凡的工夫类型没有解决,最终会转换成 Timestamp,而后强转就报错了。
最初调用到 ResultSetUtil#convertTimestampValue
办法,能够看到的确是这样哈。
那如果批改源码的话其实很简略了,getObject
判断多加一个,convertTimestampValue
再加一个,就这样很简略啊。
@Override
public <T> T getObject(final int columnIndex, final Class<T> type) throws SQLException {if (BigInteger.class.equals(type)) {return (T) BigInteger.valueOf(getLong(columnIndex));
} else if (Blob.class.equals(type)) {return (T) getBlob(columnIndex);
} else if (Clob.class.equals(type)) {return (T) getClob(columnIndex);
} else if (LocalDateTime.class.equals(type) || LocalDate.class.equals(type) || LocalTime.class.equals(type) || OffsetDateTime.class.equals(type)) {return (T) ResultSetUtil.convertValue(mergeResultSet.getValue(columnIndex, Timestamp.class), type);
} else {return (T) ResultSetUtil.convertValue(mergeResultSet.getValue(columnIndex, type), type);
}
}
private static Object convertTimestampValue(final Object value, final Class<?> convertType) {Timestamp timestamp = (Timestamp) value;
if (LocalDateTime.class.equals(convertType)) {return timestamp.toInstant().atZone(ZoneId.systemDefault()).toLocalDateTime();}
if (LocalDate.class.equals(convertType)) {return timestamp.toInstant().atZone(ZoneId.systemDefault()).toLocalDate();}
if (LocalTime.class.equals(convertType)) {return timestamp.toInstant().atZone(ZoneId.systemDefault()).toLocalTime();}
if (OffsetDateTime.class.equals(convertType)) {return timestamp.toInstant().atZone(ZoneId.systemDefault()).toOffsetDateTime();}
return value;
}
修复
最开始我其实并不想改源码,我在想其余的实现计划,搜寻后发现引入一个包就能够解决,也就是 mybatis 的 JSR310 标准。
<dependency>
<groupId>org.mybatis</groupId>
<artifactId>mybatis-typehandlers-jsr310</artifactId>
<version>1.0.1</version>
</dependency>
他为什么能解决这个问题?我看了下他的包外面的代码,这不就是加了个 TypeHandler 本人解决了嘛。
再去看了下 OffsetDateTimeTypeHandler
的实现,其实就是本人就解决了,间接给返回OffsetDateTime
,基本不会走到 ShardingSphere 的逻辑外面去,这也就是他能解决这个问题的起因了。
当然,如果不想那么麻烦引入一个包,也能够独自把他拎进去本人去指定一下,这个很简略,就不多说了。
提 PR
于是我想,这事件这么简略,我不如提个 PR 给官网吧,这里教下大家怎么提 PR。
因为不是咱们的我的项目,是没法 push 代码的,所以进入到我的项目,而后fork
,fork 好了当前,间接把我的项目 clone 下来,而后执行命令。
git remote add upstream https://github.com/apache/shardingsphere.git
通过命令咱们能够看到胜利了,这样就 OK 了,而后失常拉分支写代码吧。
写完之后,失常去咱们的我的项目界面提交 PR,而后就能够了。
麻烦
当然,过程并没有这么顺利,尽管说只是很简略的批改。
首先,这个校验就给我提醒谬误了,第一点叫我不要用 *
号去援用。
这个其实是 IDEA
的锅,如果援用同一个包下类过多的话,会主动帮咱们转成星号,这个咱们能够在Editor-Code Style-Java
,而后找到 Imports 下的这两个选项,把他们都改成 99 就能够了,避免他主动给咱们改成星号。
还有一些其余的比方 if 前面没跟空格之类的,这是我遗记格式化了!
而后大佬回复感觉看不下去,这代码太恶心了,说咱们是不是能够用 java.time.temporal.TemporalAccessor
来判断,不然这么多工夫类型,搞个毛线呢。
而后我就翻译了一段英文,我也不晓得大佬看没看懂,我通知他,这个不好整啊,你看这个接口啊,很多乌七八糟的类实现了他,实际上我感觉咱们笼罩罕用的一些就行了,其余的非凡工夫类型让他们本人用 TypeHandler 解决吧。
大佬说,嗯当然,没方法判断这个接口那咱们也没辙了,我说那不可就是嘛。
其实,还有很多工夫类型他都会报错的,最好的方法这个都形象进去和 Mybatis 独自用实现类,不过那样的话就得大工作了,我太懒了,就这样。
。。。
工夫过来了几天,看了下他还没合我代码,是不是厌弃我?