说来惭愧干了 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 ClassT 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 groupIdorg.mybatis/groupId artifactIdmybatis-typehandlers-jsr310/artifactId version1.0.1/version /dependency他为什么能解决这个问题我看了下他的包里面的代码这不就是加了个 TypeHandler 自己处理了嘛。再去看了下OffsetDateTimeTypeHandler的实现其实就是自己就解决了直接给返回OffsetDateTime根本不会走到 ShardingSphere 的逻辑里面去这也就是他能解决这个问题的原因了。当然如果不想那么麻烦引入一个包也可以单独把他拎出来自己去指定一下这个很简单就不多说了。提PR于是我想这事情这么简单我不如提个 PR 给官方吧这里教下大家怎么提 PR 。因为不是咱们的项目是没法 push 代码的所以进入到项目然后forkfork 好了以后直接把项目 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单独用实现类不过那样的话就得大工作了我太懒了就这样。。。。最后要看了一下大佬们已经把这个bug Fixed了我这也算是为开源做过贡献的人了身上光环锃亮