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

修复key缓存错误的问题 #14

Merged
merged 5 commits into from
Jul 2, 2021
Merged

修复key缓存错误的问题 #14

merged 5 commits into from
Jul 2, 2021

Conversation

q629988171
Copy link
Contributor

  1. 修复连接默认数据库错误的问题。
  2. 根据errcode存在且不为0判断为失败，否则为成功。而errmsg仅作参考，后续可能会有变动，因此不可作为是否调用成功的判据。
  3. 修复判断查询key结果的逻辑错误。
  4. 修复key缓存重复写入的问题。

q629988171 added 5 commits June 4, 2021 22:27
@q629988171
Merge pull request #1 from easychen/main
13dbee3 
同步更新
@q629988171
Merge pull request #2 from easychen/main
44ca537 
同步更新
@q629988171
Merge pull request #3 from easychen/main
59dfd75 
同步更新
@q629988171
Merge pull request #4 from easychen/main
dc627cb 
同步更新
@q629988171
修复key缓存错误的问题
e1d340d 
1. 修复连接默认数据库错误的问题。
2. 根据errcode存在且不为0判断为失败，否则为成功。而errmsg仅作参考，后续可能会有变动，因此不可作为是否调用成功的判据。
3. 修复判断查询数据库结果的逻辑错误。
4. 修复key缓存重复写入的问题。
@easychen easychen merged commit 98ac546 into easychen:main Jul 2, 2021
q629988171 added a commit to q629988171/wecomchan that referenced this pull request Jul 3, 2021
@q629988171
Merge pull request #5 from easychen/main
caf38eb 
Merge pull request easychen#14 from q629988171/main
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.
2 participants
@q629988171 @easychen