Skip to content
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

fix: Add timeouts to go-redis client and ping on client creation #2404

Merged
merged 2 commits into from
Jul 13, 2024

Conversation

torwig
Copy link
Contributor

@torwig torwig commented Jul 11, 2024

This prevents failing tests on MacOS13 here: #2324

@git-hulk
Copy link
Member

https://github.com/apache/kvrocks/blob/unstable/tests/gocase/tls/tls_test.go#L57 This case would not return the NOAUTH error since it sent the plain command to the TLS port. So it will return the connection closed by peer error.

@torwig
Copy link
Contributor Author

torwig commented Jul 13, 2024

@git-hulk Yes, today I'll try to make some changes to the initial ones.

@torwig torwig force-pushed the set_test_client_timeouts branch 2 times, most recently from bcaf3ce to 4f30bfd Compare July 13, 2024 16:20
@torwig
Copy link
Contributor Author

torwig commented Jul 13, 2024

Please, review.

Copy link

sonarcloud bot commented Jul 13, 2024

@tisonkun tisonkun merged commit d3115ba into apache:unstable Jul 13, 2024
28 checks passed
@tisonkun
Copy link
Member

Please, review.

I suggest you can avoid using imperative sentences. It reads rudely.

@torwig
Copy link
Contributor Author

torwig commented Jul 14, 2024

@tisonkun I'm sorry, I didn't want to sound rude or offensive.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants