apache / apache/brpc

Support server protocol parsing order without changing ProtocolType(options.proto)

Open
#990 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C++
Stars
17.6k
Forks
4.1k
Avg merge
2d 12h
Merged PRs (30d)
69

Description

One possible plan: define an order for every protocol,
```
TypeOrder[] = {
{PROTOCOL_BAIDU_STD , 100} ,
{PROTOCOL_STREAMING_RPC, 200},
}
```
and sort the protocol in handlers according to the number that followed. The interval between two protocol should be large enough to adapt to more protocols in future.

Contributor guide

Open the contributing guide

Research direction

Start by reading the protocol handlers and the current ProtocolType(options.proto) parsing path. Evaluate the proposed TypeOrder mapping and how protocol order should be applied without changing that conversion. Done means protocol parsing follows the defined order while preserving the existing ProtocolType(options.proto) behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend-api-design, distributed-systems, networking
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.