[hornetq-commits] JBoss hornetq SVN: r12180 - trunk/hornetq-journal/src/main/java/org/hornetq/core/journal/impl.

[do-not-reply at jboss.org]

Author: borges
Date: 2012-02-23 09:18:22 -0500 (Thu, 23 Feb 2012)
New Revision: 12180

Modified:
   trunk/hornetq-journal/src/main/java/org/hornetq/core/journal/impl/JournalImpl.java
Log:
Comparators should implement Serializable if possible. Because:
If a comparator is used to construct an ordered collection (ex: TreeMap),
the Collection will only be serializable if the Comparator is as well.

Modified: trunk/hornetq-journal/src/main/java/org/hornetq/core/journal/impl/JournalImpl.java
===================================================================
--- trunk/hornetq-journal/src/main/java/org/hornetq/core/journal/impl/JournalImpl.java	2012-02-23 09:41:02 UTC (rev 12179)
+++ trunk/hornetq-journal/src/main/java/org/hornetq/core/journal/impl/JournalImpl.java	2012-02-23 14:18:22 UTC (rev 12180)
@@ -13,6 +13,7 @@
 
 package org.hornetq.core.journal.impl;
 
+import java.io.Serializable;
 import java.nio.ByteBuffer;
 import java.util.ArrayList;
 import java.util.Collections;
@@ -2953,8 +2954,10 @@
 
    }
 
-   private static class JournalFileComparator implements Comparator<JournalFile>
+   private static class JournalFileComparator implements Comparator<JournalFile>, Serializable
    {
+      private static final long serialVersionUID = -6264728973604070321L;
+
       public int compare(final JournalFile f1, final JournalFile f2)
       {
          long id1 = f1.getFileID();