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 support for retry param for Whoosh file writer #2

Open
wants to merge 50 commits into
base: master
Choose a base branch
from

Conversation

RouganStriker
Copy link
Member

  • Can be used to prevent writer from hanging indefinitly
  • Added HAYSTACK_WHOOSH_WRITER_ACQUIRE_MAX_RETRIES setting
  • Added HAYSTACK_WHOOSH_WRITER_ACQUIRE_DELAY setting

Hey, thanks for contributing to Haystack. Please review the contributor guidelines and confirm that the tests pass with at least one search engine.

Once your pull request has been submitted, the full test suite will be executed on https://travis-ci.org/django-haystack/django-haystack/pull_requests. Pull requests with passing tests are far more likely to be reviewed and merged.

tomkins and others added 30 commits July 28, 2017 21:03
__name__ for django-haystack is haystack, but the installed module is django-haystack - so we need to use that instead.
For consistency, this will match the version from get_distribution
add the settings of the Elasticsearch 5.x
Remove Python 2.7, which is end of life. (https://pythonclock.org/)
Add Python 3.7 to the test-matrix.
Remove django.six, that has been deprecated with Django2, from loading.py
Make setuptools-scm requirement pip-compatible
# Conflicts:
#	haystack/__init__.py
Always use `django-haystack` as the package name
Remove libxapian, should be installed as dependency of python-xapian
The version of solr used in the tests is not compatible with the more
current openjdk 11.
@sibartlett
Copy link

+1

@RouganStriker RouganStriker force-pushed the fix/7718/sde_use_writer_retry_param branch 2 times, most recently from 38210cc to aeed5ff Compare April 30, 2020 16:37
* Can be used to prevent writer from hanging indefinitly
* Added HAYSTACK_WHOOSH_WRITER_ACQUIRE_MAX_RETRIES setting
* Added HAYSTACK_WHOOSH_WRITER_ACQUIRE_DELAY setting
@RouganStriker RouganStriker force-pushed the fix/7718/sde_use_writer_retry_param branch from aeed5ff to 6a25a4e Compare April 30, 2020 16:38
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.