perf(gatsby): Use a Map instead of Object for job id lookups (#20605)

A Map is designed for this work where an object will deoptimize after a handful of different keys.
gatsbyjs · Jan 15, 2020 · 54fb530 · 54fb530
1 parent b1e1d99
commit 54fb530
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/gatsby/src/query/query-runner.js b/packages/gatsby/src/query/query-runner.js
@@ -10,7 +10,7 @@ const pageDataUtil = require(`../utils/page-data`)
 const { getCodeFrame } = require(`./graphql-errors`)
 const { default: errorParser } = require(`./error-parser`)
 
-const resultHashes = {}
+const resultHashes = new Map()
 
 type QueryJob = {
   id: string,
@@ -99,8 +99,8 @@ module.exports = async (graphqlRunner, queryJob: QueryJob) => {
     .update(resultJSON)
     .digest(`base64`)
 
-  if (resultHashes[queryJob.id] !== resultHash) {
-    resultHashes[queryJob.id] = resultHash
+  if (resultHash !== resultHashes.get(queryJob.id)) {
+    resultHashes.set(queryJob.id, resultHash)
 
     if (queryJob.isPage) {
       const publicDir = path.join(program.directory, `public`)