@fluidframework/container-runtime 2.0.0-rc.1.0.3 → 2.0.0-rc.2.0.0

package/dist/summary/runningSummarizer.d.ts

@@ -5,10 +5,10 @@
 import { IDisposable, ITelemetryBaseLogger } from "@fluidframework/core-interfaces";
 import { TypedEventEmitter } from "@fluid-internal/client-utils";
 import { ISequencedDocumentMessage } from "@fluidframework/protocol-definitions";
-import { ISummaryConfiguration } from "../containerRuntime";
-import { IEnqueueSummarizeOptions, ISummarizeHeuristicData, IOnDemandSummarizeOptions, EnqueueSummarizeResult, SummarizerStopReason, ISubmitSummaryOptions, SubmitSummaryResult, ISummaryCancellationToken, ISummarizeResults, ISummarizerRuntime, IRefreshSummaryAckOptions, ISummarizerEvents } from "./summarizerTypes";
-import { IClientSummaryWatcher, SummaryCollection } from "./summaryCollection";
-import { SummarizeResultBuilder } from "./summaryGenerator";
+import { ISummaryConfiguration } from "../containerRuntime.js";
+import { IEnqueueSummarizeOptions, ISummarizeHeuristicData, IOnDemandSummarizeOptions, EnqueueSummarizeResult, SummarizerStopReason, ISubmitSummaryOptions, SubmitSummaryResult, ISummaryCancellationToken, ISummarizeResults, ISummarizerRuntime, IRefreshSummaryAckOptions, ISummarizerEvents } from "./summarizerTypes.js";
+import { IClientSummaryWatcher, SummaryCollection } from "./summaryCollection.js";
+import { SummarizeResultBuilder } from "./summaryGenerator.js";
 /**
  * The maximum number of summarization attempts that will be done by default in case of failures
  * that can be retried.
@@ -55,13 +55,14 @@ export declare class RunningSummarizer extends TypedEventEmitter<ISummarizerEven
     private readonly maxAttemptsForSubmitFailures;
     private constructor();
     private handleSummaryAck;
+    private readonly refreshLatestSummaryAckAndHandleError;
     /**
-     * Responsible for receiving and processing all the summaryAcks.
-     * In case there was a summary ack processed by the running summarizer before processIncomingSummaryAcks is called,
-     * it will wait for the summary ack that is newer than the one indicated by the lastAckRefSeq.
-     * @param lastAckRefSeq - Identifies the minimum reference sequence number the summarizer needs to wait for.
-     * In case of a negative number, the summarizer will wait for ANY summary ack that is greater than the deltaManager's initial sequence number,
-     * and, in case of a positive one, it will wait for a summary ack that is greater than this current reference sequence number.
+     * Responsible for receiving and processing all the summary acks.
+     * It starts processing ACKs after the one for the summary this client loaded from (initialSequenceNumber). Any
+     * ACK before that is not interesting as it will simply be ignored.
+     *
+     * @param referenceSequenceNumber - The referenceSequenceNumber of the summary from which to start processing
+     * acks.
      */
     private processIncomingSummaryAcks;
     dispose(): void;

package/dist/summary/runningSummarizer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"runningSummarizer.d.ts","sourceRoot":"","sources":["../../src/summary/runningSummarizer.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,WAAW,EAAE,oBAAoB,EAAE,MAAM,iCAAiC,CAAC;AASpF,OAAO,EAAE,iBAAiB,EAAE,MAAM,8BAA8B,CAAC;AAEjE,OAAO,EAAE,yBAAyB,EAAe,MAAM,sCAAsC,CAAC;AAC9F,OAAO,EAAE,qBAAqB,EAAE,MAAM,qBAAqB,CAAC;AAG5D,OAAO,EACN,wBAAwB,EAExB,uBAAuB,EAEvB,yBAAyB,EACzB,sBAAsB,EACtB,oBAAoB,EACpB,qBAAqB,EACrB,mBAAmB,EACnB,yBAAyB,EACzB,iBAAiB,EAEjB,kBAAkB,EAElB,yBAAyB,EACzB,iBAAiB,EAEjB,MAAM,mBAAmB,CAAC;AAC3B,OAAO,EAAiB,qBAAqB,EAAE,iBAAiB,EAAE,MAAM,qBAAqB,CAAC;AAC9F,OAAO,EAGN,sBAAsB,EAEtB,MAAM,oBAAoB,CAAC;AAI5B;;;GAGG;AACH,eAAO,MAAM,kBAAkB,IAAI,CAAC;AACpC;;;GAGG;AACH,eAAO,MAAM,mCAAmC,IAAI,CAAC;AAErD;;;;;;GAMG;AACH,qBAAa,iBAAkB,SAAQ,iBAAiB,CAAC,iBAAiB,CAAE,YAAW,WAAW;IAqGhG,OAAO,CAAC,QAAQ,CAAC,cAAc;IAC/B,OAAO,CAAC,QAAQ,CAAC,aAAa;IAC9B,OAAO,CAAC,QAAQ,CAAC,qBAAqB;IAGtC,OAAO,CAAC,QAAQ,CAAC,+BAA+B;IAGhD,OAAO,CAAC,QAAQ,CAAC,aAAa;IAC9B,OAAO,CAAC,QAAQ,CAAC,iBAAiB;IAClC,OAAO,CAAC,QAAQ,CAAC,iBAAiB;IAClC,OAAO,CAAC,QAAQ,CAAC,sBAAsB;IACvC,OAAO,CAAC,QAAQ,CAAC,OAAO;WAhHL,KAAK,CACxB,MAAM,EAAE,oBAAoB,EAC5B,cAAc,EAAE,qBAAqB,EACrC,aAAa,EAAE,qBAAqB,EACpC,qBAAqB,EAAE,CAAC,OAAO,EAAE,qBAAqB,KAAK,OAAO,CAAC,mBAAmB,CAAC,EACvF,+BAA+B,EAAE,CAAC,OAAO,EAAE,yBAAyB,KAAK,OAAO,CAAC,IAAI,CAAC,EACtF,aAAa,EAAE,uBAAuB,EACtC,iBAAiB,EAAE,iBAAiB,EACpC,iBAAiB,EAAE,yBAAyB,EAC5C,sBAAsB,EAAE,CAAC,MAAM,EAAE,oBAAoB,KAAK,IAAI,EAC9D,OAAO,EAAE,kBAAkB,GACzB,OAAO,CAAC,iBAAiB,CAAC;IA0D7B,IAAW,QAAQ,YAElB;IACD,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,SAAS,CAAS;IAC1B,OAAO,CAAC,eAAe,CAA4B;IACnD,OAAO,CAAC,mBAAmB,CAAS;IACpC,OAAO,CAAC,QAAQ,CAAC,eAAe,CAAe;IAC/C,OAAO,CAAC,eAAe,CAAC,CAA4B;IACpD,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAmB;IAC7C,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAoB;IAEvC,OAAO,CAAC,eAAe,CAOV;IACb,OAAO,CAAC,cAAc,CAAK;IAC3B,OAAO,CAAC,uBAAuB,CAAK;IACpC,OAAO,CAAC,WAAW,CAAS;IAE5B,OAAO,CAAC,QAAQ,CAAC,eAAe,CAAC;IAEjC,8EAA8E;IAC9E,OAAO,CAAC,QAAQ,CAAC,4BAA4B,CAAS;IAEtD,OAAO;YA4GO,gBAAgB;IA6D9B;;;;;;;OAOG;YACW,0BAA0B;IAsBjC,OAAO,IAAI,IAAI;IAYtB;;;;;OAKG;IACI,sBAAsB,sGAGf;IAEd,wGAAwG;IACxG,OAAO,CAAC,8BAA8B,CAAS;IAExC,QAAQ,CAAC,EAAE,EAAE,yBAAyB,EAAE,cAAc,EAAE,OAAO;IA8BtE;;;;;OAKG;IACH,OAAO,CAAC,mBAAmB;IAW3B,OAAO,CAAC,6BAA6B;IAWxB,QAAQ,CAAC,gBAAgB,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC;YA6BjD,SAAS;IAuBvB,OAAO,CAAC,mBAAmB;IAI3B,OAAO,CAAC,kBAAkB;IAW1B;;;;;;;;OAQG;YACW,mBAAmB;IAsBjC;;;;;;;OAOG;IACH,OAAO,CAAC,gBAAgB;IAoCxB,oCAAoC;IACpC,OAAO,CAAC,YAAY;IAyBpB;;;OAGG;YACW,8BAA8B;IAsE5C;;;OAGG;YACW,uBAAuB;IAuIrC;;;OAGG;YACW,4BAA4B;IAkB1C,8DAA8D;IACvD,iBAAiB,CACvB,OAAO,EAAE,yBAAyB,EAClC,cAAc,GAAE,sBAAqD,GACnE,iBAAiB;IA6BpB,6DAA6D;IACtD,gBAAgB,CAAC,OAAO,EAAE,wBAAwB,GAAG,sBAAsB;IAiClF,OAAO,CAAC,qBAAqB;IAwB7B,OAAO,CAAC,sBAAsB;CAS9B"}
+{"version":3,"file":"runningSummarizer.d.ts","sourceRoot":"","sources":["../../src/summary/runningSummarizer.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,WAAW,EAAE,oBAAoB,EAAE,MAAM,iCAAiC,CAAC;AASpF,OAAO,EAAE,iBAAiB,EAAE,MAAM,8BAA8B,CAAC;AAEjE,OAAO,EAAE,yBAAyB,EAAe,MAAM,sCAAsC,CAAC;AAC9F,OAAO,EAAE,qBAAqB,EAAE,MAAM,wBAAwB,CAAC;AAG/D,OAAO,EACN,wBAAwB,EAExB,uBAAuB,EAEvB,yBAAyB,EACzB,sBAAsB,EACtB,oBAAoB,EACpB,qBAAqB,EACrB,mBAAmB,EACnB,yBAAyB,EACzB,iBAAiB,EAEjB,kBAAkB,EAElB,yBAAyB,EACzB,iBAAiB,EAEjB,MAAM,sBAAsB,CAAC;AAC9B,OAAO,EAAiB,qBAAqB,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC;AACjG,OAAO,EAGN,sBAAsB,EAEtB,MAAM,uBAAuB,CAAC;AAI/B;;;GAGG;AACH,eAAO,MAAM,kBAAkB,IAAI,CAAC;AACpC;;;GAGG;AACH,eAAO,MAAM,mCAAmC,IAAI,CAAC;AAErD;;;;;;GAMG;AACH,qBAAa,iBAAkB,SAAQ,iBAAiB,CAAC,iBAAiB,CAAE,YAAW,WAAW;IAgHhG,OAAO,CAAC,QAAQ,CAAC,cAAc;IAC/B,OAAO,CAAC,QAAQ,CAAC,aAAa;IAC9B,OAAO,CAAC,QAAQ,CAAC,qBAAqB;IAGtC,OAAO,CAAC,QAAQ,CAAC,+BAA+B;IAGhD,OAAO,CAAC,QAAQ,CAAC,aAAa;IAC9B,OAAO,CAAC,QAAQ,CAAC,iBAAiB;IAClC,OAAO,CAAC,QAAQ,CAAC,iBAAiB;IAClC,OAAO,CAAC,QAAQ,CAAC,sBAAsB;IACvC,OAAO,CAAC,QAAQ,CAAC,OAAO;WA3HL,KAAK,CACxB,MAAM,EAAE,oBAAoB,EAC5B,cAAc,EAAE,qBAAqB,EACrC,aAAa,EAAE,qBAAqB,EACpC,qBAAqB,EAAE,CAAC,OAAO,EAAE,qBAAqB,KAAK,OAAO,CAAC,mBAAmB,CAAC,EACvF,+BAA+B,EAAE,CAAC,OAAO,EAAE,yBAAyB,KAAK,OAAO,CAAC,IAAI,CAAC,EACtF,aAAa,EAAE,uBAAuB,EACtC,iBAAiB,EAAE,iBAAiB,EACpC,iBAAiB,EAAE,yBAAyB,EAC5C,sBAAsB,EAAE,CAAC,MAAM,EAAE,oBAAoB,KAAK,IAAI,EAC9D,OAAO,EAAE,kBAAkB,GACzB,OAAO,CAAC,iBAAiB,CAAC;IAqE7B,IAAW,QAAQ,YAElB;IACD,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,SAAS,CAAS;IAC1B,OAAO,CAAC,eAAe,CAA4B;IACnD,OAAO,CAAC,mBAAmB,CAAS;IACpC,OAAO,CAAC,QAAQ,CAAC,eAAe,CAAe;IAC/C,OAAO,CAAC,eAAe,CAAC,CAA4B;IACpD,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAmB;IAC7C,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAoB;IAEvC,OAAO,CAAC,eAAe,CAOV;IACb,OAAO,CAAC,cAAc,CAAK;IAC3B,OAAO,CAAC,uBAAuB,CAAK;IACpC,OAAO,CAAC,WAAW,CAAS;IAE5B,OAAO,CAAC,QAAQ,CAAC,eAAe,CAAC;IAEjC,8EAA8E;IAC9E,OAAO,CAAC,QAAQ,CAAC,4BAA4B,CAAS;IAEtD,OAAO;YAoHO,gBAAgB;IAiC9B,OAAO,CAAC,QAAQ,CAAC,qCAAqC,CA2BpD;IAEF;;;;;;;OAOG;YACW,0BAA0B;IAYjC,OAAO,IAAI,IAAI;IAYtB;;;;;OAKG;IACI,sBAAsB,sGAGf;IAEd,wGAAwG;IACxG,OAAO,CAAC,8BAA8B,CAAS;IAExC,QAAQ,CAAC,EAAE,EAAE,yBAAyB,EAAE,cAAc,EAAE,OAAO;IA8BtE;;;;;OAKG;IACH,OAAO,CAAC,mBAAmB;IAW3B,OAAO,CAAC,6BAA6B;IAWxB,QAAQ,CAAC,gBAAgB,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC;YA6BjD,SAAS;IAuBvB,OAAO,CAAC,mBAAmB;IAI3B,OAAO,CAAC,kBAAkB;IAW1B;;;;;;;;OAQG;YACW,mBAAmB;IAsBjC;;;;;;;OAOG;IACH,OAAO,CAAC,gBAAgB;IAgDxB,oCAAoC;IACpC,OAAO,CAAC,YAAY;IAyBpB;;;OAGG;YACW,8BAA8B;IAkF5C;;;OAGG;YACW,uBAAuB;IAqJrC;;;OAGG;YACW,4BAA4B;IAkB1C,8DAA8D;IACvD,iBAAiB,CACvB,OAAO,EAAE,yBAAyB,EAClC,cAAc,GAAE,sBAAqD,GACnE,iBAAiB;IA6BpB,6DAA6D;IACtD,gBAAgB,CAAC,OAAO,EAAE,wBAAwB,GAAG,sBAAsB;IAiClF,OAAO,CAAC,qBAAqB;IAwB7B,OAAO,CAAC,sBAAsB;CAS9B"}

package/dist/summary/runningSummarizer.js

@@ -10,9 +10,9 @@ const core_utils_1 = require("@fluidframework/core-utils");
 const client_utils_1 = require("@fluid-internal/client-utils");
 const driver_definitions_1 = require("@fluidframework/driver-definitions");
 const protocol_definitions_1 = require("@fluidframework/protocol-definitions");
-const opProperties_1 = require("../opProperties");
-const summarizerHeuristics_1 = require("./summarizerHeuristics");
-const summaryGenerator_1 = require("./summaryGenerator");
+const opProperties_js_1 = require("../opProperties.js");
+const summarizerHeuristics_js_1 = require("./summarizerHeuristics.js");
+const summaryGenerator_js_1 = require("./summaryGenerator.js");
 const maxSummarizeAckWaitTime = 10 * 60 * 1000; // 10 minutes
 /**
  * The maximum number of summarization attempts that will be done by default in case of failures
@@ -34,13 +34,22 @@ exports.defaultMaxAttemptsForSubmitFailures = 5;
 class RunningSummarizer extends client_utils_1.TypedEventEmitter {
     static async start(logger, summaryWatcher, configuration, submitSummaryCallback, refreshLatestSummaryAckCallback, heuristicData, summaryCollection, cancellationToken, stopSummarizerCallback, runtime) {
         const summarizer = new RunningSummarizer(logger, summaryWatcher, configuration, submitSummaryCallback, refreshLatestSummaryAckCallback, heuristicData, summaryCollection, cancellationToken, stopSummarizerCallback, runtime);
-        // Before doing any heuristics or proceeding with its refreshing, if there is a summary ack received while
-        // this summarizer catches up, let's refresh state before proceeding with the summarization.
-        const lastAckRefSeq = await summarizer.handleSummaryAck();
+        // If there have been any acks newer that the one this client loaded from until now, process them before
+        // starting the running summarizer which will trigger summary heuristics.
+        // This is done primarily to handle scenarios where the summarizer loads from a cached snapshot and there
+        // is newer one available. The ack for the newer summary is processed before summarizing because otherwise
+        // that summary would fail as it has an older parent.
+        let nextReferenceSequenceNumber = runtime.deltaManager.initialSequenceNumber + 1;
+        const latestAck = summaryCollection.latestAck;
+        if (latestAck !== undefined &&
+            latestAck.summaryOp.referenceSequenceNumber >= nextReferenceSequenceNumber) {
+            await summarizer.handleSummaryAck(latestAck);
+            nextReferenceSequenceNumber = latestAck.summaryOp.referenceSequenceNumber + 1;
+        }
         await summarizer.waitStart();
-        // Handle summary acks asynchronously
+        // Process summary acks asynchronously
         // Note: no exceptions are thrown from processIncomingSummaryAcks handler as it handles all exceptions
-        summarizer.processIncomingSummaryAcks(lastAckRefSeq).catch((error) => {
+        summarizer.processIncomingSummaryAcks(nextReferenceSequenceNumber).catch((error) => {
             (0, telemetry_utils_1.createChildLogger)({ logger }).sendErrorEvent({ eventName: "HandleSummaryAckFatalError" }, error);
         });
         // Update heuristic counts
@@ -86,6 +95,27 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
         this.summarizeCount = 0;
         this.totalSuccessfulAttempts = 0;
         this.initialized = false;
+        this.refreshLatestSummaryAckAndHandleError = async (options) => {
+            return this.refreshLatestSummaryAckCallback(options).catch(async (error) => {
+                // If the error is 404, so maybe the fetched version no longer exists on server. We just
+                // ignore this error in that case, as that means we will have another summaryAck for the
+                // latest version with which we will refresh the state. However in case of single commit
+                // summary, we might be missing a summary ack, so in that case we are still fine as the
+                // code in `submitSummary` function in container runtime, will refresh the latest state
+                // by calling `prefetchLatestSummaryThenClose`. We will load the next summarizer from the
+                // updated state and be fine.
+                const isIgnoredError = (0, telemetry_utils_1.isFluidError)(error) &&
+                    error.errorType === driver_definitions_1.DriverErrorTypes.fileNotFoundOrAccessDeniedError;
+                options.summaryLogger.sendTelemetryEvent({
+                    eventName: isIgnoredError
+                        ? "HandleSummaryAckErrorIgnored"
+                        : "HandleLastSummaryAckError",
+                    referenceSequenceNumber: options.summaryRefSeq,
+                    proposalHandle: options.proposalHandle,
+                    ackHandle: options.ackHandle,
+                }, error);
+            });
+        };
         /**
          * RunningSummarizer's logger includes the sequenced index of the current summary on each event.
          * If some other Summarizer code wants that event on their logs they can get it here,
@@ -110,7 +140,7 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
         });
         if (configuration.state !== "disableHeuristics") {
             (0, core_utils_1.assert)(this.configuration.state === "enabled", 0x2ea /* "Configuration state should be enabled" */);
-            this.heuristicRunner = new summarizerHeuristics_1.SummarizeHeuristicRunner(heuristicData, this.configuration, (reason) => this.trySummarize(reason), this.mc.logger);
+            this.heuristicRunner = new summarizerHeuristics_js_1.SummarizeHeuristicRunner(heuristicData, this.configuration, (reason) => this.trySummarize(reason), this.mc.logger);
         }
         (0, core_utils_1.assert)(this.configuration.state !== "disabled", 0x2eb /* "Summary not supported with configuration disabled" */);
         // Cap the maximum amount of time client will wait for a summarize op ack to maxSummarizeAckWaitTime
@@ -140,8 +170,14 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
                 this.pendingAckTimer.clear();
             }
         });
-        this.generator = new summaryGenerator_1.SummaryGenerator(this.pendingAckTimer, this.heuristicData, this.submitSummaryCallback, () => {
+        const immediatelyRefreshLatestSummaryAck = this.mc.config.getBoolean("Fluid.Summarizer.immediatelyRefreshLatestSummaryAck") ??
+            true;
+        this.generator = new summaryGenerator_js_1.SummaryGenerator(this.pendingAckTimer, this.heuristicData, this.submitSummaryCallback, () => {
             this.totalSuccessfulAttempts++;
+        }, async (options) => {
+            if (immediatelyRefreshLatestSummaryAck) {
+                await this.refreshLatestSummaryAckAndHandleError(options);
+            }
         }, this.summaryWatcher, this.mc.logger);
         // Listen to runtime for ops
         this.runtimeListener = (op, runtimeMessage) => {
@@ -157,78 +193,48 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
                 ? overrideMaxAttempts
                 : exports.defaultMaxAttemptsForSubmitFailures;
     }
-    async handleSummaryAck() {
-        const lastAck = this.summaryCollection.latestAck;
-        let refSequenceNumber = -1;
-        // In case we haven't received the lastestAck yet, just return.
-        if (lastAck !== undefined) {
-            refSequenceNumber = lastAck.summaryOp.referenceSequenceNumber;
-            const summaryLogger = this.tryGetCorrelatedLogger(refSequenceNumber) ?? this.mc.logger;
-            const summaryOpHandle = lastAck.summaryOp.contents.handle;
-            const summaryAckHandle = lastAck.summaryAck.contents.handle;
-            while (this.summarizingLock !== undefined) {
-                summaryLogger.sendTelemetryEvent({
-                    eventName: "RefreshAttemptWithSummarizerRunning",
-                    referenceSequenceNumber: refSequenceNumber,
-                    proposalHandle: summaryOpHandle,
-                    ackHandle: summaryAckHandle,
-                });
-                await this.summarizingLock;
-            }
-            // Make sure we block any summarizer from being executed/enqueued while
-            // executing the refreshLatestSummaryAck.
-            // https://dev.azure.com/fluidframework/internal/_workitems/edit/779
-            await this.lockedSummaryAction(() => { }, async () => this.refreshLatestSummaryAckCallback({
+    async handleSummaryAck(ack) {
+        const refSequenceNumber = ack.summaryOp.referenceSequenceNumber;
+        const summaryLogger = this.tryGetCorrelatedLogger(refSequenceNumber) ?? this.mc.logger;
+        const summaryOpHandle = ack.summaryOp.contents.handle;
+        const summaryAckHandle = ack.summaryAck.contents.handle;
+        while (this.summarizingLock !== undefined) {
+            summaryLogger.sendTelemetryEvent({
+                eventName: "RefreshAttemptWithSummarizerRunning",
+                referenceSequenceNumber: refSequenceNumber,
+                proposalHandle: summaryOpHandle,
+                ackHandle: summaryAckHandle,
+            });
+            await this.summarizingLock;
+        }
+        // Make sure we block any summarizer from being executed/enqueued while
+        // executing the refreshLatestSummaryAck.
+        // https://dev.azure.com/fluidframework/internal/_workitems/edit/779
+        await this.lockedSummaryAction(() => { }, async () => {
+            const options = {
                 proposalHandle: summaryOpHandle,
                 ackHandle: summaryAckHandle,
                 summaryRefSeq: refSequenceNumber,
                 summaryLogger,
-            }).catch(async (error) => {
-                // If the error is 404, so maybe the fetched version no longer exists on server. We just
-                // ignore this error in that case, as that means we will have another summaryAck for the
-                // latest version with which we will refresh the state. However in case of single commit
-                // summary, we might me missing a summary ack, so in that case we are still fine as the
-                // code in `submitSummary` function in container runtime, will refresh the latest state
-                // by calling `prefetchLatestSummaryThenClose`. We will load the next summarizer from the
-                // updated state and be fine.
-                const isIgnoredError = (0, telemetry_utils_1.isFluidError)(error) &&
-                    error.errorType === driver_definitions_1.DriverErrorTypes.fileNotFoundOrAccessDeniedError;
-                summaryLogger.sendTelemetryEvent({
-                    eventName: isIgnoredError
-                        ? "HandleSummaryAckErrorIgnored"
-                        : "HandleLastSummaryAckError",
-                    referenceSequenceNumber: refSequenceNumber,
-                    proposalHandle: summaryOpHandle,
-                    ackHandle: summaryAckHandle,
-                }, error);
-            }), () => { });
-            refSequenceNumber++;
-        }
-        return refSequenceNumber;
+            };
+            await this.refreshLatestSummaryAckAndHandleError(options);
+        }, () => { });
     }
     /**
-     * Responsible for receiving and processing all the summaryAcks.
-     * In case there was a summary ack processed by the running summarizer before processIncomingSummaryAcks is called,
-     * it will wait for the summary ack that is newer than the one indicated by the lastAckRefSeq.
-     * @param lastAckRefSeq - Identifies the minimum reference sequence number the summarizer needs to wait for.
-     * In case of a negative number, the summarizer will wait for ANY summary ack that is greater than the deltaManager's initial sequence number,
-     * and, in case of a positive one, it will wait for a summary ack that is greater than this current reference sequence number.
+     * Responsible for receiving and processing all the summary acks.
+     * It starts processing ACKs after the one for the summary this client loaded from (initialSequenceNumber). Any
+     * ACK before that is not interesting as it will simply be ignored.
+     *
+     * @param referenceSequenceNumber - The referenceSequenceNumber of the summary from which to start processing
+     * acks.
      */
-    async processIncomingSummaryAcks(lastAckRefSeq) {
-        let refSequenceNumber = lastAckRefSeq > 0 ? lastAckRefSeq : this.runtime.deltaManager.initialSequenceNumber;
+    async processIncomingSummaryAcks(referenceSequenceNumber) {
+        // Start waiting for acks that are for summaries newer that the one this client loaded from.
+        let nextReferenceSequenceNumber = referenceSequenceNumber;
         while (!this.disposed) {
-            const summaryLogger = this.tryGetCorrelatedLogger(refSequenceNumber) ?? this.mc.logger;
-            // Initialize ack with undefined if exception happens inside of waitSummaryAck on second iteration,
-            // we record undefined, not previous handles.
-            await this.summaryCollection.waitSummaryAck(refSequenceNumber);
-            summaryLogger.sendTelemetryEvent({
-                eventName: "processIncomingSummaryAcks",
-                referenceSequenceNumber: refSequenceNumber,
-                lastAckRefSeq,
-            });
-            refSequenceNumber = await this.handleSummaryAck();
-            // A valid Summary Ack must have been processed.
-            (0, core_utils_1.assert)(refSequenceNumber >= 0, 0x58f /* Invalid ref sequence number */);
+            const ackedSummary = await this.summaryCollection.waitSummaryAck(nextReferenceSequenceNumber);
+            await this.handleSummaryAck(ackedSummary);
+            nextReferenceSequenceNumber = ackedSummary.summaryOp.referenceSequenceNumber + 1;
         }
     }
     dispose() {
@@ -250,7 +256,7 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
         else {
             this.heuristicData.numNonRuntimeOps++;
         }
-        this.heuristicData.totalOpsSize += (0, opProperties_1.opSize)(op);
+        this.heuristicData.totalOpsSize += (0, opProperties_js_1.opSize)(op);
         // Check for enqueued on-demand summaries; Intentionally do nothing otherwise
         if (this.initialized &&
             this.opCanTriggerSummary(op, runtimeMessage) &&
@@ -315,7 +321,7 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
     }
     async waitStart() {
         // Wait no longer than ack timeout for all pending
-        const waitStartResult = await (0, summaryGenerator_1.raceTimer)(this.summaryWatcher.waitFlushed(), this.pendingAckTimer.start());
+        const waitStartResult = await (0, summaryGenerator_js_1.raceTimer)(this.summaryWatcher.waitFlushed(), this.pendingAckTimer.start());
         this.pendingAckTimer.clear();
         // Remove pending ack wait timeout by op timestamp comparison, because
         // it has race conditions with summaries submitted by this same client.
@@ -370,7 +376,7 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
      * @param resultsBuilder - optional, result builder to use.
      * @returns ISummarizeResult - result of running a summary.
      */
-    trySummarizeOnce(summarizeProps, options, resultsBuilder = new summaryGenerator_1.SummarizeResultBuilder()) {
+    trySummarizeOnce(summarizeProps, options, resultsBuilder = new summaryGenerator_js_1.SummarizeResultBuilder()) {
         this.lockedSummaryAction(() => {
             this.beforeSummaryAction();
         }, async () => {
@@ -382,10 +388,18 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
                 ...options,
                 summaryLogger,
                 cancellationToken: this.cancellationToken,
+                latestSummaryRefSeqNum: this.heuristicData.lastSuccessfulSummary.refSequenceNumber,
             };
             const summarizeResult = this.generator.summarize(summaryOptions, resultsBuilder);
             // ensure we wait till the end of the process
-            return summarizeResult.receivedSummaryAckOrNack;
+            const result = await summarizeResult.receivedSummaryAckOrNack;
+            if (!result.success) {
+                this.mc.logger.sendErrorEvent({
+                    eventName: "SummarizeFailed",
+                    maxAttempts: 1,
+                    summaryAttempts: 1,
+                }, result.error);
+            }
         }, () => {
             this.afterSummaryAction();
         }).catch((error) => {
@@ -428,6 +442,7 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
         let summaryAttempts = 0;
         let summaryAttemptsPerPhase = 0;
         let summaryAttemptPhase = 0;
+        let error;
         while (summaryAttemptPhase < attemptOptions.length) {
             if (this.cancellationToken.cancelled) {
                 return;
@@ -453,6 +468,7 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
                 ...summarizeOptions,
                 summaryLogger,
                 cancellationToken: this.cancellationToken,
+                latestSummaryRefSeqNum: this.heuristicData.lastSuccessfulSummary.refSequenceNumber,
             };
             // Note: no need to account for cancellationToken.waitCancelled here, as
             // this is accounted SummaryGenerator.summarizeCore that controls receivedSummaryAckOrNack.
@@ -461,6 +477,7 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
             if (ackNackResult.success) {
                 return;
             }
+            error = ackNackResult.error;
             // Check for retryDelay that can come from summaryNack, upload summary or submit summary flows.
             // Retry the same step only once per retryAfter response.
             const submitResult = await resultSummarize.summarySubmitted;
@@ -481,6 +498,11 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
                 await (0, core_utils_1.delay)(delaySeconds * 1000);
             }
         }
+        this.mc.logger.sendErrorEvent({
+            eventName: "SummarizeFailed",
+            maxAttempts: attemptOptions.length,
+            summaryAttempts: summaryAttemptPhase,
+        }, error);
         this.stopSummarizerCallback("failToSummarize");
     }
     /**
@@ -508,6 +530,7 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
                 summaryLogger,
                 cancellationToken: this.cancellationToken,
                 finalAttempt,
+                latestSummaryRefSeqNum: this.heuristicData.lastSuccessfulSummary.refSequenceNumber,
             };
             const summarizeResult = this.generator.summarize(summaryOptions);
             return { summarizeProps, summarizeResult };
@@ -527,6 +550,7 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
         let done = false;
         let status = "success";
         let results;
+        let error;
         do {
             currentAttempt++;
             if (this.cancellationToken.cancelled) {
@@ -558,11 +582,12 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
             }
             // Emit "summarize" event for this failed attempt.
             status = "failure";
+            error = ackNackResult.error;
             const eventProps = {
                 result: status,
                 currentAttempt,
                 maxAttempts,
-                error: ackNackResult.error,
+                error,
             };
             this.emit("summarize", eventProps);
             // If the failure doesn't have "retryAfterSeconds" or the max number of attempts have been done, we're done.
@@ -595,6 +620,9 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
             // Ack / nack is the final step, so if it succeeds we're done.
             const ackNackResult = await summarizeResult.receivedSummaryAckOrNack;
             status = ackNackResult.success ? "success" : "failure";
+            if (!ackNackResult.success) {
+                error = ackNackResult.error;
+            }
             const eventProps = {
                 result: status,
                 currentAttempt,
@@ -606,6 +634,11 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
         }
         // If summarization is still unsuccessful, stop the summarizer.
         if (status === "failure") {
+            this.mc.logger.sendErrorEvent({
+                eventName: "SummarizeFailed",
+                maxAttempts,
+                summaryAttempts: currentAttempt,
+            }, error);
             this.stopSummarizerCallback("failToSummarize");
         }
         return results;
@@ -629,7 +662,7 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
         return resultsBuilder.build();
     }
     /** {@inheritdoc (ISummarizer:interface).summarizeOnDemand} */
-    summarizeOnDemand(options, resultsBuilder = new summaryGenerator_1.SummarizeResultBuilder()) {
+    summarizeOnDemand(options, resultsBuilder = new summaryGenerator_js_1.SummarizeResultBuilder()) {
         if (this.stopping) {
             resultsBuilder.fail("RunningSummarizer stopped or disposed", undefined);
             return resultsBuilder.build();
@@ -668,7 +701,7 @@ class RunningSummarizer extends client_utils_1.TypedEventEmitter {
             reason: `enqueue;${reason}`,
             afterSequenceNumber,
             summarizeOptions,
-            resultsBuilder: new summaryGenerator_1.SummarizeResultBuilder(),
+            resultsBuilder: new summaryGenerator_js_1.SummarizeResultBuilder(),
         };
         const results = this.enqueuedSummary.resultsBuilder.build();
         this.tryRunEnqueuedSummary();