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

Convert sleep_for into count_subscriptions logic #628

Closed
mjcarroll opened this issue Apr 21, 2020 · 4 comments
Closed

Convert sleep_for into count_subscriptions logic #628

mjcarroll opened this issue Apr 21, 2020 · 4 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@mjcarroll
Copy link
Member

Feature request

Feature description

Converting from TODO for visibility:

// TODO(wjwwood): add logic to wait for the connection to be established
// probably using the count_subscriptions busy wait mechanism
// until then we will sleep for a short period of time

There are a few places in the test_subscription code (and possibly other places) where we make use of a fixed delay to wait for publications/subscriptions to become available. This should be converted to make use of count_publications/count_subscriptions when possible.

@Barry-Xu-2018
Copy link
Contributor

@mjcarroll

I have an interest in implementing it.
I start to implement it.

@mjcarroll
Copy link
Member Author

Thanks @Barry-Xu-2018! Looking forward to your pull request, let me know if you run into any issues.

@Barry-Xu-2018
Copy link
Contributor

@mjcarroll

I have made PR.
Please review #631.

@jacobperron
Copy link
Member

Looks like this was resolved in #631.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants