Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

查询limit改写有问题[ bug ] #647

Closed
fancy-lee opened this issue Mar 2, 2020 · 1 comment
Closed

查询limit改写有问题[ bug ] #647

fancy-lee opened this issue Mar 2, 2020 · 1 comment

Comments

@fancy-lee
Copy link
Contributor

fancy-lee commented Mar 2, 2020

在提交 issue 前,请查阅以下资源,请先进行搜索来保证没有类似的 issue。
文档 | FAQ

重现步骤

select * from empower_0 pcr order by id limit 100,200
改写后成了
select * from empower_0 pcr order by id limit 100;

版本信息

应用版本/分支:

部署方式:手工部署
应用版本/分支:Release v1.7.2

@fancy-lee fancy-lee reopened this Mar 2, 2020
@fancy-lee
Copy link
Contributor Author

刚才看了一下代码,好像是故意这样设置的,可能是出于安全,不过感觉不太合理,我们的业务还是有offset的需求

mxlxm added a commit to mxlxm/Archery that referenced this issue Aug 29, 2021
@mxlxm mxlxm mentioned this issue Aug 29, 2021
hhyo added a commit that referenced this issue Sep 16, 2021
* support offset
fix #847,#739,#647.

* update limit ut

Co-authored-by: 小圈圈 <rtttte@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant