Fix interval with 0 time causing high CPU (#2614)

* Fix interval with 0 time causing high CPU * Update changelog * Update fallback check
subquery · Dec 2, 2024 · 8027f34 · 8027f34
1 parent ba33f6b
commit 8027f34
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/packages/query/CHANGELOG.md b/packages/query/CHANGELOG.md
@@ -5,6 +5,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Fixed
+- High CPU usage caused by interval with 0ms (#2614)
 
 ## [2.17.1] - 2024-11-28
 ### Fixed

diff --git a/packages/query/src/graphql/graphql.module.ts b/packages/query/src/graphql/graphql.module.ts
@@ -118,6 +118,8 @@ export class GraphqlModule implements OnModuleInit, OnModuleDestroy {
   }
 
   private setupKeepAlive(pgClient: PoolClient) {
+    const interval = argv['sl-keep-alive-interval'] || 180000;
+    logger.info(`Setup PG Pool keep alive. interval ${interval} ms`);
     setInterval(() => {
       void (async () => {
         try {
@@ -126,7 +128,7 @@ export class GraphqlModule implements OnModuleInit, OnModuleDestroy {
           getLogger('db').error('Schema listener client keep-alive query failed: ', err);
         }
       })();
-    }, this.config.get('sl-keep-alive-interval'));
+    }, interval);
   }
 
   private async createServer() {