[exo-jcr-commits] exo-jcr SVN: r3971 - jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/indexing.

[do-not-reply at jboss.org]

Author: nfilotto
Date: 2011-02-11 15:14:07 -0500 (Fri, 11 Feb 2011)
New Revision: 3971

Modified:
   jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/indexing/JdbcNodeDataIndexingIterator.java
Log:
EXOJCR-1104: Fix a bug in the info message that gives more details while indexing

Modified: jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/indexing/JdbcNodeDataIndexingIterator.java
===================================================================
--- jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/indexing/JdbcNodeDataIndexingIterator.java	2011-02-11 19:44:55 UTC (rev 3970)
+++ jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/storage/jdbc/indexing/JdbcNodeDataIndexingIterator.java	2011-02-11 20:14:07 UTC (rev 3971)
@@ -118,7 +118,12 @@
          long time = System.currentTimeMillis();
          List<NodeDataIndexing> result = conn.getNodesAndProperties(currentLastNodeId, currentOffset, pageSize);
          if (PropertyManager.isDevelopping())
-            LOG.info("Page = " + currentPage + " Offset = " + currentOffset + " LastNodeId = '" + currentLastNodeId + "', query time = " + (System.currentTimeMillis() - time) + " ms, from '" + result.get(0).getIdentifier() + "' to '" + result.get(result.size() - 1).getIdentifier() + "'");
+         {
+            LOG.info("Page = " + currentPage + " Offset = " + currentOffset + " LastNodeId = '" + currentLastNodeId
+               + "', query time = " + (System.currentTimeMillis() - time) + " ms, from '"
+               + (result.isEmpty() ? "unknown" : result.get(0).getIdentifier()) + "' to '"
+               + (result.isEmpty() ? "unknown" : result.get(result.size() - 1).getIdentifier()) + "'");
+         }
          hasNext.compareAndSet(true, result.size() == pageSize);
          if (hasNext())
          {