AnDiXL commented on PR #3196:
URL: https://github.com/apache/brpc/pull/3196#issuecomment-5749380770

   > @Q1ngbo 我们最近基于 #3196 和 #3197 的 FlatBuffers 支持代码进行了进一步验证和完善。 目前补充的工作主要包括:
   > 
   > * 修复 Message、MessageBuilder 和 allocator 移动过程中的内存所有权问题;
   > * 增加消息生命周期、移动语义、非法 FlatBuffers 数据校验等单元测试;
   > * 增加完整 RPC 往返、非法请求拒绝以及拒绝后服务继续可用的回归测试;
   > * 将 FlatBuffers 支持整理为默认关闭的可选功能,并验证 CMake、Make、Bazel 以及未安装 FlatBuffers 
时的构建兼容性;
   > * 将代码更新并适配到较新的 master。
   >   为了降低评审复杂度,我们计划把完善后的代码拆分成三个较小的 PR:
   > 
   > 1. IOBuf-backed FlatBuffers Message / MessageBuilder 基础能力;
   > 2. FlatBuffers RPC 协议及 Channel、Controller、Server 集成;
   > 3. 示例、文档和剩余工程化完善。
   > 
   > 我们想先征求一下你的意见:你是否希望我们一起更新现有 PR,还是可以接受基于现有实现重新整理并提交一组便于评审的新 PR?
   
    Hi,due to some previous issues, we have been delayed. Over the next couple 
of days, we will reorganize and submit a new set of PRs that are more suitable 
for review.
    Thanks.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to