GitHub user wwbmmm edited a comment on the discussion: LZ4性能比snappy好,而且我看很多年前就有人尝试在brpc中支持lz4(但被closed/或无人推进)中,这是为什么?
看了下pr历史的[4个尝试合并lz4的PR](https://github.com/search?q=repo%3Aapache%2Fbrpc+lz4&type=pullrequests),都有brpc维护人员的review问题尚未解决,因此没有合入。主要是代码细节问题,并非有什么顾虑。 我们欢迎关于lz4的PR,如果您感兴趣,可以考虑在之前PR的基础上,解决相应review问题之后提交一个新的PR。 GitHub link: https://github.com/apache/brpc/discussions/3221#discussioncomment-15811578 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
