apache / apache/brpc

PartitionChannel和DynamicPartitionChannel支持thrift

Open
#2,236 0 comments 1 reaction 0 assignees View on GitHub
feature
Dominant language
C++
Stars
17.6k
Forks
4.1k
Avg merge
2d 12h
Merged PRs (30d)
69

Description

**Is your feature request related to a problem? (你需要的功能是否与某个问题有关?)**
PartitionChannel和DynamicPartitionChannel支持thrift。
我使用上面两种channle,但是会报错:ThriftFramedMessage does not support MergeFrom,
1、然后我根据文档准备重写ResponseMerger的Merge接口,但是发现只提供了protobuf版本的接口。
2、DynamicPartitionChannel的AddChannel并没有加载用户实现的ResponseMerger,只有PartitionChannel实现了。

**Describe the solution you'd like (描述你期望的解决方法)**
PartitionChannel和DynamicPartitionChannel支持thrift。

**Describe alternatives you've considered (描述你想到的折衷方案)**
提供接口允许自定义ResponseMerger,并且在DynamicPartitionChannel生效。

**Additional context/screenshots (更多上下文/截图)**

Contributor guide

Open the contributing guide

Research direction

Start by reading the documented ResponseMerger interface and the implementations of PartitionChannel and DynamicPartitionChannel, focusing on AddChannel and the ThriftFramedMessage error. The work is complete when both channel types support thrift and DynamicPartitionChannel applies a user-provided ResponseMerger, including the required merger interface changes.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
api, backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.