Skip to content

Commit 3aaf7ad

Browse files
authored
Minor refactor of IndexBackfiller to eliminate private property. (#3689)
1 parent 672c6dc commit 3aaf7ad

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

firebase-firestore/src/main/java/com/google/firebase/firestore/local/IndexBackfiller.java

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,6 @@ public void setIndexManager(IndexManager indexManager) {
5959
}
6060

6161
public class Scheduler implements com.google.firebase.firestore.local.Scheduler {
62-
private boolean hasRun = false;
6362
@Nullable private AsyncQueue.DelayedTask backfillTask;
6463
private final AsyncQueue asyncQueue;
6564

@@ -69,7 +68,7 @@ public Scheduler(AsyncQueue asyncQueue) {
6968

7069
@Override
7170
public void start() {
72-
scheduleBackfill();
71+
scheduleBackfill(INITIAL_BACKFILL_DELAY_MS);
7372
}
7473

7574
@Override
@@ -79,17 +78,15 @@ public void stop() {
7978
}
8079
}
8180

82-
private void scheduleBackfill() {
83-
long delay = hasRun ? REGULAR_BACKFILL_DELAY_MS : INITIAL_BACKFILL_DELAY_MS;
81+
private void scheduleBackfill(long delay) {
8482
backfillTask =
8583
asyncQueue.enqueueAfterDelay(
8684
AsyncQueue.TimerId.INDEX_BACKFILL,
8785
delay,
8886
() -> {
8987
int documentsProcessed = backfill();
9088
Logger.debug(LOG_TAG, "Documents written: %s", documentsProcessed);
91-
hasRun = true;
92-
scheduleBackfill();
89+
scheduleBackfill(REGULAR_BACKFILL_DELAY_MS);
9390
});
9491
}
9592
}

0 commit comments

Comments
 (0)