apache / apache/brpc

Remove implementation specific function name of EventDispatcher

Open
#1,894 3 comments 0 reactions 0 assignees View on GitHub
good first issue refactor
Dominant language
C++
Stars
17.6k
Forks
4.1k
Avg merge
2d 12h
Merged PRs (30d)
69

Description

Some of the member functions of EventDispatcher are implementation specific, for example AddEpollOut.

We need to change that specific function name and adopt more general name then we can easily support brpc to more platforms.

Contributor guide

Open the contributing guide

Research direction

Locate EventDispatcher and inspect the implementation-specific member functions, starting with AddEpollOut. Inventory the names that prevent support for more platforms and determine the general naming needed; done means the dispatcher uses platform-neutral names and can support brpc on more platforms.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend, distributed-systems
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.