[FIX] Set job to failed after non-retryable OperationalError #132

Merged
merged 1 commit into from Oct 27, 2015

Conversation

Projects
None yet
5 participants
@npiganeau
Contributor

npiganeau commented Oct 26, 2015

Split of #131

connector/controllers/main.py
+ if err.pgcode not in PG_CONCURRENCY_ERRORS_TO_RETRY:
+ raise
+
+ session.env.clear()

This comment has been minimized.

@guewen

guewen Oct 26, 2015

Member

This line should be in the other PR

@guewen

guewen Oct 26, 2015

Member

This line should be in the other PR

@guewen

This comment has been minimized.

Show comment
Hide comment
@guewen

guewen Oct 26, 2015

Member

Could you squash the 2 commits in one?

Thanks! 👍

Member

guewen commented Oct 26, 2015

Could you squash the 2 commits in one?

Thanks! 👍

@yvaucher

This comment has been minimized.

Show comment
Hide comment
@yvaucher

yvaucher Oct 27, 2015

Member

👍

Member

yvaucher commented Oct 27, 2015

👍

@mdietrichc2c

This comment has been minimized.

Show comment
Hide comment
@mdietrichc2c

mdietrichc2c Oct 27, 2015

Contributor

👍

Contributor

mdietrichc2c commented Oct 27, 2015

👍

pedrobaeza added a commit that referenced this pull request Oct 27, 2015

Merge pull request #132 from npiganeau/8.0-fail-nr-operror
[FIX] Set job to failed after non-retryable OperationalError

@pedrobaeza pedrobaeza merged commit a859a52 into OCA:8.0 Oct 27, 2015

2 checks passed

ci/runbot runbot build 3118261-132-73bab2 (runtime 30s)
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment