Defect #514
closed
Transaction - noRollbackOn
100%
Description
Using annotation @Transactional(noRollbackFor=ProvisioningException.class) is problematic in case, when is call some other service with method marked with @Transactional.
In this case will exception (thorwed in this method) marks transaction as TO ROLLBACK.
Related issues
Updated by Vít Švanda about 7 years ago
- Target version changed from Diamond (7.4.0) to Emerald (7.5.0)
Updated by Radek Tomiška about 7 years ago
- Target version deleted (
Emerald (7.5.0))
Updated by Radek Tomiška about 7 years ago
- Has duplicate Defect #804: Error in connector or in script is swallow added
Updated by Radek Tomiška over 5 years ago
- Related to Task #1588: Entity events - transactions added
Updated by Radek Tomiška over 5 years ago
- Status changed from New to In Progress
- Target version set to Quartz (9.6.0)
- % Done changed from 0 to 70
Updated by Radek Tomiška over 5 years ago
- Status changed from In Progress to Needs feedback
- Assignee changed from Radek Tomiška to Vít Švanda
- % Done changed from 70 to 90
Using transactions in provisioning queue was improved:
- Generalize 'ProvisioningException' is not needed (but recommended) - all exception types are saved into queue
- long running tasks, event queue and provisioning queue integration was checked
- logging exception was improved (e.g. password change failures, event processors, prevalidate password ...)
Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/e8fc104cf94d7a2f072ee9fd3bc2dc92ce853381
Exception improvement:
https://github.com/bcvsolutions/CzechIdMng/commit/5f46489c02a82e9b8d5e3b3a42c57ee525140275
Could you do a feedback, please?
Updated by Vít Švanda over 5 years ago
- Status changed from Needs feedback to Resolved
- Assignee changed from Vít Švanda to Radek Tomiška
- % Done changed from 90 to 100
I tested it. I tried to many test cause (included exceptions in provisioning, realization of the request ...). Everything works fine. Propagate.NOT_SUPPORTED is new feature for me, but it is interesting approach.
Thanks for that.
Updated by Radek Tomiška over 5 years ago
- Status changed from Resolved to Closed