fix: remove (now) useless worker thread support check (#1406)

Co-authored-by: andrew <andrew@emergetools.com>
dherault · May 12, 2022 · 1b2ae00 · 1b2ae00
1 parent ed9d6cd
commit 1b2ae00
Showing 1 changed file with 0 additions and 35 deletions.
diff --git a/src/lambda/handler-runner/HandlerRunner.js b/src/lambda/handler-runner/HandlerRunner.js
@@ -7,7 +7,6 @@ import {
   supportedJava,
   supportedGo,
 } from '../../config/index.js'
-import { satisfiesVersionRange } from '../../utils/index.js'
 
 export default class HandlerRunner {
   #env = null
@@ -101,9 +100,6 @@ export default class HandlerRunner {
       }
 
       if (useWorkerThreads) {
-        // worker threads
-        this._verifyWorkerThreadCompatibility()
-
         const { default: WorkerThreadRunner } = await import(
           './worker-thread-runner/index.js'
         )
@@ -163,37 +159,6 @@ export default class HandlerRunner {
     throw new Error('Unsupported runtime')
   }
 
-  _verifyWorkerThreadCompatibility() {
-    const { node: currentVersion } = process.versions
-    const requiredVersionRange = '>=11.7.0'
-
-    const versionIsSatisfied = satisfiesVersionRange(
-      currentVersion,
-      requiredVersionRange,
-    )
-
-    // we're happy
-    if (!versionIsSatisfied) {
-      if (this.log) {
-        this.log.warning(
-          `"worker threads" require node.js version ${requiredVersionRange}, but found version ${currentVersion}.
-         To use this feature you have to update node.js to a later version.
-        `,
-        )
-      } else {
-        logWarning(
-          `"worker threads" require node.js version ${requiredVersionRange}, but found version ${currentVersion}.
-         To use this feature you have to update node.js to a later version.
-        `,
-        )
-      }
-
-      throw new Error(
-        '"worker threads" are not supported with this node.js version',
-      )
-    }
-  }
-
   // TEMP TODO FIXME
   isDockerRunner() {
     return this.#runner && this.#runner.constructor.name === 'DockerRunner'