[identity] Refactor devices data deletion to allow filtering
ClosedPublic
Actions

Authored by bartek on Mar 25 2025, 6:52 AM.

Details

Reviewers

kamil

Commits

rCOMMca6dc08b6f22: [identity] Refactor devices data deletion to allow filtering

Summary

Part of ENG-10413.
Refactored function to delete device data from the identity-devices table.
Added option to exclude certain device IDs, and made it slightly more efficient by reducing number of iterator conversions.

Depends on D14499

Test Plan

Tested on staging along with the rest of the stack.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Mar 25 2025, 6:52 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMar 25 2025, 6:52 AM

bartek added a child revision: D14501: [identity] Skip keyserver during primary logout and restore.Mar 25 2025, 7:01 AM

Harbormaster failed remote builds in B33824: Diff 47539!Mar 25 2025, 7:13 AM

bartek published this revision for review.Mar 25 2025, 7:44 AM

kamil accepted this revision.Mar 25 2025, 11:49 AM

This revision is now accepted and ready to land.Mar 25 2025, 11:49 AM

This revision was landed with ongoing or failed builds.Mar 26 2025, 7:55 AM

Closed by commit rCOMMca6dc08b6f22: [identity] Refactor devices data deletion to allow filtering (authored by bartek). · Explain Why

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMca6dc08b6f22: [identity] Refactor devices data deletion to allow filtering.

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

database/

device_list.rs

53 lines

Diff 47539

View Options

services/identity/src/database/device_list.rs

Show First 20 Lines • Show All 494 Lines • ▼ Show 20 Lines	messageStoreThreads: sqliteQueryExecutor
})),		})),
reports: sqliteQueryExecutor.getAllReports(),		reports: sqliteQueryExecutor.getAllReports(),
users: sqliteQueryExecutor.getAllUsers(),		users: sqliteQueryExecutor.getAllUsers(),
keyservers: sqliteQueryExecutor.getAllKeyservers(),		keyservers: sqliteQueryExecutor.getAllKeyservers(),
communities: sqliteQueryExecutor.getAllCommunities(),		communities: sqliteQueryExecutor.getAllCommunities(),
integrityThreadHashes: sqliteQueryExecutor.getAllIntegrityThreadHashes(),		integrityThreadHashes: sqliteQueryExecutor.getAllIntegrityThreadHashes(),
syncedMetadata: sqliteQueryExecutor.getAllSyncedMetadata(),		syncedMetadata: sqliteQueryExecutor.getAllSyncedMetadata(),
auxUserInfos: sqliteQueryExecutor.getAllAuxUserInfos(),		auxUserInfos: sqliteQueryExecutor.getAllAuxUserInfos(),
		threadActivityEntries: [],
};		};
}		}

export {		export {
processDBStoreOperations,		processDBStoreOperations,
getProcessingStoreOpsExceptionMessage,		getProcessingStoreOpsExceptionMessage,
getClientStoreFromQueryExecutor,		getClientStoreFromQueryExecutor,
};		};

[identity] Refactor devices data deletion to allow filteringClosedPublicActions