The backfill check should take more precedence when determining wheth…

…er to reset data (#1846)
faros-ai · Dec 9, 2024 · 3de2ec3 · 3de2ec3
1 parent b26b5a8
commit 3de2ec3
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/destinations/airbyte-faros-destination/src/destination.ts b/destinations/airbyte-faros-destination/src/destination.ts
@@ -997,6 +997,12 @@ export class FarosDestination extends AirbyteDestination<DestinationConfig> {
       isFarosSource,
       sourceSucceeded,
     } = syncInfo;
+    if (isBackfillSync) {
+      this.logger.info(
+        'Running a backfill sync. Skipping reset of non-incremental models.'
+      );
+      return false;
+    }
     if (streamStatusReceived) {
       if (syncErrors.dst.length) {
         this.logger.warn(
@@ -1026,12 +1032,6 @@ export class FarosDestination extends AirbyteDestination<DestinationConfig> {
       this.logger.info('Running a reset sync. Resetting all models.');
       return true;
     }
-    if (isBackfillSync) {
-      this.logger.info(
-        'Running a backfill sync. Skipping reset of non-incremental models.'
-      );
-      return false;
-    }
     if (sourceSucceeded) {
       this.logger.info('Source succeeded. Resetting non-incremental models.');
       return true;