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

SnapshotService - cleanSnaphostHistory #78

Closed
rsoika opened this issue Mar 27, 2020 · 1 comment
Closed

SnapshotService - cleanSnaphostHistory #78

rsoika opened this issue Mar 27, 2020 · 1 comment
Milestone

Comments

@rsoika
Copy link
Member

rsoika commented Mar 27, 2020

The method cleanSnaphostHistory uses a JQPL query to test for deprecated snapshots. As the query method runs within a new transaction (new feature since 5.1.9) we can not be sure if the result set is valid within the current transaction. So it might be that the call

 documentService.remove(oldSnapshot)

failed with a EJBException. To avoid this situation (which is rolling back the wole transaction) we need to test if the current document is available. For that we can simply call the load method and see if the document was found in this transaction.

@rsoika rsoika added this to the 2.1.1 milestone Mar 27, 2020
rsoika added a commit that referenced this issue Mar 27, 2020
Issue #78
@rsoika
Copy link
Member Author

rsoika commented Mar 27, 2020

See also imixs/imixs-workflow#634

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant