From eaa6ad769a240882aa41a0247ee92997b29c05ae Mon Sep 17 00:00:00 2001 From: Michael Date: Mon, 10 Jun 2024 13:12:15 +0000 Subject: [PATCH] fix(server): resource timings where off due to async functionality --- server/src/services/movie/movie-service.ts | 6 +++--- server/src/services/person/person-service.ts | 6 +++--- server/src/services/show/show-service.ts | 6 +++--- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/server/src/services/movie/movie-service.ts b/server/src/services/movie/movie-service.ts index 32860aba..00729d54 100644 --- a/server/src/services/movie/movie-service.ts +++ b/server/src/services/movie/movie-service.ts @@ -54,7 +54,7 @@ export class MovieService { options && Object.keys(options).length ? toQueryString(options) : ''; const cacheName = `movie.${id}${optionsAsString}`; try { - const start = Date.now(); + const start = performance.now(); const result = await this.cacheProvider.wrap( cacheName, async () => { @@ -109,10 +109,10 @@ export class MovieService { ); return None; } - const end = Date.now(); + const end = performance.now(); this.logger.debug( { movieId: id, name: result.title }, - `got movie details in ${end - start}ms`, + `got movie details in ${Math.round(end - start)}ms`, ); return Some(MovieEntity.create(result)); } catch (error) { diff --git a/server/src/services/person/person-service.ts b/server/src/services/person/person-service.ts index 13d93aa4..32c949de 100644 --- a/server/src/services/person/person-service.ts +++ b/server/src/services/person/person-service.ts @@ -45,7 +45,7 @@ export class PersonService { options?: GetDetailsOptions, ): Promise> { try { - const start = Date.now(); + const start = performance.now(); const personResult = await this.personDetailsProvider.getDetails(id); if (!personResult.isOk()) { return None; @@ -68,10 +68,10 @@ export class PersonService { ), ]); } - const end = Date.now(); + const end = performance.now(); this.logger.debug( { personId: id, name: person.name }, - `got person details in ${end - start}ms`, + `got person details in ${Math.round(end - start)}ms`, ); return Some(person); } catch (error) { diff --git a/server/src/services/show/show-service.ts b/server/src/services/show/show-service.ts index 6bdd9c65..cc7a532e 100644 --- a/server/src/services/show/show-service.ts +++ b/server/src/services/show/show-service.ts @@ -54,7 +54,7 @@ export class ShowService { options && Object.keys(options).length ? toQueryString(options) : ''; const cacheName = `show.${id}${optionsAsString}`; try { - const start = Date.now(); + const start = performance.now(); const result = await this.cacheProvider.wrap( cacheName, async () => { @@ -109,10 +109,10 @@ export class ShowService { this.logger.debug({ showId: id }, 'show not found'); return None; } - const end = Date.now(); + const end = performance.now(); this.logger.debug( { showId: id, name: result.title, seasons: result.seasons.length }, - `got show details in ${end - start}ms`, + `got show details in ${Math.round(end - start)}ms`, ); return Some(ShowEntity.create(result)); } catch (error) {