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

Optionally keep partial search results even after an exception #180

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

painter1
Copy link
Contributor

@painter1 painter1 commented Apr 7, 2021

Add an option to save partial search results. That is, if a paginated search raises an exception, sdproxy.searchAPIProxy.call_web_service__PAGINATION will nevertheless return whatever incomplete results it has, rather than re-raise the exception.

This is very useful for production-scale replication where paginated searches are very large and failures not unknown. But by default this is turned off because a naive user would not be aware that the search was incomplete. To turn it on you have
to set the constant NEVER_RAISE_EXCEPTION to True.

… search

raises an exception, sdproxy.searchAPIProxy.call_web_service__PAGINATION will
nevertheless return whatever incomplete results it has.  This is very useful
for production-scale replication where paginated searches are very large and
failures not unkown.  But by default this is turned off because a naive user
would not be aware that the search was incomplete.  To turn it on you have
to set the constant NEVER_RAISE_EXCEPTION to True.
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