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 basic handling for transient errors #171

Merged
merged 3 commits into from
Feb 26, 2015
Merged

Conversation

subvertallchris
Copy link
Contributor

There are at least four statuses that Neo says you should just retry, so that's what we'll do. The number may be a little light in some environments, so it is controllable using the NEO4J_RETRY_COUNT environment variable.

In my staging environment at work, this catches most of our errors. The remaining are going to be handled at an app level.

@coveralls
Copy link

Coverage Status

Coverage increased (+2.18%) to 94.48% when pulling ff926b6 on retryable_errors into 08499c2 on master.

subvertallchris added a commit that referenced this pull request Feb 26, 2015
add basic handling for transient errors
@subvertallchris subvertallchris merged commit 7297005 into master Feb 26, 2015
@subvertallchris subvertallchris deleted the retryable_errors branch March 8, 2015 04:36
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.

2 participants