Polishing
This commit is contained in:
parent
a7949ac84a
commit
3d6476da46
|
|
@ -1,5 +1,5 @@
|
|||
/*
|
||||
* Copyright 2002-2018 the original author or authors.
|
||||
* Copyright 2002-2019 the original author or authors.
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
|
|
@ -287,7 +287,8 @@ public abstract class TransactionAspectSupport implements BeanFactoryAware, Init
|
|||
if (txAttr == null || !(tm instanceof CallbackPreferringPlatformTransactionManager)) {
|
||||
// Standard transaction demarcation with getTransaction and commit/rollback calls.
|
||||
TransactionInfo txInfo = createTransactionIfNecessary(tm, txAttr, joinpointIdentification);
|
||||
Object retVal = null;
|
||||
|
||||
Object retVal;
|
||||
try {
|
||||
// This is an around advice: Invoke the next interceptor in the chain.
|
||||
// This will normally result in a target object being invoked.
|
||||
|
|
@ -508,8 +509,8 @@ public abstract class TransactionAspectSupport implements BeanFactoryAware, Init
|
|||
// The TransactionInfo.hasTransaction() method will return false. We created it only
|
||||
// to preserve the integrity of the ThreadLocal stack maintained in this class.
|
||||
if (logger.isTraceEnabled()) {
|
||||
logger.trace("Don't need to create transaction for [" + joinpointIdentification +
|
||||
"]: This method isn't transactional.");
|
||||
logger.trace("No need to create transaction for [" + joinpointIdentification +
|
||||
"]: This method is not transactional.");
|
||||
}
|
||||
}
|
||||
|
||||
|
|
@ -592,7 +593,7 @@ public abstract class TransactionAspectSupport implements BeanFactoryAware, Init
|
|||
|
||||
|
||||
/**
|
||||
* Opaque object used to hold Transaction information. Subclasses
|
||||
* Opaque object used to hold transaction information. Subclasses
|
||||
* must pass it back to methods on this class, but not see its internals.
|
||||
*/
|
||||
protected final class TransactionInfo {
|
||||
|
|
|
|||
|
|
@ -51,7 +51,7 @@ public interface ExchangeStrategies {
|
|||
// Static builder methods
|
||||
|
||||
/**
|
||||
* Return a new {@code ExchangeStrategies} with default configuration
|
||||
* Return an {@code ExchangeStrategies} instance with default configuration
|
||||
* provided by {@link ClientCodecConfigurer}.
|
||||
*/
|
||||
static ExchangeStrategies withDefaults() {
|
||||
|
|
|
|||
Loading…
Reference in New Issue