Open
!69 使用 close 方法来统一关闭数据库连接

Alan Yeh:3.0 闲.大赋:3.0

Alan Yeh Created on: 2020-09-20 11:36
Reviewer: 29 xiandafu 1578913709

initDefaultSchema 在获取 Connection 后,是直接关闭了这条 Connection,我觉得应该统一使用 close() 方法来关闭这条 Connection。

由于业务需要,我这边对 BeetlSql 进行深入定制,需要同时支持多数据库、每个数据库都支持读写分离。因此我在自己的框架下去保存了多个 SQLManager。由于在我的框架下, SQLManager 是延迟初始化的,SQLManager 在初始化的时候会去初始化 schema,初始化之后直接将这条 Connection 关闭了,导致该事务后续的读写都发生了异常。

我在是 2.x 版本发现这个问题,看到 3.0 还存在这个问题,所以发起个 Pull Request 修复该问题

0 comments, 1 participants 458191 alan yeh 1578924482

Check whether the system can automatically merge this Pull Request...
The Pull Request cannot be automatically merged.
The Pull Request cannot be automatically merged.
Gitee trying to handle conflicts...
Gitee handled conflicts timeout, please fetch to the local handling
The current branch merge status has been updated, continue to operate after refreshing the page
The Pull Request can be automatically merged.
Merging Pull Request. Wait for a moment, the page will refresh automatically.