feat: Use klauspost/compress for gzip#14
Conversation
d72ad35 to
5f304cb
Compare
5f304cb to
7e7b1a2
Compare
|
Hello @klauspost,I copied https://github.com/cloudwego/hertz/tree/develop/pkg/common/compress over and changed the gzip dependency, but the original single test was not used. Hope you can help me look at the unit test when you have time, thank you very much. |
|
@li-jin-gou The test seems to expect the gzip package to produce a specific compressed output. That is a wrong expectation. While you can change the expected output, it will break again. So either A) check the decompressed output or B) Check that the size isn't 5% bigger than you expect. Exact compressed size will change over time as the algorithm is improved. |
Thanks |
What type of PR is this?
feat
What this PR does / why we need it (English/Chinese):
Which issue(s) this PR fixes: