Skip to content

Commit d29984a

Browse files
committed
Remove createCustomWriteCheckpoint in favor of
batchCreateCustomWriteCheckpoints.
1 parent 6865479 commit d29984a

File tree

4 files changed

+0
-60
lines changed

4 files changed

+0
-60
lines changed

modules/module-mongodb-storage/src/storage/implementation/MongoWriteCheckpointAPI.ts

-23
Original file line numberDiff line numberDiff line change
@@ -36,29 +36,6 @@ export class MongoWriteCheckpointAPI implements storage.WriteCheckpointAPI {
3636
return batchCreateCustomWriteCheckpoints(this.db, checkpoints);
3737
}
3838

39-
async createCustomWriteCheckpoint(options: storage.CustomWriteCheckpointOptions): Promise<bigint> {
40-
if (this.writeCheckpointMode !== storage.WriteCheckpointMode.CUSTOM) {
41-
throw new framework.errors.ValidationError(
42-
`Creating a custom Write Checkpoint when the current Write Checkpoint mode is set to "${this.writeCheckpointMode}"`
43-
);
44-
}
45-
46-
const { checkpoint, user_id, sync_rules_id } = options;
47-
const doc = await this.db.custom_write_checkpoints.findOneAndUpdate(
48-
{
49-
user_id: user_id,
50-
sync_rules_id
51-
},
52-
{
53-
$set: {
54-
checkpoint
55-
}
56-
},
57-
{ upsert: true, returnDocument: 'after' }
58-
);
59-
return doc!.checkpoint;
60-
}
61-
6239
async createManagedWriteCheckpoint(checkpoint: storage.ManagedWriteCheckpointOptions): Promise<bigint> {
6340
if (this.writeCheckpointMode !== storage.WriteCheckpointMode.MANAGED) {
6441
throw new framework.errors.ValidationError(

modules/module-postgres-storage/src/storage/PostgresSyncRulesStorage.ts

-7
Original file line numberDiff line numberDiff line change
@@ -113,13 +113,6 @@ export class PostgresSyncRulesStorage
113113
);
114114
}
115115

116-
createCustomWriteCheckpoint(checkpoint: storage.BatchedCustomWriteCheckpointOptions): Promise<bigint> {
117-
return this.writeCheckpointAPI.createCustomWriteCheckpoint({
118-
...checkpoint,
119-
sync_rules_id: this.group_id
120-
});
121-
}
122-
123116
lastWriteCheckpoint(filters: storage.SyncStorageLastWriteCheckpointFilters): Promise<bigint | null> {
124117
return this.writeCheckpointAPI.lastWriteCheckpoint({
125118
...filters,

modules/module-postgres-storage/src/storage/checkpoints/PostgresWriteCheckpointAPI.ts

-28
Original file line numberDiff line numberDiff line change
@@ -30,34 +30,6 @@ export class PostgresWriteCheckpointAPI implements storage.WriteCheckpointAPI {
3030
return batchCreateCustomWriteCheckpoints(this.db, checkpoints);
3131
}
3232

33-
async createCustomWriteCheckpoint(options: storage.CustomWriteCheckpointOptions): Promise<bigint> {
34-
if (this.writeCheckpointMode !== storage.WriteCheckpointMode.CUSTOM) {
35-
throw new framework.errors.ValidationError(
36-
`Creating a custom Write Checkpoint when the current Write Checkpoint mode is set to "${this.writeCheckpointMode}"`
37-
);
38-
}
39-
40-
const { checkpoint, user_id, sync_rules_id } = options;
41-
const row = await this.db.sql`
42-
INSERT INTO
43-
custom_write_checkpoints (user_id, write_checkpoint, sync_rules_id)
44-
VALUES
45-
(
46-
${{ type: 'varchar', value: user_id }},
47-
${{ type: 'int8', value: checkpoint }},
48-
${{ type: 'int4', value: sync_rules_id }}
49-
)
50-
ON CONFLICT DO UPDATE
51-
SET
52-
write_checkpoint = EXCLUDED.write_checkpoint
53-
RETURNING
54-
*;
55-
`
56-
.decoded(models.CustomWriteCheckpoint)
57-
.first();
58-
return row!.write_checkpoint;
59-
}
60-
6133
async createManagedWriteCheckpoint(checkpoint: storage.ManagedWriteCheckpointOptions): Promise<bigint> {
6234
if (this.writeCheckpointMode !== storage.WriteCheckpointMode.MANAGED) {
6335
throw new framework.errors.ValidationError(

packages/service-core/src/storage/WriteCheckpointAPI.ts

-2
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,6 @@ export interface BaseWriteCheckpointAPI {
9494
*/
9595
export interface SyncStorageWriteCheckpointAPI extends BaseWriteCheckpointAPI {
9696
batchCreateCustomWriteCheckpoints(checkpoints: BatchedCustomWriteCheckpointOptions[]): Promise<void>;
97-
createCustomWriteCheckpoint(checkpoint: BatchedCustomWriteCheckpointOptions): Promise<bigint>;
9897
lastWriteCheckpoint(filters: SyncStorageLastWriteCheckpointFilters): Promise<bigint | null>;
9998
}
10099

@@ -104,7 +103,6 @@ export interface SyncStorageWriteCheckpointAPI extends BaseWriteCheckpointAPI {
104103
*/
105104
export interface WriteCheckpointAPI extends BaseWriteCheckpointAPI {
106105
batchCreateCustomWriteCheckpoints(checkpoints: CustomWriteCheckpointOptions[]): Promise<void>;
107-
createCustomWriteCheckpoint(checkpoint: CustomWriteCheckpointOptions): Promise<bigint>;
108106
lastWriteCheckpoint(filters: LastWriteCheckpointFilters): Promise<bigint | null>;
109107

110108
watchUserWriteCheckpoint(options: WatchUserWriteCheckpointOptions): AsyncIterable<WriteCheckpointResult>;

0 commit comments

Comments
 (0)