-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
protocole 3 (RESP3) outputs debug info with pubsub #111
Comments
jfgaudreault-p
changed the title
protocole 3 outputs debug info with pubsub
protocole 3 (RESP3) outputs debug info with pubsub
Oct 13, 2024
Yep, it looks like the resp3 handling code has some extra "log.info" calls that are missing in the resp2 code. |
aiven-sal
added a commit
that referenced
this issue
Oct 15, 2024
Close #111 Signed-off-by: Salvatore Mesoraca <salvatore.mesoraca@aiven.io>
mkmkme
pushed a commit
that referenced
this issue
Oct 16, 2024
Close #111 Signed-off-by: Salvatore Mesoraca <salvatore.mesoraca@aiven.io>
Rafiot
pushed a commit
to Rafiot/valkey-py
that referenced
this issue
Dec 3, 2024
Close valkey-io#111 Signed-off-by: Salvatore Mesoraca <salvatore.mesoraca@aiven.io> Signed-off-by: Raphaël Vinot <raphael@vinot.info>
Rafiot
pushed a commit
to Rafiot/valkey-py
that referenced
this issue
Dec 3, 2024
Close valkey-io#111 Signed-off-by: Salvatore Mesoraca <salvatore.mesoraca@aiven.io> Signed-off-by: Raphaël Vinot <raphael@vinot.info>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When setting protocol=3 (RESP3), and using the pubsub features, on a message reception it seems some data is outputted in stdout by the library (or stderr not sure), here's an example of this data:
Push response: [b'subscribe', b'mychannel', 1]
This does not happen with the default protocol.
I'm using valkey-py 6.0.2
The text was updated successfully, but these errors were encountered: