You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oodt.apache.org by rl...@apache.org on 2013/03/14 15:16:05 UTC

svn commit: r1456449 - in /oodt/trunk: CHANGES.txt filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/LuceneCatalog.java

Author: rlaidlaw
Date: Thu Mar 14 14:16:04 2013
New Revision: 1456449

URL: http://svn.apache.org/r1456449
Log:
OODT-573: Refactored the return statement in getTopNProducts method in LuceneCatalog class.

Modified:
    oodt/trunk/CHANGES.txt
    oodt/trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/LuceneCatalog.java

Modified: oodt/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/oodt/trunk/CHANGES.txt?rev=1456449&r1=1456448&r2=1456449&view=diff
==============================================================================
--- oodt/trunk/CHANGES.txt (original)
+++ oodt/trunk/CHANGES.txt Thu Mar 14 14:16:04 2013
@@ -2,6 +2,9 @@ Apache OODT Change Log
 ======================
 Release 0.6 - Current Development
 --------------------------------------------
+* OODT-573 Refactored the return statement in the getTopNProducts method in the
+LuceneCatalog class (rlaidlaw)
+
 * OODT-571 Updated assignments in setWorkflowInst and setWaitforConditionSatisfy
 methods in IterativeWorkflowProcessorThread class (rlaidlaw)
 

Modified: oodt/trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/LuceneCatalog.java
URL: http://svn.apache.org/viewvc/oodt/trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/LuceneCatalog.java?rev=1456449&r1=1456448&r2=1456449&view=diff
==============================================================================
--- oodt/trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/LuceneCatalog.java (original)
+++ oodt/trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/LuceneCatalog.java Thu Mar 14 14:16:04 2013
@@ -753,10 +753,7 @@ public class LuceneCatalog implements Ca
             products.addAll(pageProducts);
         }
 
-        if (products != null) {
-            return products;
-        } else
-            return null;
+        return products;
     }
 
     /*