Reputation: 9374
I wonder whether it makes sense to use instead of
@Transactional(propagation = Propagation.REQUIRED, rollbackFor = Exception.class)
to use Throwable
@Transactional(propagation = Propagation.REQUIRED, rollbackFor = Throwable.class)
As I understand catching Error
will help us behave correctly even when something really bad happen. Or maybe it wouldn't help?
Upvotes: 45
Views: 126081
Reputation: 51483
As I understand catching Error will help us behave correctly even when something really bad happen. Or maybe it wouldn't help?
You don't need to explicitly specify rollbackFor = Throwable.class
, because Spring will by default rollback the transaction if an Error
occurs.
See 1.4.3. Rolling Back a Declarative Transaction
In its default configuration, the Spring Framework’s transaction infrastructure code marks a transaction for rollback only in the case of runtime, unchecked exceptions. That is, when the thrown exception is an instance or subclass of
RuntimeException
. (Error instances also, by default, result in a rollback). Checked exceptions that are thrown from a transactional method do not result in rollback in the default configuration.
Or take a look at the DefaultTransactionAttribute
public boolean rollbackOn(Throwable ex) {
return (ex instanceof RuntimeException || ex instanceof Error);
}
Upvotes: 65
Reputation: 280178
Since you are using @Transactional
, we can safely assume you are doing your database operations through Spring, Hibernate, or other JDBC wrappers. These JDBC wrappers don't typically throw checked exceptions, they throw runtime exceptions that wrap the JDBC SQLException
types.
@Transactional
is setup to rollback, by default, only when an unchecked exception is thrown.
Consider a use case like so
@Transactional
public void persistAndWrite(Bean someBean) throws IOException {
// DB operation
getSession().save(someBean);
// File IO operation which throws IOException
someFileService.writeToFile(someBean);
}
You wouldn't necessarily want to rollback the DB operation just because we couldn't write something to a file.
Similarly
@Transactional
public void persistAndThrowOutOfMemory(Bean someBean) {
// DB operation
getSession().save(someBean);
// consumes all memory, throws OutOfMemoryError
someService.hugeOperationThrowsOutOfMemoryError();
}
You wouldn't necessarily want to rollback the saved entity just because some service cause too much memory to be consumed.
@Transactional
gives you the option. Use it where appropriate.
Upvotes: 18
Reputation: 2587
Default value of rollback is register on Error Exception but when u register try{}catch{}
manually it overriding the error, so in this case use
catch {
TransactionAspectSupport.currentTransactionStatus().setRollbackOnly();
}
to do it manually or remove try catch
also you may register exception type in transactional annotation such:
@Transactional(propagation = Propagation.REQUIRED, rollbackFor = Exception.class)
Upvotes: 2
Reputation: 3141
I don't know is it possible or not but handling Throwable
s like Error
s is a bad style of programming, it is not the developers responsibility to handle this kind of fatal errors. There always can happen bad things which You cannot handle. You should handle checked exceptions if necessary, which are known to your system like some type of logical errors.
Upvotes: 0