@fluidframework/container-runtime 1.1.0 → 1.2.0

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/container-runtime";
-export const pkgVersion = "1.1.0";
+export const pkgVersion = "1.2.0";

package/src/summarizerTypes.ts

@@ -152,7 +152,7 @@ export interface IGeneratedSummaryStats extends ISummaryStats {
     readonly gcBlobNodeCount?: number;
     /** Sum of the sizes of all op contents since the last summary */
     readonly opsSizesSinceLastSummary: number;
-    /** Number of non-system ops since the last summary @see isSystemMessage */
+    /** Number of non-system ops since the last summary. See {@link @fluidframework/protocol-base#isSystemMessage} */
     readonly nonSystemOpsSinceLastSummary: number;
     /** The summary number for a container's summary. Incremented on summaries throughout its lifetime. */
     readonly summaryNumber: number;
@@ -300,7 +300,7 @@ export interface ISummarizerEvents extends IEvent {
 }
 
 export interface ISummarizer extends
-    IEventProvider<ISummarizerEvents>, IFluidLoadable, Partial<IProvideSummarizer>{
+    IEventProvider<ISummarizerEvents>, IFluidLoadable, Partial<IProvideSummarizer> {
     /*
      * Asks summarizer to move to exit.
      * Summarizer will finish current processes, which may take a while.
@@ -428,7 +428,10 @@ type SummaryGeneratorOptionalTelemetryProperties =
     /** Delta in sum of op sizes between the current reference sequence number and the reference
      *  sequence number of the last summary */
     "opsSizesSinceLastSummary" |
-    /** Delta between the number of non-system ops since the last summary @see isSystemMessage */
+    /**
+     * Delta between the number of non-system ops since the last summary.
+     * See {@link @fluidframework/protocol-base#isSystemMessage}
+     */
     "nonSystemOpsSinceLastSummary" |
     /** Time it took to generate the summary tree and stats. */
     "generateDuration" |

package/src/summaryFormat.ts

@@ -187,8 +187,12 @@ export const dataStoreAttributesBlobName = ".component";
 
 /**
  * Modifies summary tree and stats to put tree under .channels tree.
+ *
+ * @param summarizeResult - Summary tree and stats to modify
+ *
+ * @example
  * Converts from:
- * ```ts
+ * ```typescript
  * {
  *     type: SummaryType.Tree,
  *     tree: { a: {...}, b: {...}, c: {...} },
@@ -197,7 +201,7 @@ export const dataStoreAttributesBlobName = ".component";
  *
  * to:
  *
- * ```ts
+ * ```typescript
  * {
  *     type: SummaryType.Tree,
  *     tree: {
@@ -209,7 +213,6 @@ export const dataStoreAttributesBlobName = ".component";
  * }
  * ```
  * And adds +1 to treeNodeCount in stats.
- * @param summarizeResult - summary tree and stats to modify
  */
 export function wrapSummaryInChannelsTree(summarizeResult: ISummaryTreeWithStats): void {
     summarizeResult.summary = {

package/src/summaryGenerator.ts

@@ -393,8 +393,6 @@ export class SummaryGenerator {
 
                 // pre-0.58 error message prefix: summaryNack
                 const error = new LoggingError(`Received summaryNack: ${message}`, { retryAfterSeconds });
-                logger.sendErrorEvent(
-                    { eventName: "SummaryNack", ...summarizeTelemetryProps, retryAfterSeconds }, error);
 
                 assert(getRetryDelaySecondsFromError(error) === retryAfterSeconds, 0x25f /* "retryAfterSeconds" */);
                 // This will only set resultsBuilder.receivedSummaryAckOrNack, as other promises are already set.