feat(restore-indices): add multithreading and add aspectName, urn filter (#5712)

* feat(upgrade): improve logging to get idea of progress

* division by zero check

* division by zero fix

* wip

* temp work

* feat(datahub-upgrade): add multi-threading and aspect filter

* revert change on local

* fix bugs

* add back batchDelayMs

* remove redundant format

* report stats finally, shutdown executor

* Add time metrics

* clarify logging

* improve logging

datahub-upgrade/src/main/java/com/linkedin/datahub/upgrade/restoreindices/RestoreIndices.java

@@ -18,6 +18,9 @@ import java.util.List;
 public class RestoreIndices implements Upgrade {
   public static final String BATCH_SIZE_ARG_NAME = "batchSize";
   public static final String BATCH_DELAY_MS_ARG_NAME = "batchDelayMs";
+  public static final String NUM_THREADS_ARG_NAME = "numThreads";
+  public static final String ASPECT_NAME_ARG_NAME = "aspectName";
+  public static final String URN_ARG_NAME = "urn";
 
   private final List<UpgradeStep> _steps;
 

datahub-upgrade/src/main/java/com/linkedin/datahub/upgrade/restoreindices/SendMAEStep.java

@@ -8,7 +8,6 @@ import com.linkedin.datahub.upgrade.UpgradeContext;
 import com.linkedin.datahub.upgrade.UpgradeStep;
 import com.linkedin.datahub.upgrade.UpgradeStepResult;
 import com.linkedin.datahub.upgrade.impl.DefaultUpgradeStepResult;
-import com.linkedin.datahub.upgrade.nocode.NoCodeUpgrade;
 import com.linkedin.events.metadata.ChangeType;
 import com.linkedin.metadata.entity.EntityService;
 import com.linkedin.metadata.entity.EntityUtils;
@@ -18,9 +17,18 @@ import com.linkedin.metadata.models.EntitySpec;
 import com.linkedin.metadata.models.registry.EntityRegistry;
 import com.linkedin.mxe.SystemMetadata;
 import io.ebean.EbeanServer;
+import io.ebean.ExpressionList;
 import io.ebean.PagedList;
+
+import java.util.ArrayList;
+import java.util.List;
 import java.util.Map;
 import java.util.Optional;
+import java.util.concurrent.Callable;
+import java.util.concurrent.ExecutionException;
+import java.util.concurrent.Executors;
+import java.util.concurrent.Future;
+import java.util.concurrent.ThreadPoolExecutor;
 import java.util.concurrent.TimeUnit;
 import java.util.function.Function;
 
@@ -32,11 +40,114 @@ public class SendMAEStep implements UpgradeStep {
 
   private static final int DEFAULT_BATCH_SIZE = 1000;
   private static final long DEFAULT_BATCH_DELAY_MS = 250;
+  private static final int DEFAULT_THREADS = 1;
 
   private final EbeanServer _server;
   private final EntityService _entityService;
   private final EntityRegistry _entityRegistry;
 
+  public static class KafkaJobResult {
+    public int ignored;
+    public int rowsMigrated;
+    public KafkaJobResult(int ignored, int rowsMigrated) {
+      this.ignored = ignored;
+      this.rowsMigrated = rowsMigrated;
+    }
+  }
+
+  public class KafkaJob implements Callable<KafkaJobResult> {
+      UpgradeContext context;
+      int start;
+      int batchSize;
+      long batchDelayMs;
+      Optional<String> aspectName;
+      Optional<String> urn;
+      public KafkaJob(UpgradeContext context, int start, int batchSize, long batchDelayMs, Optional<String> aspectName,
+                      Optional<String> urn) {
+        this.context = context;
+        this.start = start;
+        this.batchSize = batchSize;
+        this.batchDelayMs = batchDelayMs;
+        this.aspectName = aspectName;
+        this.urn = urn;
+      }
+      @Override
+      public KafkaJobResult call() {
+        int ignored = 0;
+        int rowsMigrated = 0;
+        context.report()
+                .addLine(String.format("Reading rows %s through %s from the aspects table.", start, start + batchSize));
+        PagedList<EbeanAspectV2> rows = getPagedAspects(start, batchSize, aspectName, urn);
+
+        for (EbeanAspectV2 aspect : rows.getList()) {
+          // 1. Extract an Entity type from the entity Urn
+          Urn urn;
+          try {
+            urn = Urn.createFromString(aspect.getKey().getUrn());
+          } catch (Exception e) {
+            context.report()
+                    .addLine(String.format("Failed to bind Urn with value %s into Urn object: %s. Ignoring row.",
+                            aspect.getKey().getUrn(), e));
+            ignored = ignored + 1;
+            continue;
+          }
+
+          // 2. Verify that the entity associated with the aspect is found in the registry.
+          final String entityName = urn.getEntityType();
+          final EntitySpec entitySpec;
+          try {
+            entitySpec = _entityRegistry.getEntitySpec(entityName);
+          } catch (Exception e) {
+            context.report()
+                    .addLine(String.format("Failed to find entity with name %s in Entity Registry: %s. Ignoring row.",
+                            entityName, e));
+            ignored = ignored + 1;
+            continue;
+          }
+          final String aspectName = aspect.getKey().getAspect();
+
+          // 3. Verify that the aspect is a valid aspect associated with the entity
+          AspectSpec aspectSpec = entitySpec.getAspectSpec(aspectName);
+          if (aspectSpec == null) {
+            context.report()
+                    .addLine(String.format("Failed to find aspect with name %s associated with entity named %s", aspectName,
+                            entityName));
+            ignored = ignored + 1;
+            continue;
+          }
+
+          // 4. Create record from json aspect
+          final RecordTemplate aspectRecord;
+          try {
+            aspectRecord = EntityUtils.toAspectRecord(entityName, aspectName, aspect.getMetadata(), _entityRegistry);
+          } catch (Exception e) {
+            context.report()
+                    .addLine(String.format("Failed to deserialize row %s for entity %s, aspect %s: %s. Ignoring row.",
+                            aspect.getMetadata(), entityName, aspectName, e));
+            ignored = ignored + 1;
+            continue;
+          }
+
+          SystemMetadata latestSystemMetadata = EntityUtils.parseSystemMetadata(aspect.getSystemMetadata());
+
+          // 5. Produce MAE events for the aspect record
+          _entityService.produceMetadataChangeLog(urn, entityName, aspectName, aspectSpec, null, aspectRecord, null,
+                  latestSystemMetadata,
+                  new AuditStamp().setActor(UrnUtils.getUrn(SYSTEM_ACTOR)).setTime(System.currentTimeMillis()),
+                  ChangeType.RESTATE);
+
+          rowsMigrated++;
+        }
+
+        try {
+          TimeUnit.MILLISECONDS.sleep(batchDelayMs);
+        } catch (InterruptedException e) {
+          throw new RuntimeException("Thread interrupted while sleeping after successful batch migration.");
+        }
+        return new KafkaJobResult(ignored, rowsMigrated);
+      }
+  }
+
   public SendMAEStep(final EbeanServer server, final EntityService entityService, final EntityRegistry entityRegistry) {
     _server = server;
     _entityService = entityService;
@@ -53,100 +164,143 @@ public class SendMAEStep implements UpgradeStep {
     return 0;
   }
 
+  private KafkaJobResult iterateFutures(List<Future<KafkaJobResult>> futures) {
+    int beforeSize = futures.size();
+    int afterSize = futures.size();
+    while (beforeSize == afterSize) {
+      try {
+        TimeUnit.SECONDS.sleep(1);
+      } catch (InterruptedException e) {
+        // suppress
+      }
+      for (Future<KafkaJobResult> future: new ArrayList<>(futures)) {
+        if (future.isDone()) {
+          try {
+            KafkaJobResult result = future.get();
+            futures.remove(future);
+            return result;
+          } catch (InterruptedException | ExecutionException e) {
+            e.printStackTrace();
+          }
+        }
+      }
+      afterSize = futures.size();
+    }
+    return null;
+  }
+
   @Override
   public Function<UpgradeContext, UpgradeStepResult> executable() {
     return (context) -> {
 
-      context.report().addLine("Sending MAE from local DB...");
+      long startTime = System.currentTimeMillis();
-      final int rowCount =
+
-          _server.find(EbeanAspectV2.class).where().eq(EbeanAspectV2.VERSION_COLUMN, ASPECT_LATEST_VERSION).findCount();
+      int batchSize = getBatchSize(context.parsedArgs());
+      int numThreads = getThreadCount(context.parsedArgs());
+      long batchDelayMs = getBatchDelayMs(context.parsedArgs());
+      Optional<String> aspectName;
+      if (containsKey(context.parsedArgs(), RestoreIndices.ASPECT_NAME_ARG_NAME)) {
+        aspectName = context.parsedArgs().get(RestoreIndices.ASPECT_NAME_ARG_NAME);
+        context.report().addLine(String.format("Found aspectName arg as %s", aspectName));
+      } else {
+        aspectName = Optional.empty();
+        context.report().addLine("No aspectName arg present");
+      }
+      Optional<String> urn;
+      if (containsKey(context.parsedArgs(), RestoreIndices.URN_ARG_NAME)) {
+        urn = context.parsedArgs().get(RestoreIndices.URN_ARG_NAME);
+        context.report().addLine(String.format("Found urn arg as %s", urn));
+      } else {
+        urn = Optional.empty();
+        context.report().addLine("No urn arg present");
+      }
+
+      ThreadPoolExecutor executor = (ThreadPoolExecutor) Executors.newFixedThreadPool(numThreads);
+
+      context.report().addLine(
+              String.format("Sending MAE from local DB with %s batch size, %s threads, %s batchDelayMs",
+                      batchSize, numThreads, batchDelayMs));
+      ExpressionList<EbeanAspectV2> countExp =
+          _server.find(EbeanAspectV2.class)
+            .where()
+            .eq(EbeanAspectV2.VERSION_COLUMN, ASPECT_LATEST_VERSION);
+      if (aspectName.isPresent()) {
+        countExp = countExp.eq(EbeanAspectV2.ASPECT_COLUMN, aspectName.get());
+      }
+      if (urn.isPresent()) {
+        countExp = countExp.eq(EbeanAspectV2.URN_COLUMN, urn.get());
+      }
+      final int rowCount = countExp.findCount();
       context.report().addLine(String.format("Found %s latest aspects in aspects table", rowCount));
 
       int totalRowsMigrated = 0;
       int start = 0;
-      int count = getBatchSize(context.parsedArgs());
+      int ignored = 0;
+
+      List<Future<KafkaJobResult>> futures = new ArrayList<>();
       while (start < rowCount) {
-
+        while (futures.size() < numThreads) {
-        context.report()
+          futures.add(executor.submit(new KafkaJob(context, start, batchSize, batchDelayMs, aspectName, urn)));
-            .addLine(String.format("Reading rows %s through %s from the aspects table.", start, start + count));
+          start = start + batchSize;
-        PagedList<EbeanAspectV2> rows = getPagedAspects(start, count);
-
-        for (EbeanAspectV2 aspect : rows.getList()) {
-          // 1. Extract an Entity type from the entity Urn
-          Urn urn;
-          try {
-            urn = Urn.createFromString(aspect.getKey().getUrn());
-          } catch (Exception e) {
-            context.report()
-                .addLine(String.format("Failed to bind Urn with value %s into Urn object: %s. Ignoring row.",
-                    aspect.getKey().getUrn(), e));
-            continue;
         }
-
+        KafkaJobResult tmpResult = iterateFutures(futures);
-          // 2. Verify that the entity associated with the aspect is found in the registry.
+        if (tmpResult != null) {
-          final String entityName = urn.getEntityType();
+          totalRowsMigrated += tmpResult.rowsMigrated;
-          final EntitySpec entitySpec;
+          ignored += tmpResult.ignored;
-          try {
+          reportStats(context, totalRowsMigrated, ignored, rowCount, startTime);
-            entitySpec = _entityRegistry.getEntitySpec(entityName);
-          } catch (Exception e) {
-            context.report()
-                .addLine(String.format("Failed to find entity with name %s in Entity Registry: %s. Ignoring row.",
-                    entityName, e));
-            continue;
-          }
-          final String aspectName = aspect.getKey().getAspect();
-
-          // 3. Verify that the aspect is a valid aspect associated with the entity
-          AspectSpec aspectSpec = entitySpec.getAspectSpec(aspectName);
-          if (aspectSpec == null) {
-            context.report()
-                .addLine(String.format("Failed to find aspect with name %s associated with entity named %s", aspectName,
-                    entityName));
-            continue;
-          }
-
-          // 4. Create record from json aspect
-          final RecordTemplate aspectRecord;
-          try {
-            aspectRecord = EntityUtils.toAspectRecord(entityName, aspectName, aspect.getMetadata(), _entityRegistry);
-          } catch (Exception e) {
-            context.report()
-                .addLine(String.format("Failed to deserialize row %s for entity %s, aspect %s: %s. Ignoring row.",
-                    aspect.getMetadata(), entityName, aspectName, e));
-            continue;
-          }
-
-          SystemMetadata latestSystemMetadata = EntityUtils.parseSystemMetadata(aspect.getSystemMetadata());
-
-          // 5. Produce MAE events for the aspect record
-          _entityService.produceMetadataChangeLog(urn, entityName, aspectName, aspectSpec, null, aspectRecord, null,
-              latestSystemMetadata,
-              new AuditStamp().setActor(UrnUtils.getUrn(SYSTEM_ACTOR)).setTime(System.currentTimeMillis()),
-              ChangeType.RESTATE);
-
-          totalRowsMigrated++;
-        }
-        context.report().addLine(String.format("Successfully sent MAEs for %s rows", totalRowsMigrated));
-        start = start + count;
-        try {
-          TimeUnit.MILLISECONDS.sleep(getBatchDelayMs(context.parsedArgs()));
-        } catch (InterruptedException e) {
-          throw new RuntimeException("Thread interrupted while sleeping after successful batch migration.");
         }
       }
+      while (futures.size() > 0) {
+        KafkaJobResult tmpResult = iterateFutures(futures);
+        if (tmpResult != null) {
+          totalRowsMigrated += tmpResult.rowsMigrated;
+          ignored += tmpResult.ignored;
+          reportStats(context, totalRowsMigrated, ignored, rowCount, startTime);
+        }
+      }
+      executor.shutdown();
       if (totalRowsMigrated != rowCount) {
-        context.report().addLine(String.format("Failed to send MAEs for %d rows...", rowCount - totalRowsMigrated));
+        float percentFailed = 0.0f;
+        if (rowCount > 0) {
+          percentFailed = (float) (rowCount - totalRowsMigrated) * 100 / rowCount;
+        }
+        context.report().addLine(String.format(
+                "Failed to send MAEs for %d rows (%.2f%% of total).", rowCount - totalRowsMigrated, percentFailed));
       }
       return new DefaultUpgradeStepResult(id(), UpgradeStepResult.Result.SUCCEEDED);
     };
   }
 
-  private PagedList<EbeanAspectV2> getPagedAspects(final int start, final int pageSize) {
+  private static void reportStats(UpgradeContext context, int totalRowsMigrated, int ignored, int rowCount,
-    return _server.find(EbeanAspectV2.class)
+                                  long startTime) {
+    long currentTime = System.currentTimeMillis();
+    float timeSoFarMinutes = (float) (currentTime - startTime) / 1000 / 60;
+    float percentSent = (float) totalRowsMigrated * 100 / rowCount;
+    float percentIgnored = (float) ignored * 100 / rowCount;
+    float estimatedTimeMinutesComplete = -1;
+    if (percentSent > 0) {
+      estimatedTimeMinutesComplete = timeSoFarMinutes * (100 - percentSent) / percentSent;
+    }
+    context.report().addLine(String.format(
+            "Successfully sent MAEs for %s/%s rows (%.2f%% of total). %s rows ignored (%.2f%% of total)",
+            totalRowsMigrated, rowCount, percentSent, ignored, percentIgnored));
+    context.report().addLine(String.format("%.2f minutes taken. %.2f estimate minutes to completion",
+            timeSoFarMinutes, estimatedTimeMinutesComplete));
+  }
+
+  private PagedList<EbeanAspectV2> getPagedAspects(final int start, final int pageSize, Optional<String> aspectName,
+                                                   Optional<String> urn) {
+    ExpressionList<EbeanAspectV2> exp = _server.find(EbeanAspectV2.class)
             .select(EbeanAspectV2.ALL_COLUMNS)
             .where()
-        .eq(EbeanAspectV2.VERSION_COLUMN, ASPECT_LATEST_VERSION)
+            .eq(EbeanAspectV2.VERSION_COLUMN, ASPECT_LATEST_VERSION);
-        .orderBy()
+    if (aspectName.isPresent()) {
+      exp = exp.eq(EbeanAspectV2.ASPECT_COLUMN, aspectName.get());
+    }
+    if (urn.isPresent()) {
+      exp = exp.eq(EbeanAspectV2.URN_COLUMN, urn.get());
+    }
+    return  exp.orderBy()
         .asc(EbeanAspectV2.URN_COLUMN)
         .orderBy()
         .asc(EbeanAspectV2.ASPECT_COLUMN)
@@ -156,20 +310,30 @@ public class SendMAEStep implements UpgradeStep {
   }
 
   private int getBatchSize(final Map<String, Optional<String>> parsedArgs) {
-    int resolvedBatchSize = DEFAULT_BATCH_SIZE;
+    return getInt(parsedArgs, DEFAULT_BATCH_SIZE, RestoreIndices.BATCH_SIZE_ARG_NAME);
-    if (parsedArgs.containsKey(RestoreIndices.BATCH_SIZE_ARG_NAME) && parsedArgs.get(NoCodeUpgrade.BATCH_SIZE_ARG_NAME)
-        .isPresent()) {
-      resolvedBatchSize = Integer.parseInt(parsedArgs.get(RestoreIndices.BATCH_SIZE_ARG_NAME).get());
-    }
-    return resolvedBatchSize;
   }
 
   private long getBatchDelayMs(final Map<String, Optional<String>> parsedArgs) {
     long resolvedBatchDelayMs = DEFAULT_BATCH_DELAY_MS;
-    if (parsedArgs.containsKey(RestoreIndices.BATCH_DELAY_MS_ARG_NAME) && parsedArgs.get(
+    if (containsKey(parsedArgs, RestoreIndices.BATCH_DELAY_MS_ARG_NAME)) {
-        NoCodeUpgrade.BATCH_DELAY_MS_ARG_NAME).isPresent()) {
       resolvedBatchDelayMs = Long.parseLong(parsedArgs.get(RestoreIndices.BATCH_DELAY_MS_ARG_NAME).get());
     }
     return resolvedBatchDelayMs;
   }
+
+  private int getThreadCount(final Map<String, Optional<String>> parsedArgs) {
+    return getInt(parsedArgs, DEFAULT_THREADS, RestoreIndices.NUM_THREADS_ARG_NAME);
+  }
+
+  private int getInt(final Map<String, Optional<String>> parsedArgs, int defaultVal, String argKey) {
+    int result = defaultVal;
+    if (containsKey(parsedArgs, argKey)) {
+      result = Integer.parseInt(parsedArgs.get(argKey).get());
+    }
+    return result;
+  }
+
+  public static boolean containsKey(final Map<String, Optional<String>> parsedArgs, String key) {
+    return parsedArgs.containsKey(key) && parsedArgs.get(key).isPresent();
+  }
 }