You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@brpc.apache.org by GitBox <gi...@apache.org> on 2019/12/01 03:03:22 UTC
[GitHub] [incubator-brpc] guoyuhong commented on issue #979: Add LZ4
compression method
guoyuhong commented on issue #979: Add LZ4 compression method
URL: https://github.com/apache/incubator-brpc/pull/979#issuecomment-560045429
@jamesge I checked the streaming/framing interfaces in lz4. It looks like that the output buffer may still be cut off by `butil::IOBuf`, so we still need heap allocation. In current implementation, the var-int encode may need only 2 bytes and it still need handle the case cross multiple `butil::IOBuf`s.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@brpc.apache.org
For additional commands, e-mail: dev-help@brpc.apache.org