@fluidframework/container-runtime 2.0.0-rc.1.0.6 → 2.0.0-rc.2.0.1

package/src/dataStoreContext.ts

@@ -9,18 +9,14 @@ import {
 	IRequest,
 	IResponse,
 	IFluidHandle,
-	ITelemetryProperties,
+	ITelemetryBaseProperties,
 } from "@fluidframework/core-interfaces";
-import {
-	IAudience,
-	IDeltaManager,
-	AttachState,
-	ILoaderOptions,
-} from "@fluidframework/container-definitions";
+import { IAudience, IDeltaManager, AttachState } from "@fluidframework/container-definitions";
 import { TypedEventEmitter } from "@fluid-internal/client-utils";
-import { assert, Deferred, LazyPromise } from "@fluidframework/core-utils";
+import { assert, LazyPromise } from "@fluidframework/core-utils";
 import { IDocumentStorageService } from "@fluidframework/driver-definitions";
 import { BlobTreeEntry, readAndParse } from "@fluidframework/driver-utils";
+import type { IIdCompressor } from "@fluidframework/id-compressor";
 import {
 	IClientDetails,
 	IDocumentMessage,
@@ -29,7 +25,6 @@ import {
 	ISnapshotTree,
 	ITreeEntry,
 } from "@fluidframework/protocol-definitions";
-import { IContainerRuntime } from "@fluidframework/container-runtime-definitions";
 import {
 	channelsTreeName,
 	CreateChildSummarizerNodeFn,
@@ -37,6 +32,8 @@ import {
 	FluidDataStoreRegistryEntry,
 	IFluidDataStoreChannel,
 	IFluidDataStoreContext,
+	IFluidParentContext,
+	IContainerRuntimeBase,
 	IFluidDataStoreContextDetached,
 	IFluidDataStoreContextEvents,
 	IFluidDataStoreRegistry,
@@ -49,10 +46,9 @@ import {
 	ISummarizerNodeWithGC,
 	SummarizeInternalFn,
 	ITelemetryContext,
-	IIdCompressor,
-	IIdCompressorCore,
-	VisibilityState,
 	ISummaryTreeWithStats,
+	IDataStore,
+	gcDataBlobKey,
 } from "@fluidframework/runtime-definitions";
 import { addBlobToSummary } from "@fluidframework/runtime-utils";
 import {
@@ -61,7 +57,6 @@ import {
 	DataProcessingError,
 	extractSafePropertiesFromMessage,
 	generateStack,
-	ITelemetryLoggerExt,
 	LoggingError,
 	MonitoringContext,
 	tagCodeArtifacts,
@@ -76,9 +71,8 @@ import {
 	getAttributesFormatVersion,
 	getFluidDataStoreAttributes,
 	summarizerClientType,
-} from "./summary";
-import { ContainerRuntime } from "./containerRuntime";
-import { detectOutboundRoutesViaDDSKey, sendGCUnexpectedUsageEvent } from "./gc";
+} from "./summary/index.js";
+import { detectOutboundRoutesViaDDSKey, sendGCUnexpectedUsageEvent } from "./gc/index.js";
 
 function createAttributes(
 	pkg: readonly string[],
@@ -100,21 +94,18 @@ interface ISnapshotDetails {
 	pkg: readonly string[];
 	isRootDataStore: boolean;
 	snapshot?: ISnapshotTree;
-}
-
-interface FluidDataStoreMessage {
-	content: any;
-	type: string;
+	sequenceNumber?: number;
 }
 
 /** Properties necessary for creating a FluidDataStoreContext */
 export interface IFluidDataStoreContextProps {
 	readonly id: string;
-	readonly runtime: ContainerRuntime;
+	readonly parentContext: IFluidParentContext;
 	readonly storage: IDocumentStorageService;
 	readonly scope: FluidObject;
 	readonly createSummarizerNodeFn: CreateChildSummarizerNodeFn;
 	readonly pkg?: Readonly<string[]>;
+	readonly loadingGroupId?: string;
 }
 
 /** Properties necessary for creating a local FluidDataStoreContext */
@@ -129,6 +120,11 @@ export interface ILocalFluidDataStoreContextProps extends IFluidDataStoreContext
 	readonly createProps?: any;
 }
 
+/** Properties necessary for creating a local FluidDataStoreContext */
+export interface ILocalDetachedFluidDataStoreContextProps extends ILocalFluidDataStoreContextProps {
+	readonly channelToDataStoreFn: (channel: IFluidDataStoreChannel) => IDataStore;
+}
+
 /** Properties necessary for creating a remote FluidDataStoreContext */
 export interface IRemoteFluidDataStoreContextProps extends IFluidDataStoreContextProps {
 	readonly snapshotTree: ISnapshotTree | undefined;
@@ -139,47 +135,47 @@ export interface IRemoteFluidDataStoreContextProps extends IFluidDataStoreContex
  */
 export abstract class FluidDataStoreContext
 	extends TypedEventEmitter<IFluidDataStoreContextEvents>
-	implements IFluidDataStoreContext, IDisposable
+	implements IFluidDataStoreContext, IFluidParentContext, IDisposable
 {
 	public get packagePath(): readonly string[] {
 		assert(this.pkg !== undefined, 0x139 /* "Undefined package path" */);
 		return this.pkg;
 	}
 
-	public get options(): ILoaderOptions {
-		return this._containerRuntime.options;
+	public get options(): Record<string | number, any> {
+		return this.parentContext.options;
 	}
 
 	public get clientId(): string | undefined {
-		return this._containerRuntime.clientId;
+		return this.parentContext.clientId;
 	}
 
 	public get clientDetails(): IClientDetails {
-		return this._containerRuntime.clientDetails;
+		return this.parentContext.clientDetails;
 	}
 
-	public get logger(): ITelemetryLoggerExt {
-		return this._containerRuntime.logger;
+	public get logger() {
+		return this.parentContext.logger;
 	}
 
 	public get deltaManager(): IDeltaManager<ISequencedDocumentMessage, IDocumentMessage> {
-		return this._containerRuntime.deltaManager;
+		return this.parentContext.deltaManager;
 	}
 
 	public get connected(): boolean {
-		return this._containerRuntime.connected;
+		return this.parentContext.connected;
 	}
 
 	public get IFluidHandleContext() {
-		return this._containerRuntime.IFluidHandleContext;
+		return this.parentContext.IFluidHandleContext;
 	}
 
-	public get containerRuntime(): IContainerRuntime {
+	public get containerRuntime(): IContainerRuntimeBase {
 		return this._containerRuntime;
 	}
 
 	public ensureNoDataModelChanges<T>(callback: () => T): T {
-		return this._containerRuntime.ensureNoDataModelChanges(callback);
+		return this.parentContext.ensureNoDataModelChanges(callback);
 	}
 
 	public get isLoaded(): boolean {
@@ -190,8 +186,8 @@ export abstract class FluidDataStoreContext
 		return this._baseSnapshot;
 	}
 
-	public get idCompressor(): (IIdCompressorCore & IIdCompressor) | undefined {
-		return this._containerRuntime.idCompressor;
+	public get idCompressor(): IIdCompressor | undefined {
+		return this.parentContext.idCompressor;
 	}
 
 	private _disposed = false;
@@ -208,7 +204,8 @@ export abstract class FluidDataStoreContext
 		return this._tombstoned;
 	}
 	/** If true, throw an error when a tombstone data store is used. */
-	private readonly throwOnTombstoneUsage: boolean;
+	public readonly gcThrowOnTombstoneUsage: boolean;
+	public readonly gcTombstoneEnforcementAllowed: boolean;
 
 	/** If true, this means that this data store context and its children have been removed from the runtime */
 	private deleted: boolean = false;
@@ -218,9 +215,15 @@ export abstract class FluidDataStoreContext
 	}
 
 	public get IFluidDataStoreRegistry(): IFluidDataStoreRegistry | undefined {
+		assert(
+			this.channel !== undefined,
+			0x8f3 /* This should be called after the channel is created, when the registry is populated */,
+		);
 		return this.registry;
 	}
 
+	private baseSnapshotSequenceNumber: number | undefined;
+
 	/**
 	 * A datastore is considered as root if it
 	 * 1. is root in memory - see isInMemoryRoot
@@ -248,8 +251,8 @@ export abstract class FluidDataStoreContext
 	protected channel: IFluidDataStoreChannel | undefined;
 	private loaded = false;
 	protected pending: ISequencedDocumentMessage[] | undefined = [];
-	protected channelDeferred: Deferred<IFluidDataStoreChannel> | undefined;
-	private _baseSnapshot: ISnapshotTree | undefined;
+	protected channelP: Promise<IFluidDataStoreChannel> | undefined;
+	protected _baseSnapshot: ISnapshotTree | undefined;
 	protected _attachState: AttachState;
 	private _isInMemoryRoot: boolean = false;
 	protected readonly summarizerNode: ISummarizerNodeWithGC;
@@ -269,9 +272,12 @@ export abstract class FluidDataStoreContext
 	private lastUsedRoutes: string[] | undefined;
 
 	public readonly id: string;
-	private readonly _containerRuntime: ContainerRuntime;
+	private readonly _containerRuntime: IContainerRuntimeBase;
+	private readonly parentContext: IFluidParentContext;
 	public readonly storage: IDocumentStorageService;
 	public readonly scope: FluidObject;
+	// Represents the group to which the data store belongs too.
+	public readonly loadingGroupId: string | undefined;
 	protected pkg?: readonly string[];
 
 	constructor(
@@ -282,19 +288,21 @@ export abstract class FluidDataStoreContext
 	) {
 		super();
 
-		this._containerRuntime = props.runtime;
+		this._containerRuntime = props.parentContext.containerRuntime;
+		this.parentContext = props.parentContext;
 		this.id = props.id;
 		this.storage = props.storage;
 		this.scope = props.scope;
 		this.pkg = props.pkg;
+		this.loadingGroupId = props.loadingGroupId;
 
 		// URIs use slashes as delimiters. Handles use URIs.
 		// Thus having slashes in types almost guarantees trouble down the road!
 		assert(!this.id.includes("/"), 0x13a /* Data store ID contains slash */);
 
 		this._attachState =
-			this.containerRuntime.attachState !== AttachState.Detached && this.existing
-				? this.containerRuntime.attachState
+			this.parentContext.attachState !== AttachState.Detached && this.existing
+				? this.parentContext.attachState
 				: AttachState.Detached;
 
 		const thisSummarizeInternal = async (
@@ -325,7 +333,8 @@ export abstract class FluidDataStoreContext
 			this.mc.logger,
 		);
 
-		this.throwOnTombstoneUsage = this._containerRuntime.gcThrowOnTombstoneUsage;
+		this.gcThrowOnTombstoneUsage = this.parentContext.gcThrowOnTombstoneUsage;
+		this.gcTombstoneEnforcementAllowed = this.parentContext.gcTombstoneEnforcementAllowed;
 
 		// By default, a data store can log maximum 10 local changes telemetry in summarizer.
 		this.localChangesTelemetryCount =
@@ -339,9 +348,9 @@ export abstract class FluidDataStoreContext
 		this._disposed = true;
 
 		// Dispose any pending runtime after it gets fulfilled
-		// Errors are logged where this.channelDeferred is consumed/generated (realizeCore(), bindRuntime())
-		if (this.channelDeferred) {
-			this.channelDeferred.promise
+		// Errors are logged where this.channelP is consumed/generated (realizeCore(), bindRuntime())
+		if (this.channelP) {
+			this.channelP
 				.then((runtime) => {
 					runtime.dispose();
 				})
@@ -382,9 +391,8 @@ export abstract class FluidDataStoreContext
 
 	public async realize(): Promise<IFluidDataStoreChannel> {
 		assert(!this.detachedRuntimeCreation, 0x13d /* "Detached runtime creation on realize()" */);
-		if (!this.channelDeferred) {
-			this.channelDeferred = new Deferred<IFluidDataStoreChannel>();
-			this.realizeCore(this.existing).catch((error) => {
+		if (!this.channelP) {
+			this.channelP = this.realizeCore(this.existing).catch((error) => {
 				const errorWrapped = DataProcessingError.wrapIfUnrecognized(
 					error,
 					"realizeFluidDataStoreContext",
@@ -395,22 +403,22 @@ export abstract class FluidDataStoreContext
 						fluidDataStoreId: this.id,
 					}),
 				);
-				this.channelDeferred?.reject(errorWrapped);
 				this.mc.logger.sendErrorEvent({ eventName: "RealizeError" }, errorWrapped);
+				throw errorWrapped;
 			});
 		}
-		return this.channelDeferred.promise;
+		return this.channelP;
 	}
 
-	protected async factoryFromPackagePath(packages?: readonly string[]) {
-		assert(this.pkg === packages, 0x13e /* "Unexpected package path" */);
+	protected async factoryFromPackagePath() {
+		const packages = this.pkg;
 		if (packages === undefined) {
 			this.rejectDeferredRealize("packages is undefined");
 		}
 
 		let entry: FluidDataStoreRegistryEntry | undefined;
 		let registry: IFluidDataStoreRegistry | undefined =
-			this._containerRuntime.IFluidDataStoreRegistry;
+			this.parentContext.IFluidDataStoreRegistry;
 		let lastPkg: string | undefined;
 		for (const pkg of packages) {
 			if (!registry) {
@@ -432,33 +440,33 @@ export abstract class FluidDataStoreContext
 			this.rejectDeferredRealize("Can't find factory for package", lastPkg, packages);
 		}
 
-		return { factory, registry };
+		assert(this.registry === undefined, 0x157 /* "datastore registry already attached" */);
+		this.registry = registry;
+
+		return factory;
 	}
 
-	private async realizeCore(existing: boolean): Promise<void> {
+	private async realizeCore(existing: boolean) {
 		const details = await this.getInitialSnapshotDetails();
 		// Base snapshot is the baseline where pending ops are applied to.
 		// It is important that this be in sync with the pending ops, and also
 		// that it is set here, before bindRuntime is called.
 		this._baseSnapshot = details.snapshot;
-		const packages = details.pkg;
+		this.baseSnapshotSequenceNumber = details.sequenceNumber;
+		assert(this.pkg === details.pkg, 0x13e /* "Unexpected package path" */);
 
-		const { factory, registry } = await this.factoryFromPackagePath(packages);
-
-		assert(
-			this.registry === undefined,
-			0x13f /* "datastore context registry is already set" */,
-		);
-		this.registry = registry;
+		const factory = await this.factoryFromPackagePath();
 
 		const channel = await factory.instantiateDataStore(this, existing);
 		assert(channel !== undefined, 0x140 /* "undefined channel on datastore context" */);
-		this.bindRuntime(channel);
+		await this.bindRuntime(channel, existing);
 		// This data store may have been disposed before the channel is created during realization. If so,
 		// dispose the channel now.
 		if (this.disposed) {
 			channel.dispose();
 		}
+
+		return channel;
 	}
 
 	/**
@@ -483,28 +491,21 @@ export abstract class FluidDataStoreContext
 	}
 
 	public process(
-		messageArg: ISequencedDocumentMessage,
+		message: ISequencedDocumentMessage,
 		local: boolean,
 		localOpMetadata: unknown,
 	): void {
-		const safeTelemetryProps = extractSafePropertiesFromMessage(messageArg);
+		const safeTelemetryProps = extractSafePropertiesFromMessage(message);
 		// On op process, tombstone error is logged in garbage collector. So, set "checkTombstone" to false when calling
 		// "verifyNotClosed" which logs tombstone errors. Throw error if tombstoned and throwing on load is configured.
 		this.verifyNotClosed("process", false /* checkTombstone */, safeTelemetryProps);
-		if (this.tombstoned && this.throwOnTombstoneUsage) {
+		if (this.tombstoned && this.gcThrowOnTombstoneUsage) {
 			throw new DataCorruptionError(
 				"Context is tombstoned! Call site [process]",
 				safeTelemetryProps,
 			);
 		}
 
-		const innerContents = messageArg.contents as FluidDataStoreMessage;
-		const message = {
-			...messageArg,
-			type: innerContents.type,
-			contents: innerContents.content,
-		};
-
 		this.summarizerNode.recordChange(message);
 
 		if (this.loaded) {
@@ -529,11 +530,11 @@ export abstract class FluidDataStoreContext
 	}
 
 	public getQuorum(): IQuorumClients {
-		return this._containerRuntime.getQuorum();
+		return this.parentContext.getQuorum();
 	}
 
 	public getAudience(): IAudience {
-		return this._containerRuntime.getAudience();
+		return this.parentContext.getAudience();
 	}
 
 	/**
@@ -581,6 +582,11 @@ export abstract class FluidDataStoreContext
 			summarizeResult.stats.unreferencedBlobSize = summarizeResult.stats.totalBlobSize;
 		}
 
+		// Add loadingGroupId to the summary
+		if (this.loadingGroupId !== undefined) {
+			summarizeResult.summary.groupId = this.loadingGroupId;
+		}
+
 		return {
 			...summarizeResult,
 			id: this.id,
@@ -664,7 +670,7 @@ export abstract class FluidDataStoreContext
 		// By default, skip this call since the ContainerRuntime will detect the outbound route directly.
 		if (this.mc.config.getBoolean(detectOutboundRoutesViaDDSKey) === true) {
 			// Note: The ContainerRuntime code will check this same setting to avoid double counting.
-			this._containerRuntime.addedGCOutboundReference(srcHandle, outboundHandle);
+			this.parentContext.addedGCOutboundReference?.(srcHandle, outboundHandle);
 		}
 	}
 
@@ -678,7 +684,7 @@ export abstract class FluidDataStoreContext
 	 * @param toPath - The absolute path of the outbound node that is referenced.
 	 */
 	public addedGCOutboundRoute(fromPath: string, toPath: string) {
-		this._containerRuntime.addedGCOutboundReference(
+		this.parentContext.addedGCOutboundReference?.(
 			{ absolutePath: fromPath },
 			{ absolutePath: toPath },
 		);
@@ -720,15 +726,10 @@ export abstract class FluidDataStoreContext
 	public submitMessage(type: string, content: any, localOpMetadata: unknown): void {
 		this.verifyNotClosed("submitMessage");
 		assert(!!this.channel, 0x146 /* "Channel must exist when submitting message" */);
-		const fluidDataStoreContent: FluidDataStoreMessage = {
-			content,
-			type,
-		};
-
 		// Summarizer clients should not submit messages.
 		this.identifyLocalChangeInSummarizer("DataStoreMessageSubmittedInSummarizer", type);
 
-		this._containerRuntime.submitDataStoreOp(this.id, fluidDataStoreContent, localOpMetadata);
+		this.parentContext.submitMessage(type, content, localOpMetadata);
 	}
 
 	/**
@@ -765,7 +766,7 @@ export abstract class FluidDataStoreContext
 		this.verifyNotClosed("submitSignal");
 
 		assert(!!this.channel, 0x147 /* "Channel must exist on submitting signal" */);
-		return this._containerRuntime.submitDataStoreSignal(this.id, type, content, targetClientId);
+		return this.parentContext.submitSignal(type, content, targetClientId);
 	}
 
 	/**
@@ -774,72 +775,71 @@ export abstract class FluidDataStoreContext
 	 */
 	public makeLocallyVisible() {
 		assert(this.channel !== undefined, 0x2cf /* "undefined channel on datastore context" */);
-		assert(
-			this.channel.visibilityState === VisibilityState.LocallyVisible,
-			0x590 /* Channel must be locally visible */,
-		);
 		this.makeLocallyVisibleFn();
 	}
 
-	protected bindRuntime(channel: IFluidDataStoreChannel) {
+	protected async bindRuntime(channel: IFluidDataStoreChannel, existing: boolean) {
 		if (this.channel) {
 			throw new Error("Runtime already bound");
 		}
 
-		try {
-			assert(
-				!this.detachedRuntimeCreation,
-				0x148 /* "Detached runtime creation on runtime bind" */,
-			);
-			assert(this.channelDeferred !== undefined, 0x149 /* "Undefined channel deferral" */);
-			assert(this.pkg !== undefined, 0x14a /* "Undefined package path" */);
+		assert(
+			!this.detachedRuntimeCreation,
+			0x148 /* "Detached runtime creation on runtime bind" */,
+		);
+		assert(this.pkg !== undefined, 0x14a /* "Undefined package path" */);
 
+		if (existing) {
 			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
 			const pending = this.pending!;
 
 			// Apply all pending ops
 			for (const op of pending) {
-				channel.process(op, false, undefined /* localOpMetadata */);
+				// Only process ops whose seq number is greater than snapshot sequence number from which it loaded.
+				const seqNumber = this.baseSnapshotSequenceNumber ?? -1;
+				if (op.sequenceNumber > seqNumber) {
+					channel.process(op, false, undefined /* localOpMetadata */);
+				}
 			}
 
 			this.thresholdOpsCounter.send("ProcessPendingOps", pending.length);
-			this.pending = undefined;
-
-			// And now mark the runtime active
-			this.loaded = true;
-			this.channel = channel;
+		} else {
+			assert(this.pending?.length === 0, 0x8f4 /* no pending ops */);
+
+			// Execute data store's entry point to make sure that for a local (aka detached from container) data store, the
+			// entryPoint initialization function is called before the data store gets attached and potentially connected to
+			// the delta stream, so it gets a chance to do things while the data store is still "purely local".
+			// This preserves the behavior from before we introduced entryPoints, where the instantiateDataStore method
+			// of data store factories tends to construct the data object (at least kick off an async method that returns
+			// it); that code moved to the entryPoint initialization function, so we want to ensure it still executes
+			// before the data store is attached.
+			await channel.entryPoint.get();
+		}
+		this.pending = undefined;
 
-			// Freeze the package path to ensure that someone doesn't modify it when it is
-			// returned in packagePath().
-			Object.freeze(this.pkg);
+		// And now mark the runtime active
+		this.loaded = true;
+		this.channel = channel;
 
-			/**
-			 * Update the used routes of the channel. If GC has run before this data store was realized, we will have
-			 * the used routes saved. So, this will ensure that all the child contexts have up-to-date used routes as
-			 * per the last time GC was run.
-			 * Also, this data store may have been realized during summarize. In that case, the child contexts need to
-			 * have their used routes updated to determine if its needs to summarize again and to add it to the summary.
-			 */
-			this.updateChannelUsedRoutes();
+		// Freeze the package path to ensure that someone doesn't modify it when it is
+		// returned in packagePath().
+		Object.freeze(this.pkg);
 
-			// And notify the pending promise it is now available
-			this.channelDeferred.resolve(this.channel);
-		} catch (error) {
-			this.channelDeferred?.reject(error);
-			this.mc.logger.sendErrorEvent(
-				{
-					eventName: "BindRuntimeError",
-				},
-				error,
-			);
-		}
+		/**
+		 * Update the used routes of the channel. If GC has run before this data store was realized, we will have
+		 * the used routes saved. So, this will ensure that all the child contexts have up-to-date used routes as
+		 * per the last time GC was run.
+		 * Also, this data store may have been realized during summarize. In that case, the child contexts need to
+		 * have their used routes updated to determine if its needs to summarize again and to add it to the summary.
+		 */
+		this.updateChannelUsedRoutes();
 	}
 
 	public async getAbsoluteUrl(relativeUrl: string): Promise<string | undefined> {
 		if (this.attachState !== AttachState.Attached) {
 			return undefined;
 		}
-		return this._containerRuntime.getAbsoluteUrl(relativeUrl);
+		return this.parentContext.getAbsoluteUrl(relativeUrl);
 	}
 
 	/**
@@ -854,7 +854,6 @@ export abstract class FluidDataStoreContext
 	): {
 		attachSummary: ISummaryTreeWithStats;
 		type: string;
-		gcData?: IGarbageCollectionData;
 	};
 
 	public abstract getInitialSnapshotDetails(): Promise<ISnapshotDetails>;
@@ -875,21 +874,19 @@ export abstract class FluidDataStoreContext
 		return {};
 	}
 
-	public reSubmit(contents: any, localOpMetadata: unknown) {
+	public reSubmit(type: string, contents: any, localOpMetadata: unknown) {
 		assert(!!this.channel, 0x14b /* "Channel must exist when resubmitting ops" */);
-		const innerContents = contents as FluidDataStoreMessage;
-		this.channel.reSubmit(innerContents.type, innerContents.content, localOpMetadata);
+		this.channel.reSubmit(type, contents, localOpMetadata);
 	}
 
-	public rollback(contents: any, localOpMetadata: unknown) {
+	public rollback(type: string, contents: any, localOpMetadata: unknown) {
 		if (!this.channel) {
 			throw new Error("Channel must exist when rolling back ops");
 		}
 		if (!this.channel.rollback) {
 			throw new Error("Channel doesn't support rollback");
 		}
-		const innerContents = contents as FluidDataStoreMessage;
-		this.channel.rollback(innerContents.type, innerContents.content, localOpMetadata);
+		this.channel.rollback(type, contents, localOpMetadata);
 	}
 
 	public async applyStashedOp(contents: any): Promise<unknown> {
@@ -903,7 +900,7 @@ export abstract class FluidDataStoreContext
 	private verifyNotClosed(
 		callSite: string,
 		checkTombstone = true,
-		safeTelemetryProps: ITelemetryProperties = {},
+		safeTelemetryProps: ITelemetryBaseProperties = {},
 	) {
 		if (this.deleted) {
 			const messageString = `Context is deleted! Call site [${callSite}]`;
@@ -931,15 +928,14 @@ export abstract class FluidDataStoreContext
 				this.mc,
 				{
 					eventName: "GC_Tombstone_DataStore_Changed",
-					category: this.throwOnTombstoneUsage ? "error" : "generic",
-					gcTombstoneEnforcementAllowed:
-						this._containerRuntime.gcTombstoneEnforcementAllowed,
+					category: this.gcThrowOnTombstoneUsage ? "error" : "generic",
+					gcTombstoneEnforcementAllowed: this.gcTombstoneEnforcementAllowed,
 					callSite,
 				},
 				this.pkg,
 				error,
 			);
-			if (this.throwOnTombstoneUsage) {
+			if (this.gcThrowOnTombstoneUsage) {
 				throw error;
 			}
 		}
@@ -979,37 +975,58 @@ export abstract class FluidDataStoreContext
 				summarizeInternal,
 				id,
 				createParam,
-				// DDS will not create failure summaries
-				{ throwOnFailure: true },
+				undefined /* config */,
 				getGCDataFn,
 			);
 	}
 
+	public deleteChildSummarizerNode(id: string) {
+		this.summarizerNode.deleteChild(id);
+	}
+
 	public async uploadBlob(
 		blob: ArrayBufferLike,
 		signal?: AbortSignal,
 	): Promise<IFluidHandle<ArrayBufferLike>> {
-		return this.containerRuntime.uploadBlob(blob, signal);
+		return this.parentContext.uploadBlob(blob, signal);
 	}
 }
 
 export class RemoteFluidDataStoreContext extends FluidDataStoreContext {
-	private readonly initSnapshotValue: ISnapshotTree | undefined;
+	// Tells whether we need to fetch the snapshot before use. This is to support Data Virtualization.
+	private snapshotFetchRequired: boolean;
+	private readonly runtime: IContainerRuntimeBase;
 
 	constructor(props: IRemoteFluidDataStoreContextProps) {
 		super(props, true /* existing */, false /* isLocalDataStore */, () => {
 			throw new Error("Already attached");
 		});
 
-		this.initSnapshotValue = props.snapshotTree;
-
+		this._baseSnapshot = props.snapshotTree;
+		this.snapshotFetchRequired = !!props.snapshotTree?.omitted;
+		this.runtime = props.parentContext.containerRuntime;
 		if (props.snapshotTree !== undefined) {
 			this.summarizerNode.updateBaseSummaryState(props.snapshotTree);
 		}
 	}
 
 	private readonly initialSnapshotDetailsP = new LazyPromise<ISnapshotDetails>(async () => {
-		let tree = this.initSnapshotValue;
+		// Sequence number of the snapshot.
+		let sequenceNumber: number | undefined;
+		if (this.snapshotFetchRequired) {
+			assert(
+				this.loadingGroupId !== undefined,
+				0x8f5 /* groupId should be present to fetch snapshot */,
+			);
+			const snapshot = await this.runtime.getSnapshotForLoadingGroupId(
+				[this.loadingGroupId],
+				[this.id],
+			);
+			this._baseSnapshot = snapshot.snapshotTree;
+			sequenceNumber = snapshot.sequenceNumber;
+			this.snapshotFetchRequired = false;
+		}
+		let tree = this.baseSnapshot;
 		let isRootDataStore = true;
 
 		if (!!tree && tree.blobs[dataStoreAttributesBlobName] !== undefined) {
@@ -1049,11 +1066,12 @@ export class RemoteFluidDataStoreContext extends FluidDataStoreContext {
 			}
 		}
 
+		assert(this.pkg !== undefined, 0x8f6 /* The datastore context package should be defined */);
 		return {
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			pkg: this.pkg!,
+			pkg: this.pkg,
 			isRootDataStore,
 			snapshot: tree,
+			sequenceNumber,
 		};
 	});
 
@@ -1067,7 +1085,6 @@ export class RemoteFluidDataStoreContext extends FluidDataStoreContext {
 	public getAttachData(includeGCData: boolean): {
 		attachSummary: ISummaryTreeWithStats;
 		type: string;
-		gcData?: IGarbageCollectionData;
 	} {
 		throw new Error("Cannot attach remote store");
 	}
@@ -1128,7 +1145,6 @@ export class LocalFluidDataStoreContextBase extends FluidDataStoreContext {
 	): {
 		attachSummary: ISummaryTreeWithStats;
 		type: string;
-		gcData?: IGarbageCollectionData;
 	} {
 		assert(
 			this.channel !== undefined,
@@ -1147,11 +1163,21 @@ export class LocalFluidDataStoreContextBase extends FluidDataStoreContext {
 		// Add data store's attributes to the summary.
 		const attributes = createAttributes(this.pkg, this.isInMemoryRoot());
 		addBlobToSummary(attachSummary, dataStoreAttributesBlobName, JSON.stringify(attributes));
+		if (includeGCData) {
+			const gcData = this.channel.getAttachGCData?.(telemetryContext);
+			if (gcData !== undefined) {
+				addBlobToSummary(attachSummary, gcDataBlobKey, JSON.stringify(gcData));
+			}
+		}
+
+		// Add loadingGroupId to the summary
+		if (this.loadingGroupId !== undefined) {
+			attachSummary.summary.groupId = this.loadingGroupId;
+		}
 
 		return {
 			attachSummary,
 			type: this.pkg[this.pkg.length - 1],
-			gcData: includeGCData ? this.channel.getAttachGCData?.(telemetryContext) : undefined,
 		};
 	}
 
@@ -1239,43 +1265,47 @@ export class LocalDetachedFluidDataStoreContext
 	extends LocalFluidDataStoreContextBase
 	implements IFluidDataStoreContextDetached
 {
-	constructor(props: ILocalFluidDataStoreContextProps) {
+	constructor(props: ILocalDetachedFluidDataStoreContextProps) {
 		super(props);
 		this.detachedRuntimeCreation = true;
+		this.channelToDataStoreFn = props.channelToDataStoreFn;
 	}
+	private readonly channelToDataStoreFn: (channel: IFluidDataStoreChannel) => IDataStore;
 
 	public async attachRuntime(
 		registry: IProvideFluidDataStoreFactory,
 		dataStoreChannel: IFluidDataStoreChannel,
-	) {
+	): Promise<IDataStore> {
 		assert(this.detachedRuntimeCreation, 0x154 /* "runtime creation is already attached" */);
 		this.detachedRuntimeCreation = false;
 
-		assert(this.channelDeferred === undefined, 0x155 /* "channel deferral is already set" */);
-		this.channelDeferred = new Deferred<IFluidDataStoreChannel>();
+		assert(this.channelP === undefined, 0x155 /* "channel deferral is already set" */);
 
-		const factory = registry.IFluidDataStoreFactory;
+		this.channelP = Promise.resolve()
+			.then(async () => {
+				const factory = registry.IFluidDataStoreFactory;
 
-		const entry = await this.factoryFromPackagePath(this.pkg);
-		assert(entry.factory === factory, 0x156 /* "Unexpected factory for package path" */);
+				const factory2 = await this.factoryFromPackagePath();
+				assert(factory2 === factory, 0x156 /* "Unexpected factory for package path" */);
 
-		assert(this.registry === undefined, 0x157 /* "datastore registry already attached" */);
-		this.registry = entry.registry;
+				await super.bindRuntime(dataStoreChannel, false /* existing */);
 
-		super.bindRuntime(dataStoreChannel);
+				assert(
+					!(await this.isRoot()),
+					0x8f7 /* there are no more createRootDataStore() kind of APIs! */,
+				);
 
-		// Load the handle to the data store's entryPoint to make sure that for a detached data store, the entryPoint
-		// initialization function is called before the data store gets attached and potentially connected to the
-		// delta stream, so it gets a chance to do things while the data store is still "purely local".
-		// This preserves the behavior from before we introduced entryPoints, where the instantiateDataStore method
-		// of data store factories tends to construct the data object (at least kick off an async method that returns
-		// it); that code moved to the entryPoint initialization function, so we want to ensure it still executes
-		// before the data store is attached.
-		await dataStoreChannel.entryPoint.get();
+				return dataStoreChannel;
+			})
+			.catch((error) => {
+				this.mc.logger.sendErrorEvent({ eventName: "AttachRuntimeError" }, error);
+				// The following two lines result in same exception thrown.
+				// But we need to ensure that this.channelDeferred.promise is "observed", as otherwise
+				// out UT reports unhandled exception
+				throw error;
+			});
 
-		if (await this.isRoot()) {
-			dataStoreChannel.makeVisibleAndAttachGraph();
-		}
+		return this.channelToDataStoreFn(await this.channelP);
 	}
 
 	public async getInitialSnapshotDetails(): Promise<ISnapshotDetails> {