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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
