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

Bug: if one salve instance run "cluster failover" in redis-cluster, then all operations on this node will fail #5

Open
mwang-sticky opened this issue Jun 7, 2016 · 2 comments

Comments

@mwang-sticky
Copy link

I simply fix it by:
563 int CRedisCommand::CmdRequest(redisContext pContext)
564 {
565 if (m_nArgs <= 0)
566 return RC_PARAM_ERR;
567
568 if (!pContext)
569 return RC_RQST_ERR;
570
571 if (m_pReply)
572 {
573 freeReplyObject(m_pReply);
574 m_pReply = nullptr;
575 }
576
577 m_pReply = static_cast<redisReply *>(redisCommandArgv(pContext, m_nArgs, (const char *
)m_pszArgs, (const size_t *)m_pnArgsLen));
+578 return m_pReply ? (m_pReply->type != REDIS_REPLY_ERROR ? RC_SUCCESS : RC_RQST_ERR) : RC_RQST_ERR;
579 }

Please tell me if any errors

@shawn246
Copy link
Owner

thank you so much , i will check this later

@mwang-sticky
Copy link
Author

I though is there any API to check redis cluster status? like "cluster
info"?

2016-06-16 18:12 GMT+08:00 shawn notifications@github.com:

thank you so much , i will check this later


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#5 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ASqBphCFiHQwKiajTXjjzpMqeL8EKJBPks5qMSGcgaJpZM4IwQB6
.

Thanks!
BestRegards!
Min

Wang

School of EE&CS, Peking University
Email: mwang98@gmail.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

2 participants