Updated redo snippets to avoid extensive calling of countRedoRecords()

Author: barrycaceres

csharp/snippets/redo/LoadWithRedoViaLoop/Program.cs

@@ -109,11 +109,10 @@
   }
 
   // now that we have loaded the records, check for redos and handle them
-  while (engine.CountRedoRecords() > 0)
+  for (string redo = engine.GetRedoRecord();
+       redo != null;
+       redo = engine.GetRedoRecord())
   {
-    // get the next redo record
-    string redo = engine.GetRedoRecord();
-
     try
     {
       // process the redo record

csharp/snippets/redo/RedoContinuousViaFutures/Program.cs

@@ -116,6 +116,9 @@ TaskScheduler taskScheduler
     } while (pendingFutures.Count >= MaximumBacklog);
 
     // check if there are no redo records right now
+    // NOTE: we do NOT want to call countRedoRecords() in a loop that
+    // is processing redo records, we call it here AFTER we believe
+    // have processed all pending redos to confirm still zero
     if (engine.CountRedoRecords() == 0)
     {
       OutputRedoStatistics();

java/snippets/redo/LoadWithRedoViaLoop.java

@@ -98,10 +98,10 @@ public static void main(String[] args) {
             }
 
             // now that we have loaded the records, check for redos and handle them
-            while (engine.countRedoRecords() > 0) {
-                // get the next redo record
-                String redo = engine.getRedoRecord();
-
+            for (String redo = engine.getRedoRecord(); 
+                 redo != null; 
+                 redo = engine.getRedoRecord()) 
+            {
                 try {
                     // process the redo record
                     engine.processRedoRecord(redo, SZ_NO_FLAGS);

java/snippets/redo/RedoContinuousViaFutures.java

@@ -103,6 +103,9 @@ public static void main(String[] args) {
         } while (pendingFutures.size() >= MAXIMUM_BACKLOG);
 
         // check if there are no redo records right now
+        // NOTE: we do NOT want to call countRedoRecords() in a loop that
+        // is processing redo records, we call it here AFTER we believe
+        // have processed all pending redos to confirm still zero
         if (engine.countRedoRecords() == 0) {
           outputRedoStatistics();
           System.out.println();