-
Notifications
You must be signed in to change notification settings - Fork 876
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
fix: resolves the crash with MGET and duplicate keys #2471
Conversation
Fixes #2465 Signed-off-by: Roman Gershman <roman@dragonflydb.io>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Super nits, feel free to ignore
string key_arr; | ||
for (unsigned j = 0; j < keys.size(); ++j) { | ||
absl::StrAppend(&key_arr, keys[j], ","); | ||
} | ||
key_arr.pop_back(); | ||
LOG(WARNING) << "The keys requested are: [" << key_arr << "]"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
string key_arr; | |
for (unsigned j = 0; j < keys.size(); ++j) { | |
absl::StrAppend(&key_arr, keys[j], ","); | |
} | |
key_arr.pop_back(); | |
LOG(WARNING) << "The keys requested are: [" << key_arr << "]"; | |
LOG(WARNING) << "The keys requested are: [" << absl::StrJoin(keys, ',') << "]"; |
// bump up the key again, and the first iterator will be invalidated. | ||
// TODO: to understand better the dynamics of this scenario and to fix it. | ||
if (it->first != keys[i]) { | ||
LOG(WARNING) << "Inconcistent key(" << i << "), expected " << keys[i] << " but found " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LOG(WARNING) << "Inconcistent key(" << i << "), expected " << keys[i] << " but found " | |
LOG(WARNING) << "Inconsistent key(" << i << "), expected " << keys[i] << " but found " |
Fixes #2465 Signed-off-by: Roman Gershman <roman@dragonflydb.io>
This reverts commit f69f2ec.
Fixes #2465