Skip to content

Commit

Permalink
Remove duplicative definition of getGCData (#23519)
Browse files Browse the repository at this point in the history
  • Loading branch information
ChumpChief authored Jan 14, 2025
1 parent cc9a041 commit 6f083c3
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ export interface IFluidSerializer {
// @alpha
export interface ISharedObject<TEvent extends ISharedObjectEvents = ISharedObjectEvents> extends IChannel, IEventProvider<TEvent> {
bindToContext(): void;
getGCData(fullGC?: boolean): IGarbageCollectionData;
}

// @alpha
Expand Down
4 changes: 2 additions & 2 deletions packages/dds/shared-object-base/src/sharedObject.ts
Original file line number Diff line number Diff line change
Expand Up @@ -353,7 +353,7 @@ export abstract class SharedObjectCore<
): Promise<ISummaryTreeWithStats>;

/**
* {@inheritDoc (ISharedObject:interface).getGCData}
* {@inheritDoc @fluidframework/datastore-definitions#(IChannel:interface).getGCData}
*/
public abstract getGCData(fullGC?: boolean): IGarbageCollectionData;

Expand Down Expand Up @@ -766,7 +766,7 @@ export abstract class SharedObject<
}

/**
* {@inheritDoc (ISharedObject:interface).getGCData}
* {@inheritDoc @fluidframework/datastore-definitions#(IChannel:interface).getGCData}
*/
public getGCData(fullGC: boolean = false): IGarbageCollectionData {
// Set _isGCing to true. This flag is used to ensure that we only use SummarySerializer to serialize handles
Expand Down
8 changes: 0 additions & 8 deletions packages/dds/shared-object-base/src/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ import {
} from "@fluidframework/core-interfaces";
import { IChannel } from "@fluidframework/datastore-definitions/internal";
import { ISequencedDocumentMessage } from "@fluidframework/driver-definitions/internal";
import { IGarbageCollectionData } from "@fluidframework/runtime-definitions/internal";

/**
* Events emitted by {@link ISharedObject}.
Expand Down Expand Up @@ -66,11 +65,4 @@ export interface ISharedObject<TEvent extends ISharedObjectEvents = ISharedObjec
* the runtime attaches.
*/
bindToContext(): void;

/**
* Returns the GC data for this shared object. It contains a list of GC nodes that contains references to
* other GC nodes.
* @param fullGC - true to bypass optimizations and force full generation of GC data.
*/
getGCData(fullGC?: boolean): IGarbageCollectionData;
}
Original file line number Diff line number Diff line change
Expand Up @@ -743,7 +743,6 @@ export interface ISharedMapEvents extends ISharedObjectEvents {
// @alpha
export interface ISharedObject<TEvent extends ISharedObjectEvents = ISharedObjectEvents> extends IChannel, IEventProvider<TEvent> {
bindToContext(): void;
getGCData(fullGC?: boolean): IGarbageCollectionData;
}

// @alpha
Expand Down

0 comments on commit 6f083c3

Please sign in to comment.