Changeset 4155

Timestamp:

2010-07-19 09:12:40 (3 years ago)

Author:

bruno

Message:

comments

Location:

trunk/indexer/src

Files:

• main/java/org/lilycms/indexer/IndexUpdater.java (modified) (1 diff)
• test/java/org/lilycms/indexer/test/IndexerTest.java (modified) (1 diff)

trunk/indexer/src/main/java/org/lilycms/indexer/IndexUpdater.java

@@ -486 +486 @@
             } catch (Exception e) {
                 // TODO handle this
+                // One case to be expected here is that the record has been deleted since we read the list of referrers
                 e.printStackTrace();
             }

trunk/indexer/src/test/java/org/lilycms/indexer/test/IndexerTest.java

@@ -436 +436 @@
             verifyResultCount("@@id:" + ClientUtils.escapeQueryChars(record3.getId().toString()), 0);
 
-            // Delete record 4 (at the time of this writing, because it is unsure if we will allow deleting master
-            // records while there are variants)
+            // Delete record 4
             log.debug("Begin test NV12");
             expectEvent(DELETE, record4.getId());