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

add another safeguard to reindex test #1109

Merged
merged 1 commit into from
Mar 1, 2017

Conversation

srenatus
Copy link
Contributor

@srenatus srenatus commented Mar 1, 2017

with_search_polling's force_solr_commit could happen without every object having made it to solr before.

This addition makes the test wait for queues to be empty to reduce the likelihood of this happening.

@stevendanna stevendanna merged commit 56811d1 into master Mar 1, 2017
@srenatus srenatus deleted the ssd-sr/fix-reindex-flaky-test branch March 2, 2017 06:57
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