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.

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

fix issue 210：在最终结果为单库单表时，去掉新增的查询字段 #211

Merged
merged 2 commits into from
Dec 20, 2016
Merged

fix issue 210：在最终结果为单库单表时，去掉新增的查询字段 #211

merged 2 commits into from
Dec 20, 2016

Conversation

StormAll
Copy link

大致修改思路：
1、在SQLBuilder中对SQL片段增加是否是额外增加的标识
2、在SQLRouteEngine的routeSQL方法，根据路由结果，处理额外增加的SQL片段

@coveralls
Copy link

coveralls commented Dec 20, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-0.006%) to 93.734% when pulling 0799b8f on StormAll:210 into ab78772 on dangdangdotcom:master.

@coveralls
Copy link

coveralls commented Dec 20, 2016 •
edited
Loading

Coverage Status

Coverage increased (+0.02%) to 93.759% when pulling 3dab47e on StormAll:210 into ab78772 on dangdangdotcom:master.

@hanahmily hanahmily merged commit 3dab47e into apache:master Dec 20, 2016
@sunOnly sunOnly mentioned this pull request Oct 13, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@StormAll @coveralls @hanahmily