aljaz/ditto
1
0
Fork 0
mirror of https://github.com/aljazceru/ditto.git synced 2026-01-02 05:04:28 +01:00
Code Issues Packages Projects Releases Wiki Activity

Remove Sentry from SqliteWorker

Browse Source 
It seems to be destroying CPU performance?
This commit is contained in:
Alex Gleason
2024-02-17 14:34:38 -06:00
parent 491c3f5125
commit 7a38cfbc40
1 changed files with 6 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/workers/sqlite.worker.ts
View File

@@ -1,6 +1,6 @@
/// <reference lib="webworker" />
import { Comlink, type CompiledQuery, Debug, DenoSqlite3, type QueryResult, Sentry } from '@/deps.ts';
import { Comlink, type CompiledQuery, Debug, DenoSqlite3, type QueryResult } from '@/deps.ts';
import '@/sentry.ts';
let db: DenoSqlite3 | undefined;
@@ -13,16 +13,11 @@ export const SqliteWorker = {
executeQuery<R>({ sql, parameters }: CompiledQuery): QueryResult<R> {
if (!db) throw new Error('Database not open');
debug(sql);
const result: QueryResult<R> = Sentry.startSpan({ name: sql, op: 'db.query' }, () => {
return {
rows: db!.prepare(sql).all(...parameters as any[]) as R[],
numAffectedRows: BigInt(db!.changes),
insertId: BigInt(db!.lastInsertRowId),
};
});
return result;
return {
rows: db!.prepare(sql).all(...parameters as any[]) as R[],
numAffectedRows: BigInt(db!.changes),
insertId: BigInt(db!.lastInsertRowId),
};
},
destroy() {
db?.close();