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 Redis Connection Ping Bug #107

Merged
merged 1 commit into from
Jun 27, 2018
Merged

fix Redis Connection Ping Bug #107

merged 1 commit into from
Jun 27, 2018

Conversation

huangzhhui
Copy link
Contributor

$this->connection->ping() will return 0 when connection well, will return false when connection lost, probably only return a value but not throw an exception in coroutine client

@huangzhhui huangzhhui merged commit 0ecd931 into master Jun 27, 2018
huangzhhui added a commit to swoft-cloud/swoft-redis that referenced this pull request Jun 27, 2018
@huangzhhui
Update AbstractRedisConnection.php (swoft-cloud/swoft-component#107)
9b54860
@huangzhhui huangzhhui deleted the fixRedisConnectionPingBug branch July 10, 2018 10:14
huangzhhui added a commit that referenced this pull request Jul 10, 2018
@huangzhhui
Squashed 'src/redis/' changes from e0adf8ba..9b54860c
e9a4d31 
9b54860c Update AbstractRedisConnection.php (#107)
ed06c9f0 some update for redis (#89)
0fec662d Add Redis Method setex & setnx (#85)
2d8a0bd3  add coWrite(); use swoft/swoole-ide-helper (#83)

git-subtree-dir: src/redis
git-subtree-split: 9b54860c8d972e35851fb445abb768bc6fb6fde9
swoft-bot pushed a commit to swoft-cloud/swoft-view that referenced this pull request Jan 25, 2019
@huangzhhui
Update AbstractRedisConnection.php (swoft-cloud/swoft-component#107)
53d3a9c
stelin pushed a commit to swoft-cloud/swoft-redis that referenced this pull request May 28, 2019
@huangzhhui
Update AbstractRedisConnection.php (swoft-cloud/swoft-component#107)
664f245
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.
1 participant
@huangzhhui