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

Test uris connectable #26

Closed
wants to merge 2 commits into from
Closed

Test uris connectable #26

wants to merge 2 commits into from

Conversation

loyipsing
Copy link

image

because maintainConn use the goroutine to get connection pool. so use the time.sleep to confirm the connected is not reliable if the network delay

some project will check all connection such as mysql or redis connection is ok before start service

so i add a dial check(default open) before the NewProducer and NewConsumer to confirm the uris is connectable

@loyipsing loyipsing closed this by deleting the head repository Jul 31, 2023
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.

1 participant