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

Raft cache persist improve #5

Merged
merged 6 commits into from
Mar 1, 2019
Merged

Conversation

flowbehappy
Copy link
Contributor

@flowbehappy flowbehappy commented Feb 26, 2019

dbms/src/Common/ErrorCodes.cpp Outdated Show resolved Hide resolved

template <bool is_set, typename T, template <typename E = T, typename...> class Container, class Write, class Read>
struct PersistedContainerSetOrVector
template <bool is_map, bool is_set, class Trait>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe we can make abstraction in the concept of Iterator later if have time.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The problem is map use K and V, while others use T only. I can unify map by std::pair<K, V>, but not user friendly.

@ilovesoup ilovesoup changed the title Raft cache persit improve Raft cache persist improve Feb 27, 2019
@flowbehappy flowbehappy merged commit d9c4906 into raft Mar 1, 2019
@solotzg solotzg deleted the raft-cache-persit-improve branch July 10, 2019 05:31
windtalker added a commit to windtalker/tiflash that referenced this pull request Mar 6, 2023
Signed-off-by: xufei <xufei@pingcap.com>
JinheLin pushed a commit to JinheLin/tiflash-1 that referenced this pull request Apr 2, 2024
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

Successfully merging this pull request may close these issues.

2 participants