diff --git a/org.springframework.integration/src/main/java/org/springframework/integration/aggregator/DefaultMessageAggregator.java b/org.springframework.integration/src/main/java/org/springframework/integration/aggregator/DefaultMessageAggregator.java
index 199acef94a..39865b8f23 100644
--- a/org.springframework.integration/src/main/java/org/springframework/integration/aggregator/DefaultMessageAggregator.java
+++ b/org.springframework.integration/src/main/java/org/springframework/integration/aggregator/DefaultMessageAggregator.java
@@ -1,5 +1,5 @@
/*
- * Copyright 2002-2008 the original author or authors.
+ * Copyright 2002-2009 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.
@@ -13,6 +13,7 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
+
package org.springframework.integration.aggregator;
import java.util.ArrayList;
@@ -28,10 +29,11 @@ import org.springframework.integration.message.MessageBuilder;
* is ignored except the correlationId.
*
*
- * n The default strategy for determining whether a group is complete is based
+ * The default strategy for determining whether a group is complete is based
* on the 'sequenceSize' property of the header. Alternatively, a
* custom implementation of the {@link CompletionStrategy} may be provided.
*
+ *
*
* All considerations regarding timeout and grouping by
* correlationId from {@link AbstractMessageBarrierHandler} apply
@@ -39,7 +41,7 @@ import org.springframework.integration.message.MessageBuilder;
*
*
* @author Alex Peters
- *
+ * @since 1.0.3
*/
public class DefaultMessageAggregator extends AbstractMessageAggregator {
diff --git a/org.springframework.integration/src/main/java/org/springframework/integration/config/xml/AggregatorParser.java b/org.springframework.integration/src/main/java/org/springframework/integration/config/xml/AggregatorParser.java
index 4586a37b88..54b1a11228 100644
--- a/org.springframework.integration/src/main/java/org/springframework/integration/config/xml/AggregatorParser.java
+++ b/org.springframework.integration/src/main/java/org/springframework/integration/config/xml/AggregatorParser.java
@@ -1,5 +1,5 @@
/*
- * Copyright 2002-2008 the original author or authors.
+ * Copyright 2002-2009 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.
@@ -16,11 +16,12 @@
package org.springframework.integration.config.xml;
+import org.w3c.dom.Element;
+
import org.springframework.beans.factory.support.BeanDefinitionBuilder;
import org.springframework.beans.factory.support.BeanDefinitionReaderUtils;
import org.springframework.beans.factory.xml.ParserContext;
import org.springframework.util.StringUtils;
-import org.w3c.dom.Element;
/**
* Parser for the aggregator element of the integration namespace.
@@ -55,14 +56,14 @@ public class AggregatorParser extends AbstractConsumerEndpointParser {
private static final String CORRELATION_STRATEGY_PROPERTY = "correlationStrategy";
+
@Override
protected BeanDefinitionBuilder parseHandler(Element element, ParserContext parserContext) {
BeanDefinitionBuilder builder;
String ref = element.getAttribute(REF_ATTRIBUTE);
if (StringUtils.hasText(ref)) {
- builder = BeanDefinitionBuilder
- .genericBeanDefinition(IntegrationNamespaceUtils.BASE_PACKAGE
- + ".aggregator.MethodInvokingAggregator");
+ builder = BeanDefinitionBuilder.genericBeanDefinition(
+ IntegrationNamespaceUtils.BASE_PACKAGE + ".aggregator.MethodInvokingAggregator");
builder.addConstructorArgReference(ref);
if (StringUtils.hasText(element.getAttribute(METHOD_ATTRIBUTE))) {
String method = element.getAttribute(METHOD_ATTRIBUTE);
@@ -70,9 +71,8 @@ public class AggregatorParser extends AbstractConsumerEndpointParser {
}
}
else {
- builder = BeanDefinitionBuilder
- .genericBeanDefinition(IntegrationNamespaceUtils.BASE_PACKAGE
- + ".aggregator.DefaultMessageAggregator");
+ builder = BeanDefinitionBuilder.genericBeanDefinition(
+ IntegrationNamespaceUtils.BASE_PACKAGE + ".aggregator.DefaultMessageAggregator");
}
IntegrationNamespaceUtils.setReferenceIfAttributeDefined(builder, element,
DISCARD_CHANNEL_ATTRIBUTE);
@@ -114,9 +114,8 @@ public class AggregatorParser extends AbstractConsumerEndpointParser {
private String createAdapter(String ref, String method, String unqualifiedClassName,
ParserContext parserContext) {
- BeanDefinitionBuilder builder = BeanDefinitionBuilder
- .genericBeanDefinition(IntegrationNamespaceUtils.BASE_PACKAGE + ".aggregator."
- + unqualifiedClassName);
+ BeanDefinitionBuilder builder = BeanDefinitionBuilder.genericBeanDefinition(
+ IntegrationNamespaceUtils.BASE_PACKAGE + ".aggregator." + unqualifiedClassName);
builder.addConstructorArgReference(ref);
builder.addConstructorArgValue(method);
return BeanDefinitionReaderUtils.registerWithGeneratedName(builder.getBeanDefinition(),
diff --git a/org.springframework.integration/src/test/java/org/springframework/integration/aggregator/DefaultMessageAggregatorTests.java b/org.springframework.integration/src/test/java/org/springframework/integration/aggregator/DefaultMessageAggregatorTests.java
index d19629c717..c007ee6865 100644
--- a/org.springframework.integration/src/test/java/org/springframework/integration/aggregator/DefaultMessageAggregatorTests.java
+++ b/org.springframework.integration/src/test/java/org/springframework/integration/aggregator/DefaultMessageAggregatorTests.java
@@ -1,5 +1,5 @@
/*
- * Copyright 2002-2008 the original author or authors.
+ * Copyright 2002-2009 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.
@@ -13,6 +13,7 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
+
package org.springframework.integration.aggregator;
import static org.hamcrest.CoreMatchers.is;
@@ -23,13 +24,13 @@ import java.util.Arrays;
import java.util.List;
import org.junit.Test;
+
import org.springframework.integration.core.Message;
import org.springframework.integration.message.MessageBuilder;
/**
* @author Alex Peters
* @author Iwein Fuld
- *
*/
public class DefaultMessageAggregatorTests {
@@ -46,4 +47,5 @@ public class DefaultMessageAggregatorTests {
Message> result = aggregator.aggregateMessages(messageGroup);
assertThat((List