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

Manually Review Vnode Rewrite #103

Open
24 tasks
albsch opened this issue Jul 19, 2021 · 0 comments
Open
24 tasks

Manually Review Vnode Rewrite #103

albsch opened this issue Jul 19, 2021 · 0 comments
Labels
architecture Big changes concerning most of the code base deprecated Fix deprecated things good first issue Good for newcomers help wanted Extra attention is needed
Milestone
Up-to-date riak_core

Comments

@albsch
Copy link
Member

albsch commented Jul 19, 2021

In #80 gen_fsm was replaced by gen_statem. It's still buggy and needs more tests and verification.

Manually review and check each rewritten function:

  • sync_send_event
    • #1
  • send_event
    • #2.1
    • #2.2
    • #2.3
    • #2.4
    • #2.5
    • #2.6
    • #2.7
  • sync_send_all_state_event
    • #3.1
    • #3.2
    • #3.3
  • send_all_state_event
    • #4.1
    • #4.2
    • #4.3
    • #4.4
    • #4.5
    • #4.6
    • #4.7
  • send_event_after
    • #5
@albsch albsch added help wanted Extra attention is needed good first issue Good for newcomers deprecated Fix deprecated things architecture Big changes concerning most of the code base labels Jul 19, 2021
@albsch albsch added this to the Up-to-date riak_core milestone Jul 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
architecture Big changes concerning most of the code base deprecated Fix deprecated things good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
Up-to-date riak_core
Development

No branches or pull requests
1 participant
@albsch