Skip to content

Replace archive/zip with klauspost/compress/zip

Arran Walker requested to merge ajwalker/cache-perf/compress into master

What does this MR do?

Replaces archive/zip with github.com/klauspost/compress/zip. This is a drop-in replacement.

We're already using github.com/klauspost/compress for our gzip artifacts.

This MR contains two commits:

  • Replace compress/zip with github.com/klauspost/compress/zip
  • Update github.com/klauspost/compress to include the latest flate performance improvements.

Why was this MR needed?

klauspost's deflate library is far more efficient than the standard Go library.

Are there points in the code the reviewer needs to double check?

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Added tests for this feature/bug
  • In case of conflicts with master - branch was rebased

What are the relevant issue numbers?

Closes #26903 (closed)

&3763 (closed)

Edited by Arran Walker

Merge request reports