From 28c3e9f22826ecc0a4b4220b37b33d9a010b196c Mon Sep 17 00:00:00 2001
From: Oleg Zhurakousky <ozhurakousky@vmware.com>
Date: Fri, 26 Aug 2011 16:33:29 -0400
Subject: [PATCH 1/2] INT-2009 added documentation clarifying the use of
 MessageGroup when using POJO-based ReleaseStrategy

---
 docs/src/reference/docbook/aggregator.xml | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/docs/src/reference/docbook/aggregator.xml b/docs/src/reference/docbook/aggregator.xml
index 7d0bd2bde2..ee6752c08a 100644
--- a/docs/src/reference/docbook/aggregator.xml
+++ b/docs/src/reference/docbook/aggregator.xml
@@ -231,6 +231,28 @@
           aggregation, and false otherwise.</para>
         </listitem>
       </itemizedlist>
+      
+      For example:
+      
+      <programlisting language="java"><![CDATA[public class MyReleaseStrategy{
+
+    @ReleaseStrategy 
+    public boolean canMessagesBeReleased(List<Message<?>>){..}
+}]]></programlisting>
+
+<programlisting language="java"><![CDATA[public class MyReleaseStrategy{
+
+    @ReleaseStrategy 
+    public boolean canMessagesBeReleased(List<String>){..}
+}]]></programlisting>
+
+<para>
+As you can see based on the above signatures the POJO-based Release Strategy will be passed a <classname>Collection</classname> of unmarked Messages 
+if you need access to the whole <classname>Message</classname> or <classname>Collection</classname> of payload objects if the type parameter is 
+anything other than <classname>Message</classname>. Typically this would satisfy the majority of the use cases. However if 
+you need complete access to the <classname>MessageGroup</classname> which gives you access to <code>unmarked</code> and <code>marked</code> Messages 
+than provide the implementation of the <classname>ReleaseStrategy</classname>.
+</para>
 
       <para>When the group is released for aggregation, all its unmarked
       messages are processed and then marked so they will not be processed

From 86d43fd898daf62b2e4a88350ed8bdbe857092e7 Mon Sep 17 00:00:00 2001
From: Mark Fisher <markfisher@vmware.com>
Date: Fri, 26 Aug 2011 17:55:03 -0400
Subject: [PATCH 2/2] polishing

---
 docs/src/reference/docbook/aggregator.xml | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/docs/src/reference/docbook/aggregator.xml b/docs/src/reference/docbook/aggregator.xml
index ee6752c08a..4803064b41 100644
--- a/docs/src/reference/docbook/aggregator.xml
+++ b/docs/src/reference/docbook/aggregator.xml
@@ -234,24 +234,24 @@
       
       For example:
       
-      <programlisting language="java"><![CDATA[public class MyReleaseStrategy{
+      <programlisting language="java"><![CDATA[public class MyReleaseStrategy {
 
     @ReleaseStrategy 
-    public boolean canMessagesBeReleased(List<Message<?>>){..}
+    public boolean canMessagesBeReleased(List<Message<?>>) {...}
 }]]></programlisting>
 
-<programlisting language="java"><![CDATA[public class MyReleaseStrategy{
+<programlisting language="java"><![CDATA[public class MyReleaseStrategy {
 
     @ReleaseStrategy 
-    public boolean canMessagesBeReleased(List<String>){..}
+    public boolean canMessagesBeReleased(List<String>) {...}
 }]]></programlisting>
 
 <para>
-As you can see based on the above signatures the POJO-based Release Strategy will be passed a <classname>Collection</classname> of unmarked Messages 
+As you can see based on the above signatures, the POJO-based Release Strategy will be passed a <classname>Collection</classname> of unmarked Messages 
 if you need access to the whole <classname>Message</classname> or <classname>Collection</classname> of payload objects if the type parameter is 
-anything other than <classname>Message</classname>. Typically this would satisfy the majority of the use cases. However if 
-you need complete access to the <classname>MessageGroup</classname> which gives you access to <code>unmarked</code> and <code>marked</code> Messages 
-than provide the implementation of the <classname>ReleaseStrategy</classname>.
+anything other than <classname>Message</classname>. Typically this would satisfy the majority of use cases. However if 
+for some reason you need to access the full <classname>MessageGroup</classname> - which contains <code>unmarked</code> and <code>marked</code> Messages -
+then you should simply provide an implementation of the <classname>ReleaseStrategy</classname> interface.
 </para>
 
       <para>When the group is released for aggregation, all its unmarked