【踩坑记录】数据库连接未关闭连接，释放资源的坑

今天不聊需求，在代码review时，看到这么一段代码。大家来看一下，这段数据库代码可能引发什么问题？

def execute(self, sql):
    """执行SQL语句方法"""
    try:
        conn = self.client.connection()
        cursor = conn.cursor()
        cursor.execute(sql)
        result = cur.fetchall()
        cur.close()
        conn.close()
        return result
    except Exception as e:
        log.error(e)
        raise e

把关闭数据库连接写在try里面，要是在查询的时候异常了，连接和游标能释放么？

是否该连接会长期占用系统资源直到被超时回收？

在限定连接池大小的条件下，如果这种查询SQL异常过多，超过了连接池限定大小，是否会导致其他正常查询无法执行？

相信这是大部分开发都容易踩的坑，为什么这么说呢？现在有很多完善的框架，帮我们做好了ORM，大部分开发长期依赖框架，业务写的很优秀但却把基础丢了。

下面是个人认为没什么问题的版本，欢迎指教。

try:
    conn = self.client.connection()
    cur = conn.cursor()
    cur.execute(sql)
    result = cur.fetchall()
except Exception as e:
    log.error(e)
    raise e
else:
    return result
finally:
    if cur:
        cur.close()
    if conn:
        conn.close()