Encoder reset (for c2go translated code) #308

New Issue

claunia · 2026-01-29T20:41:44Z

claunia commented

2026-01-29 20:41:44 +00:00

Originally created by @CAFxX on GitHub (Feb 4, 2020).

In #679 it was noted that native Brotli is currently missing a way to reset an encoder. We are currently using a c2go translation of native brotli (to avoid cgo, native dependencies and all the pain that entails), and that means that all memory used by brotli is allocated on the Go heap, and is therefore subject to GC.

Unfortunately, because native brotli is missing the encoder reset, the GC overhead becomes significant as there is no effective way to reuse instantiated encoders.

Our use-case (a custom HTTP reverse-proxy) does not change encoding parameters between uses, so it would not be required to efficiently handle the case in which encoding parameters change.

Originally created by @CAFxX on GitHub (Feb 4, 2020). In #679 it was noted that native Brotli is currently missing a way to reset an encoder. We are currently using a c2go translation of native brotli (to avoid cgo, native dependencies and all the pain that entails), and that means that all memory used by brotli is allocated on the Go heap, and is therefore subject to GC. Unfortunately, because native brotli is missing the encoder reset, the GC overhead becomes significant as there is no effective way to reuse instantiated encoders. Our use-case (a custom HTTP reverse-proxy) does not change encoding parameters between uses, so it would not be required to efficiently handle the case in which encoding parameters change.

claunia referenced this issue

2026-01-29 20:51:34 +00:00

[PR #308] [MERGED] Make memmove16 safe. #904

claunia referenced this issue

2026-01-29 20:51:35 +00:00

[PR #308] Make memmove16 safe. #908

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/brotli#308