open-hand / choerodon-starters

This is the toolkit developed by Choerodon and provides some basic dependencies for use in the development process.
http://choerodon.io
Apache License 2.0
78 stars 64 forks source link

[0.6.4] MySQL中在SQL中存在排序时分页SQL错误 #9

Closed RavenCM closed 5 years ago

RavenCM commented 5 years ago

如果使用@io.choerodon.mybatis.pagehelper.annotation.SortDefault添加默认排序,在SQL中又拼写了order by语句时,最终执行的SQL会有两个order by。SQL Server不存在这个问题。

superlee007 commented 5 years ago

xml里面有order by的话,你再使用动态拼order by肯定有问题,sqlserver那种情况是同一份代码吗?如果是的话应该是jsqlparser拿到sql把order by截掉了

superlee007 commented 5 years ago

关于mysql和oracle的情况,后面会加上sql校验的,如果你xml里面写的有默认order by再用doPageAndSort方法抛异常,而不是直接放到数据库里面跑sql错误

RavenCM commented 5 years ago

是同一份代码,jsqlparser没有截掉,保留了这个排序。我说一下我这边的问题吧。。。

  1. 首先是一个SQL,简单的SELECT ... FROM ... JOIN ... WHERE,并且指定了默认排序是一个ID字段,在MySQL中没有问题。迁移到SqlServer中之后,报错:Ambiguous column name 'xxx'.。我查看原因的时候发现是SqlServerParser中默认的这个排序字段没有带上表别名,但是查询列中的字段是带表别名的,所以他认为这两个列不同,将这个不带表别名的列自动添加到了查询列中,也就是issue #7

  2. 解决上一个问题的时候,我发现如果我在自己的SQL中指定一个ORDER BY字段,这个字段是带表别名的(也是默认排序字段,我没有尝试其他字段),那么最终就能够正常执行。默认的不带表别名的排序字段并没有生效

  3. 当我添加排序之后,又切换回到MySQL中的时候,又出现issue #9所说的问题,所以我希望能够实现的效果应该和SQL Server中一样,解析到又ORDER BY字段的时候,不要添加默认的排序列。

superlee007 commented 5 years ago

首先,如果你xml里面自定义的sql有order by了,就不能用doPageAndSort方法了。如果你要自己做动态order by ,在xml里写${column}然后传参传过去就行,然后你只用分页doPage方法就行了。你用doPageAndSort,这个方法会拿你xml的sql先拼order by然后在拼分页参数。你原始sql有order by肯定是报错的,目前没做处理就报sql错误,如果我这边加上判断,也是抛个exception,不让这么用的