公司新来一个干练小伙,把 MyBatis 替换成 MyBatis-Plus,上线后哭晕在厕所 ...

打印 上一主题 下一主题

主题 977|帖子 977|积分 2931

作者:青石路
泉源:https://www.cnblogs.com/youzhibing/p/18019399
MyBatis 替换成 MyBatis-Plus

背景介绍

一个老项目,数据库用的是 MySQL 5.7.36 , ORM 框架用的 MyBatis 3.5.0 , mysql-connector-java 版本是 5.1.26
新来了一个干练的小伙,精神充沛,看着就是一个喜欢折腾的主

他就觉得 MyBatis 利用起来不够简单,要写的代码还比较多,觉得有必要替换成 MyBatis-Plus
Mybatis-Plus 替换 Mybatis

先预备一张表 tbl_order ,然后初始化 2 条数据
  1. DROP TABLE IF EXISTS `tbl_order`;
  2. CREATE TABLE `tbl_order`  (
  3.   `id` bigint(0) UNSIGNED NOT NULL AUTO_INCREMENT COMMENT '自增主键',
  4.   `order_no` varchar(50) NOT NULL COMMENT '订单号',
  5.   `pay_time` datetime(3) DEFAULT NULL COMMENT '付款时间',
  6.   `created_at` datetime(3) NOT NULL DEFAULT CURRENT_TIMESTAMP(3) COMMENT '创建时间',
  7.   `updated_at` datetime(3) NOT NULL DEFAULT CURRENT_TIMESTAMP(3) ON UPDATE CURRENT_TIMESTAMP(3) COMMENT '最终修改时间',
  8.   PRIMARY KEY (`id`) USING BTREE
  9. ) ENGINE = InnoDB COMMENT = '订单';
  10. INSERT INTO `tbl_order` VALUES (1, '123456', '2024-02-21 18:38:32.000', '2024-02-21 18:37:34.000', '2024-02-21 18:40:01.720');
  11. INSERT INTO `tbl_order` VALUES (2, '654321', '2024-02-21 19:33:32.000','2024-02-21 19:32:12.020', '2024-02-21 19:34:03.727');
复制代码

保举一个开源免费的 Spring Boot 实战项目:
https://github.com/javastacks/spring-boot-best-practice
为了简化演示,我就直接用 Mybatis-Plus 搭建一个示例 demo ,以此来模拟下 "小伙" 替换的过程
只是用 MyBatis-Plus 替换 MyBatis ,其他组件的版本暂不动
Mybatis-Plus 版本就用 "小伙" 引用的版本:3.1.1 , mysql-connector-java 版本保持不变照旧 5.1.26
示例代码:play_it_safe
此时运行 com.qsl.OrderTest#orderListAllTest ,会报错,异常信息如下
  1. org.springframework.dao.TransientDataAccessResourceException: Error attempting to get column 'pay_time' from result set.  Cause: java.sql.SQLException: Conversion not supported for type java.time.LocalDateTime
  2. ; Conversion not supported for type java.time.LocalDateTime; nested exception is java.sql.SQLException: Conversion not supported for type java.time.LocalDateTime
  3.     at org.springframework.jdbc.support.SQLStateSQLExceptionTranslator.doTranslate(SQLStateSQLExceptionTranslator.java:110)
  4.     at org.springframework.jdbc.support.AbstractFallbackSQLExceptionTranslator.translate(AbstractFallbackSQLExceptionTranslator.java:72)
  5.     at org.springframework.jdbc.support.AbstractFallbackSQLExceptionTranslator.translate(AbstractFallbackSQLExceptionTranslator.java:81)
  6.     at org.springframework.jdbc.support.AbstractFallbackSQLExceptionTranslator.translate(AbstractFallbackSQLExceptionTranslator.java:81)
  7.     at org.mybatis.spring.MyBatisExceptionTranslator.translateExceptionIfPossible(MyBatisExceptionTranslator.java:73)
  8.     at org.mybatis.spring.SqlSessionTemplate$SqlSessionInterceptor.invoke(SqlSessionTemplate.java:446)
  9.     at com.sun.proxy.$Proxy53.selectList(Unknown Source)
  10.     at org.mybatis.spring.SqlSessionTemplate.selectList(SqlSessionTemplate.java:230)
  11.     at com.baomidou.mybatisplus.core.override.MybatisMapperMethod.executeForMany(MybatisMapperMethod.java:158)
  12.     at com.baomidou.mybatisplus.core.override.MybatisMapperMethod.execute(MybatisMapperMethod.java:76)
  13.     at com.baomidou.mybatisplus.core.override.MybatisMapperProxy.invoke(MybatisMapperProxy.java:62)
  14.     at com.sun.proxy.$Proxy59.selectList(Unknown Source)
  15.     at com.qsl.OrderTest.orderListAllTest(OrderTest.java:28)
  16.     at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  17.     at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
  18.     at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
  19.     at java.lang.reflect.Method.invoke(Method.java:498)
  20.     at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
  21.     at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
  22.     at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
  23.     at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
  24.     at org.springframework.test.context.junit4.statements.RunBeforeTestExecutionCallbacks.evaluate(RunBeforeTestExecutionCallbacks.java:74)
  25.     at org.springframework.test.context.junit4.statements.RunAfterTestExecutionCallbacks.evaluate(RunAfterTestExecutionCallbacks.java:84)
  26.     at org.springframework.test.context.junit4.statements.RunBeforeTestMethodCallbacks.evaluate(RunBeforeTestMethodCallbacks.java:75)
  27.     at org.springframework.test.context.junit4.statements.RunAfterTestMethodCallbacks.evaluate(RunAfterTestMethodCallbacks.java:86)
  28.     at org.springframework.test.context.junit4.statements.SpringRepeat.evaluate(SpringRepeat.java:84)
  29.     at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
  30.     at org.springframework.test.context.junit4.SpringJUnit4ClassRunner.runChild(SpringJUnit4ClassRunner.java:251)
  31.     at org.springframework.test.context.junit4.SpringJUnit4ClassRunner.runChild(SpringJUnit4ClassRunner.java:97)
  32.     at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
  33.     at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
  34.     at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
  35.     at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
  36.     at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
  37.     at org.springframework.test.context.junit4.statements.RunBeforeTestClassCallbacks.evaluate(RunBeforeTestClassCallbacks.java:61)
  38.     at org.springframework.test.context.junit4.statements.RunAfterTestClassCallbacks.evaluate(RunAfterTestClassCallbacks.java:70)
  39.     at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
  40.     at org.springframework.test.context.junit4.SpringJUnit4ClassRunner.run(SpringJUnit4ClassRunner.java:190)
  41.     at org.junit.runner.JUnitCore.run(JUnitCore.java:137)
  42.     at com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69)
  43.     at com.intellij.rt.junit.IdeaTestRunner$Repeater$1.execute(IdeaTestRunner.java:38)
  44.     at com.intellij.rt.execution.junit.TestsRepeater.repeat(TestsRepeater.java:11)
  45.     at com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:35)
  46.     at com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:232)
  47.     at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:55)
  48. Caused by: java.sql.SQLException: Conversion not supported for type java.time.LocalDateTime
  49.     at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:1078)
  50.     at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:989)
  51.     at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:975)
  52.     at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:920)
  53.     at com.mysql.jdbc.ResultSetImpl.getObject(ResultSetImpl.java:5126)
  54.     at com.mysql.jdbc.JDBC4ResultSet.getObject(JDBC4ResultSet.java:547)
  55.     at com.mysql.jdbc.ResultSetImpl.getObject(ResultSetImpl.java:5133)
  56.     at com.zaxxer.hikari.pool.HikariProxyResultSet.getObject(HikariProxyResultSet.java)
  57.     at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  58.     at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
  59.     at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
  60.     at java.lang.reflect.Method.invoke(Method.java:498)
  61.     at org.apache.ibatis.logging.jdbc.ResultSetLogger.invoke(ResultSetLogger.java:69)
  62.     at com.sun.proxy.$Proxy71.getObject(Unknown Source)
  63.     at org.apache.ibatis.type.LocalDateTimeTypeHandler.getNullableResult(LocalDateTimeTypeHandler.java:38)
  64.     at org.apache.ibatis.type.LocalDateTimeTypeHandler.getNullableResult(LocalDateTimeTypeHandler.java:28)
  65.     at org.apache.ibatis.type.BaseTypeHandler.getResult(BaseTypeHandler.java:81)
  66.     at org.apache.ibatis.executor.resultset.DefaultResultSetHandler.applyAutomaticMappings(DefaultResultSetHandler.java:521)
  67.     at org.apache.ibatis.executor.resultset.DefaultResultSetHandler.getRowValue(DefaultResultSetHandler.java:402)
  68.     at org.apache.ibatis.executor.resultset.DefaultResultSetHandler.handleRowValuesForSimpleResultMap(DefaultResultSetHandler.java:354)
  69.     at org.apache.ibatis.executor.resultset.DefaultResultSetHandler.handleRowValues(DefaultResultSetHandler.java:328)
  70.     at org.apache.ibatis.executor.resultset.DefaultResultSetHandler.handleResultSet(DefaultResultSetHandler.java:301)
  71.     at org.apache.ibatis.executor.resultset.DefaultResultSetHandler.handleResultSets(DefaultResultSetHandler.java:194)
  72.     at org.apache.ibatis.executor.statement.PreparedStatementHandler.query(PreparedStatementHandler.java:65)
  73.     at org.apache.ibatis.executor.statement.RoutingStatementHandler.query(RoutingStatementHandler.java:79)
  74.     at com.baomidou.mybatisplus.core.executor.MybatisSimpleExecutor.doQuery(MybatisSimpleExecutor.java:67)
  75.     at org.apache.ibatis.executor.BaseExecutor.queryFromDatabase(BaseExecutor.java:324)
  76.     at org.apache.ibatis.executor.BaseExecutor.query(BaseExecutor.java:156)
  77.     at org.apache.ibatis.executor.CachingExecutor.query(CachingExecutor.java:109)
  78.     at org.apache.ibatis.executor.CachingExecutor.query(CachingExecutor.java:83)
  79.     at org.apache.ibatis.session.defaults.DefaultSqlSession.selectList(DefaultSqlSession.java:147)
  80.     at org.apache.ibatis.session.defaults.DefaultSqlSession.selectList(DefaultSqlSession.java:140)
  81.     at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  82.     at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
  83.     at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
  84.     at java.lang.reflect.Method.invoke(Method.java:498)
  85.     at org.mybatis.spring.SqlSessionTemplate$SqlSessionInterceptor.invoke(SqlSessionTemplate.java:433)
  86.     ... 39 more
复制代码
注意看 Caused by

不支持的转换类型:java.time.LocalDateTime
谁不支持?mysql-connector-java 不支持!
那 mysql-connector-java 哪个版本支持了,答案是:5.1.37

升级mysql-connector-java

将 mysql-connector-java 升级到 5.1.37 ,再执行下 com.qsl.OrderTest#orderListAllTest

不再报异常,查询结果也正确
MyBatis-Plus 替换 Mybatis 似乎就完成了
顺的让人有点怀疑

Conversion not supported for type java.time.LocalDateTime

我们再回过头去看看前面说到的异常:Conversion not supported for type java.time.LocalDateTime
Mybatis-Plus 替换 MyBatis 之前没这个异常,替换之后就有了这个异常,这不是 Mybatis-Plus 的问题?
如何找这个异常的根因了?
很简单,直接从异常堆栈入手

点了之后,你会发现方法很简单

这么简单的代码能有什么问题?
大家注意看图中左上角 MyBatis 的版本,是 3.5.1,并不是最初的 3.5.0
有小搭档可能会问了:不是用 MyBatis-Plus 替换了 MyBatis 吗,怎么尚有 Mybatis ?
这个问题问的真的好,我只想给你个大嘴巴子

你看下 MyBatis-Plus 的官方阐明

既然基于 Mybatis 3.5.0 没有抛异常,而基于 3.5.1 抛了异常, LocalDateTimeTypeHandler 在 3.5.1 肯定做了调解
我们来看下调解了什么?

看出什么了?
MyBatis 3.5.0 会处理 LocalDateTime 类型的转换(将 java.sql.Timestamp 转换成 java.time.LocalDateTime )
然而,注意了,然而来了!!!
然而从  MyBatis 3.5.1 开始,不再处理 LocalDateTime (还包括:LocalDate 、 LocalTime )类型的转换
而是交由 JDBC 组件,也就是 mysql-connector-java 来实现
而巧的是, mysql-connector-java 5.1.26 不支持类型 LocalDateTime

那它支持哪些类型了?
我们同样从异常堆栈入手

点了之后,可以看到下图
  1. public <T> T getObject(int columnIndex, Class<T> type) throws SQLException {
  2.     if (type == null) {
  3.         throw SQLError.createSQLException("Type parameter can not be null",
  4.                 SQLError.SQL_STATE_ILLEGAL_ARGUMENT, getExceptionInterceptor());
  5.     }
  6.     if (type.equals(String.class)) {
  7.         return (T) getString(columnIndex);
  8.     } else if (type.equals(BigDecimal.class)) {
  9.         return (T) getBigDecimal(columnIndex);
  10.     } else if (type.equals(Boolean.class) || type.equals(Boolean.TYPE)) {
  11.         return (T) Boolean.valueOf(getBoolean(columnIndex));
  12.     } else if (type.equals(Integer.class) || type.equals(Integer.TYPE)) {
  13.         return (T) Integer.valueOf(getInt(columnIndex));
  14.     } else if (type.equals(Long.class) || type.equals(Long.TYPE)) {
  15.         return (T) Long.valueOf(getLong(columnIndex));
  16.     } else if (type.equals(Float.class) || type.equals(Float.TYPE)) {
  17.         return (T) Float.valueOf(getFloat(columnIndex));
  18.     } else if (type.equals(Double.class) || type.equals(Double.TYPE)) {
  19.         return (T) Double.valueOf(getDouble(columnIndex));
  20.     } else if (type.equals(byte[].class)) {
  21.         return (T) getBytes(columnIndex);
  22.     } else if (type.equals(java.sql.Date.class)) {
  23.         return (T) getDate(columnIndex);
  24.     } else if (type.equals(Time.class)) {
  25.         return (T) getTime(columnIndex);
  26.     } else if (type.equals(Timestamp.class)) {
  27.         return (T) getTimestamp(columnIndex);
  28.     } else if (type.equals(Clob.class)) {
  29.         return (T) getClob(columnIndex);
  30.     } else if (type.equals(Blob.class)) {
  31.         return (T) getBlob(columnIndex);
  32.     } else if (type.equals(Array.class)) {
  33.         return (T) getArray(columnIndex);
  34.     } else if (type.equals(Ref.class)) {
  35.         return (T) getRef(columnIndex);
  36.     } else if (type.equals(URL.class)) {
  37.         return (T) getURL(columnIndex);
  38. //        } else if (type.equals(Struct.class)) {
  39. //
  40. //            }
  41. //        } else if (type.equals(RowId.class)) {
  42. //
  43. //        } else if (type.equals(NClob.class)) {
  44. //
  45. //        } else if (type.equals(SQLXML.class)) {
  46.     } else {
  47.         if (this.connection.getAutoDeserialize()) {
  48.             try {
  49.                 return (T) getObject(columnIndex);
  50.             } catch (ClassCastException cce) {
  51.                 SQLException sqlEx = SQLError.createSQLException("Conversion not supported for type " + type.getName(),
  52.                         SQLError.SQL_STATE_ILLEGAL_ARGUMENT, getExceptionInterceptor());
  53.                 sqlEx.initCause(cce);
  54.                 throw sqlEx;
  55.             }
  56.         }
  57.         throw SQLError.createSQLException("Conversion not supported for type " + type.getName(),
  58.                 SQLError.SQL_STATE_ILLEGAL_ARGUMENT, getExceptionInterceptor());
  59.     }
  60. }
复制代码
确实没有 LocalDateTime 、 LocalDate 和 LocalTime
mysql-connector-java 5.1.37 开始支持 LocalDateTime 、 LocalDate 和 LocalTime ,前面已经介绍过了,不再过多赘述
总结下异常根因:MyBatis 3.5.1 开始不再处理 LocalDateTime 、 LocalDate 和 LocalTime 的转换,而 mysql-connector-java 5.1.37 之前都不支持这些类型
弄清楚这个异常的来龙去脉之后,顺的是不是又天经地义一些了?
暴风雨的来临

版本上线没 2 天,该来的终究照旧来了
我们往表 tbl_order 中插入一条记录:INSERT INTO tbl_order VALUES (3, 'asdfgh', NULL, '2024-02-21 20:01:31.111', '2024-02-21 20:02:56.764');
再执行 com.qsl.OrderTest#orderListAllTest

现在我就想问 "小伙" :刺不刺激?

碰到了异常,那就找原因
同样从异常堆栈入手

看出什么了?
如果 getTimestamp(columnIndex) 得到的是 NULL ,不就 NullPointerException ?严谨性了?
修复问题要紧,我们先看哪个版本进行修复了?

将 mysql-connector-java 升级到 5.1.42

问题得以修复
经此一役, "小伙" 似乎成长了许多,但眼里的光却暗淡了不少
mybatis-plus-issues-1114

无意中看到了这个issue-1114,跟我们前面分析的 Conversion not supported for type java.time.LocalDateTime 是不是同一个问题?
只是我们用到的数据库连接池是默认的 HikariCP 而非 Druid
联合druid/issues/3302来看,如果利用 Druid 作为数据库连接池,出现的异常可能跟我们前面分析的确实不一样
所以大家需要根据自己的实际情况来分析,但针对异常的分析方法是通用的
修了“不应修的Bug”

这是我亲身经历的一次事故,到现在都觉得这锅背的有点冤

背景介绍

文件分为主文件和附属文件,主文件生成之后再生成附属文件
附属文件生成的时候,会校验其依赖的主文件是否都生成了,如果有任意一个主文件未生成,依赖文件不能生成并抛出异常
这个业务照旧比较简单吧
但在附属文件校验的优化上,我背上了生产事故

优化前的校验


listFileGenerateLog 作用是根据参数查询文件生成记录,具体实现不用关注
这个校验逻辑是什么?只要有任意一个主文件生成,校验就算通过了,与业务要求(主文件全部生成,才算校验通过)不匹配呀
这不是妥妥的 Bug ?

优化后的校验

碰到 Bug 你能忍?我是忍不了一点,反手就是一个优化

这是不是就符合业务要求了?
生产异常

中午升级之后,稳定运行了一段时间,期间文件正常生成,没出现任何问题
晚上 19 点,有个附属文件生成失败,异常提示:依赖的资源[abc_{yyyyMMdd}.txt]未生成
当时看到这个异常的第一眼,觉得既熟悉又陌生,熟悉的是这个异常信息的结构,陌生的是 abc_{yyyyMMdd}.txt ,这不是文件名吗?
正常来讲应该是 fileId ,是一个自增的正整数呀,怎么会是文件名了?
脑中刹时闪过一个动机:数据库数据有问题?
一查吓一跳,这个附属文件关联主文件的字段值是:4356,abc_{yyyyMMdd}.txt ,看终极修改时间是:2021-08-21 15:22:12.652
4356 文件的文件名就是 abc_{yyyyMMdd}.txt ,正常来讲,这个关联字段的值应该是:4356
敢情这个 校验Bug 完美的兼容了这个脏数据 ,所以几年了,一直没出现异常

是不是有这味了?
这可倒好,我把 Bug 修好,还出现问题了,你说我是不是手贱?
经此一役,我眼里的光又暗淡了些许
总结

关于对组件的升级,或者对旧代码的调解,都有可能牵一发动全身,影响甚大
我的观点是:能不动就不要动,改好没绩效,改出问题要背锅,吃力不讨好,又不是不能跑

如果到了不得不改的地步了,那就需要全面的测试。
更多文章保举:
1.Spring Boot 3.x 教程,太全了!
2.2,000+ 道 Java面试题及答案整理(2024最新版)
3.免费获取 IDEA 激活码的 7 种方式(2024最新版)
觉得不错,别忘了随手点赞+转发哦!

免责声明:如果侵犯了您的权益,请联系站长,我们会及时删除侵权内容,谢谢合作!更多信息从访问主页:qidao123.com:ToB企服之家,中国第一个企服评测及商务社交产业平台。

本帖子中包含更多资源

您需要 登录 才可以下载或查看,没有账号?立即注册

x
回复

使用道具 举报

0 个回复

正序浏览

快速回复

您需要登录后才可以回帖 登录 or 立即注册

本版积分规则

笑看天下无敌手

金牌会员
这个人很懒什么都没写!
快速回复 返回顶部 返回列表