Unnecessary subquery in DeptMapper.list

[wtune]

The following query defined in DeptMapper.xml is not optimal:

    <select id="list" resultType="map">
        select
...
        from sys_dept where 1 = 1
...
        <if test="deptId != null and deptId != ''">
            and (dept_id = #{deptId} or dept_id in ( select dept_id from sys_dept where pids like CONCAT('%$[', #{deptId}, '$]%') escape '$' ))
        </if>
        order by sort ASC
    </select>

The sub-query could be eliminated and changes to a predicate on pids.

and (dept_id = #{deptId} or pids like CONCAT('%$[', #{deptId}, '$]%') escape '$')