From 1cd443362196aff3fbd04548d5e92f7e9607dbd4 Mon Sep 17 00:00:00 2001 From: Juergen Hoeller Date: Thu, 22 Jan 2015 18:44:07 +0100 Subject: [PATCH] Polishing --- ...actTransactionManagementConfiguration.java | 6 ++-- .../TransactionManagementConfigurer.java | 33 +++++++++---------- 2 files changed, 19 insertions(+), 20 deletions(-) diff --git a/spring-tx/src/main/java/org/springframework/transaction/annotation/AbstractTransactionManagementConfiguration.java b/spring-tx/src/main/java/org/springframework/transaction/annotation/AbstractTransactionManagementConfiguration.java index 764201337d..91272c6f5b 100644 --- a/spring-tx/src/main/java/org/springframework/transaction/annotation/AbstractTransactionManagementConfiguration.java +++ b/spring-tx/src/main/java/org/springframework/transaction/annotation/AbstractTransactionManagementConfiguration.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2014 the original author or authors. + * Copyright 2002-2015 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. @@ -41,7 +41,7 @@ public abstract class AbstractTransactionManagementConfiguration implements Impo protected AnnotationAttributes enableTx; /** - * Default transaction manager. + * Default transaction manager, as configured through a {@link TransactionManagementConfigurer}. */ protected PlatformTransactionManager txManager; @@ -54,7 +54,7 @@ public abstract class AbstractTransactionManagementConfiguration implements Impo "@EnableTransactionManagement is not present on importing class " + importMetadata.getClassName()); } - @Autowired(required=false) + @Autowired(required = false) void setConfigurers(Collection configurers) { if (CollectionUtils.isEmpty(configurers)) { return; diff --git a/spring-tx/src/main/java/org/springframework/transaction/annotation/TransactionManagementConfigurer.java b/spring-tx/src/main/java/org/springframework/transaction/annotation/TransactionManagementConfigurer.java index 21c3ad4cc5..1c7d432960 100644 --- a/spring-tx/src/main/java/org/springframework/transaction/annotation/TransactionManagementConfigurer.java +++ b/spring-tx/src/main/java/org/springframework/transaction/annotation/TransactionManagementConfigurer.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2014 the original author or authors. + * Copyright 2002-2015 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. @@ -21,18 +21,20 @@ import org.springframework.transaction.PlatformTransactionManager; /** * Interface to be implemented by @{@link org.springframework.context.annotation.Configuration * Configuration} classes annotated with @{@link EnableTransactionManagement} that wish to - * or need to explicitly specify the default {@link PlatformTransactionManager} bean to be used - * for annotation-driven transaction management, as opposed to the default approach of a - * by-type lookup. One reason this might be necessary is if there are two + * or need to explicitly specify the default {@link PlatformTransactionManager} bean to be + * used for annotation-driven transaction management, as opposed to the default approach + * of a by-type lookup. One reason this might be necessary is if there are two * {@code PlatformTransactionManager} beans present in the container. * - *

See @{@link EnableTransactionManagement} for general examples and context; see - * {@link #annotationDrivenTransactionManager()} for detailed instructions. + *

See @{@link EnableTransactionManagement} for general examples and context; + * see {@link #annotationDrivenTransactionManager()} for detailed instructions. * *

Note that in by-type lookup disambiguation cases, an alternative approach to - * implementing this interface is to simply mark one of the offending {@code - * PlatformTransactionManager} {@code @Bean} methods as @{@link - * org.springframework.context.annotation.Primary Primary}. + * implementing this interface is to simply mark one of the offending + * {@code PlatformTransactionManager} {@code @Bean} methods as + * @{@link org.springframework.context.annotation.Primary Primary}. + * This is even generally preferred since it doesn't lead to early initialization + * of the {@code PlatformTransactionManager} bean. * * @author Chris Beams * @since 3.1 @@ -44,7 +46,6 @@ public interface TransactionManagementConfigurer { /** * Return the default transaction manager bean to use for annotation-driven database * transaction management, i.e. when processing {@code @Transactional} methods. - * *

There are two basic approaches to implementing this method: *

1. Implement the method and annotate it with {@code @Bean}

* In this case, the implementing {@code @Configuration} class implements this method, @@ -68,15 +69,13 @@ public interface TransactionManagementConfigurer { * public PlatformTransactionManager annotationDrivenTransactionManager() { * return txManager(); // reference the existing {@code @Bean} method above * } - * - * If taking approach #2, be sure that only one of the methods is marked with - * {@code @Bean}! - * + * If taking approach #2, be sure that only one of the methods is marked + * with {@code @Bean}! *

In either scenario #1 or #2, it is important that the * {@code PlatformTransactionManager} instance is managed as a Spring bean within the - * container as all {@code PlatformTransactionManager} implementations take - * advantage of Spring lifecycle callbacks such as {@code InitializingBean} and {@code - * BeanFactoryAware}. + * container as all {@code PlatformTransactionManager} implementations take advantage + * of Spring lifecycle callbacks such as {@code InitializingBean} and + * {@code BeanFactoryAware}. */ PlatformTransactionManager annotationDrivenTransactionManager();