yoav-steinberg · GitHub

added 2 commits

June 8, 2021 17:36

@yoav-steinberg

@oranagra oranagra changed the title Remove gopher. See #8989. Remove gopher protocol support.

Jun 9, 2021

oranagra

madolson

@oranagra

Closed

JackieXie168 pushed a commit to JackieXie168/redis that referenced this pull request

Sep 8, 2021
Gopher support was added mainly because it was simple (trivial to add).
But apparently even something that was trivial at the time, does cause complications
down the line when adding more features.
We recently ran into a few issues with io-threads conflicting with the gopher support.
We had to either complicate the code further in order to solve them, or drop gopher.
AFAIK it's completely unused, so we wanna chuck it, rather than keep supporting it.

Read the original on github.com ↗