Andy Balholm
Andy Balholm
I'm glad to see it's being added, but we already found another solution to our problem, so I can't really test it at this point.
In the brotli format, there is a bit in the metablock header that is reserved for future use. It's currently required to be a zero; if there is a one...
Brotli uses a pre-defined static dictionary. I don't think there is any support for using user-generated dictionaries like zstd does.
Oh, now I see that a shared dictionary feature was added a while back. My Go version of the library doesn't have any support for it.
It was generated with c2go, but changes have been made since. We can't just regenerate it from the C version now. I don't have any plans to add it, but...
I don't think the classification service uses either of those timeouts, because it uses http.DefaultClient. You could set up an http.Client that uses a dialer with a custom timeout, but...
Adding a dockerfile sounds like a good idea. It should probably go in the `startup` directory along with the init scripts. It would be good to provide some documentation (maybe...
I think the current behavior makes about as much sense as is possible for a basically meaningless operation like reading 0 bytes.
How does the speed change if you adjust the compression level?
It sounds like the issue is something other than the brotli compression itself. What brotli middleware are you using?