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.

Sign up for GitHub

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

Jump to bottom

Update SqlHelper.java #577

Open

stotem wants to merge 1 commit into abel533:master from stotem:stotem-patch-1

stotem commented Apr 19, 2019

既然都是通过主键精准获取数据，肯定是想要无论是否被逻辑删除都需要查询（典型应用场景为交易快照数据的展示），所以，建议取消掉通过主键查询时加入逻辑删除做为查询条件


          Update SqlHelper.java

既然都是通过主键精准获取数据，肯定是想要无论是否被逻辑删除都需要查询（典型应用场景为交易快照数据的展示），所以，建议取消掉通过主键查询时加入逻辑删除做为查询条件

Owner

abel533 commented Oct 16, 2022

不应该简单的去掉...尤其是2022年才处理这个PR。。考虑增加参数控制，暂时先不处理。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment