Skip to content

DataIntegrityViolationException is thrown instead of OptimisticLockingFailure for CasMismatch #1339

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.

Sign up for GitHub

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

Jump to bottom
Closed
mikereiche opened this issue Feb 10, 2022 · 0 comments · Fixed by #1341
Closed

DataIntegrityViolationException is thrown instead of OptimisticLockingFailure for CasMismatch #1339

mikereiche opened this issue Feb 10, 2022 · 0 comments · Fixed by #1341
Labels
type: bug A general bug type: documentation A documentation update
Milestone
5.0 M2 (2022.0.0)

Comments

@mikereiche
Copy link
Collaborator

This should be fixed for the 5.0 release and the documentation updated as well as mention in the 5.0 migration guide.
@mikereiche mikereiche added type: documentation A documentation update type: bug A general bug labels Feb 10, 2022
@mikereiche mikereiche added this to the 5.0 M2 (2022.0.0) milestone Feb 10, 2022
mikereiche added a commit that referenced this issue Feb 14, 2022
@mikereiche
Translate CasMismatchException to OptimisticLockingFailureException.
108070c 
Translate CasMismatchException to OptimisticLockingFailureException
instead of DataIntegrityViolationException.  CasMismatchException is
a ConcurrencyFailureException which is a TransientDataAccessException.
Versus DataIntegrityViolation, which is a NonTransientDataAccessException.

Closed #1339.
@mikereiche mikereiche mentioned this issue Feb 14, 2022
Merged
5 tasks
mikereiche added a commit that referenced this issue Feb 15, 2022
@mikereiche
Translate CasMismatchException to OptimisticLockingFailureException. (#…
1fcd252 
…1341)

Translate CasMismatchException to OptimisticLockingFailureException
instead of DataIntegrityViolationException.  CasMismatchException is
a ConcurrencyFailureException which is a TransientDataAccessException.
Versus DataIntegrityViolation, which is a NonTransientDataAccessException.

Closed #1339.
mikereiche added a commit that referenced this issue Feb 16, 2022
@mikereiche
Translate CasMismatchException to OptimisticLockingFailureException. (#…
34f4733 
…1341)

Translate CasMismatchException to OptimisticLockingFailureException
instead of DataIntegrityViolationException.  CasMismatchException is
a ConcurrencyFailureException which is a TransientDataAccessException.
Versus DataIntegrityViolation, which is a NonTransientDataAccessException.

Closed #1339.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug A general bug type: documentation A documentation update
Projects
None yet
Milestone
5.0 M2 (2022.0.0)
1 participant
@mikereiche